summaryrefslogtreecommitdiff
path: root/server/providers/ldap
diff options
context:
space:
mode:
authorSimo Sorce <ssorce@redhat.com>2009-08-14 13:51:02 -0400
committerSimo Sorce <ssorce@redhat.com>2009-08-27 09:35:53 -0400
commit3a87621a3eeb76ce6ac3b18877874141c2dc102e (patch)
tree4a484a2d22e87b784ce407870601a79500f528ff /server/providers/ldap
parenta2b19835fb7562ccf8f204b770dfbd23bc29aa24 (diff)
downloadsssd-3a87621a3eeb76ce6ac3b18877874141c2dc102e.tar.gz
sssd-3a87621a3eeb76ce6ac3b18877874141c2dc102e.tar.bz2
sssd-3a87621a3eeb76ce6ac3b18877874141c2dc102e.zip
Always save using member/memberOf
First pass to remove the legacy option and make it just a property of the provider
Diffstat (limited to 'server/providers/ldap')
-rw-r--r--server/providers/ldap/sdap_async.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/server/providers/ldap/sdap_async.c b/server/providers/ldap/sdap_async.c
index 539a0753..57ccd865 100644
--- a/server/providers/ldap/sdap_async.c
+++ b/server/providers/ldap/sdap_async.c
@@ -1488,6 +1488,11 @@ static void sdap_get_groups_done(struct sdap_op *op,
case LDAP_RES_SEARCH_ENTRY:
+ /* FIXME: we should do this in 2 steps:
+ * first save groups, then add all memberships
+ * otherwise nested memberships may go missing
+ * if the member group is added only after the
+ * parent group is added */
subreq = sdap_save_group_send(state, state->ev, state->handle,
state->opts, state->dom,
state->sh, reply);