diff options
| author | Simo Sorce <ssorce@redhat.com> | 2009-08-14 13:51:02 -0400 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2009-08-27 09:35:53 -0400 |
| commit | 3a87621a3eeb76ce6ac3b18877874141c2dc102e (patch) | |
| tree | 4a484a2d22e87b784ce407870601a79500f528ff /server/providers/ldap | |
| parent | a2b19835fb7562ccf8f204b770dfbd23bc29aa24 (diff) | |
| download | sssd-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.c | 5 |
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); |