[SSSD] [PATCH] IPA: Handle NULL members in process_members()

Jakub Hrozek jhrozek at redhat.com
Mon Oct 20 20:31:06 UTC 2014


On Tue, Oct 14, 2014 at 05:28:36PM +0200, Lukas Slebodnik wrote:
>    * add another trace debug message before calling function process_members.

I went with this last option.
-------------- next part --------------
>From 511c3f922f4c36302bed5270f019dc336b5e82dc Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Tue, 14 Oct 2014 14:15:25 +0200
Subject: [PATCH] IPA: Handle NULL members in process_members()

---
 src/providers/ipa/ipa_s2n_exop.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/providers/ipa/ipa_s2n_exop.c b/src/providers/ipa/ipa_s2n_exop.c
index 96528816a520b633f1f1caa975dee9b9515621c3..2b6bd632ca4568747d7a4c171c4e09c3ad2df55d 100644
--- a/src/providers/ipa/ipa_s2n_exop.c
+++ b/src/providers/ipa/ipa_s2n_exop.c
@@ -1193,6 +1193,11 @@ static errno_t process_members(struct sss_domain_info *domain,
     struct sss_domain_info *obj_domain;
     struct sss_domain_info *parent_domain;
 
+    if (members == NULL) {
+        DEBUG(SSSDBG_TRACE_INTERNAL, "No members\n");
+        return EOK;
+    }
+
     tmp_ctx = talloc_new(NULL);
     if (tmp_ctx == NULL) {
         DEBUG(SSSDBG_OP_FAILURE, "talloc_new failed.\n");
@@ -1728,6 +1733,7 @@ static errno_t ipa_s2n_save_objects(struct sss_domain_info *dom,
                     goto done;
                 }
             }
+            DEBUG(SSSDBG_TRACE_FUNC, "Processing group %s\n", name);
 
             ret = sysdb_attrs_add_lc_name_alias(attrs->sysdb_attrs, name);
             if (ret != EOK) {
-- 
1.9.3



More information about the sssd-devel mailing list