From 357323d3312d4f2c6a2bbbea03f2296e3368e45a Mon Sep 17 00:00:00 2001

From: Sumit Bose <sbose@redhat.com>

Date: Tue, 3 Jul 2018 11:31:12 +0200

Subject: [PATCH 08/19] sysdb: sysdb_certmap_add() handle domains more flexible

sysdb_ldb_msg_attr_to_certmap_info() creates an empty list if there are

no domains defined, sysdb_certmap_add() should be able to handle both a

missing or an empty domains list.

Related to https://pagure.io/SSSD/sssd/issue/3500

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>

(cherry picked from commit 06f7005d38d164879b727708feff80004b422f91)

---

 src/db/sysdb_certmap.c | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/db/sysdb_certmap.c b/src/db/sysdb_certmap.c

index e37f1ba830f297137991c7757af9c7c4e17b2813..0bcc54c4d1eb0ede7de088e3e945a48f1549c88c 100644

--- a/src/db/sysdb_certmap.c

+++ b/src/db/sysdb_certmap.c

@@ -131,7 +131,7 @@ static errno_t sysdb_certmap_add(struct sysdb_ctx *sysdb,

         }

     }

-    if (certmap->domains != NULL) {

+    if (certmap->domains != NULL && certmap->domains[0] != NULL) {

         for (c = 0; certmap->domains[c] != NULL; c++);

         el = talloc_zero(tmp_ctx, struct ldb_message_element);

         if (el == NULL) {

-- 

2.14.4