[SSSD] [PATCH] AD: Don't mark domain as enumerated twice

Jakub Hrozek jhrozek at redhat.com
Mon Dec 16 04:59:13 UTC 2013


Hi,

I found this small bug while testing the other enumeration patches. It
wasn't harmful, just not needed.
-------------- next part --------------
>From 4793f0fb9d8134d1fdfd2a6fd545ee4c3b761808 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Mon, 16 Dec 2013 03:36:19 +0100
Subject: [PATCH] AD: Don't mark domain as enumerated twice

The domain was already marked as enumerated using sysdb_set_enumerated
in the enumeration request itself.
---
 src/providers/ad/ad_id.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c
index 822b078439f3808b5f3fadc9ee17ceee3665df14..a7a74443f009ca6590dbc100d10caa37363cc154 100644
--- a/src/providers/ad/ad_id.c
+++ b/src/providers/ad/ad_id.c
@@ -576,18 +576,6 @@ ad_enumeration_done(struct tevent_req *subreq)
         return;
     }
 
-    /* Ok, we've completed an enumeration. Save this to the
-     * sysdb so we can postpone starting up the enumeration
-     * process on the next SSSD service restart (to avoid
-     * slowing down system boot-up
-     */
-    ret = sysdb_set_enumerated(state->sdom->dom, true);
-    if (ret != EOK) {
-        DEBUG(SSSDBG_MINOR_FAILURE,
-              ("Could not mark domain as having enumerated.\n"));
-        /* This error is non-fatal, so continue */
-    }
-
     tevent_req_done(req);
 }
 
-- 
1.8.4.2



More information about the sssd-devel mailing list