[SSSD] [PATCHES] Some assorted bugfixes and helpers

Simo Sorce simo at redhat.com
Tue Jun 12 12:07:20 UTC 2012


On Mon, 2012-06-11 at 22:03 -0400, Stephen Gallagher wrote:
> 
> On Jun 11, 2012, at 10:01 PM, Simo Sorce <simo at redhat.com> wrote:
> 
> > On Mon, 2012-06-11 at 21:23 -0400, Stephen Gallagher wrote:
> >> Subject: [PATCH 3/8] LDAP: Fix incorrect switch statement in
> >> sdap_get_initgr_done()
> >> 
> >> SDAP_SCHEMA_AD needs to be calling sdap_initgr_rfc2307bis_recv(),
> >> not sdap_initgr_nested_recv(). By coincidence both recv functions
> >> happened to be identical, but if one or the other changed, this
> >> would break unexpectedly. 
> > 
> > AD does have nested memberships, why shouldn't it use _nested_ ?
> > 
> 
> That function is poorly named. It is the recv function for IPA-only processing.

Then let's rename it please, but IIRC it wasn't meant to be ipa
specific, oh well...

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York




More information about the sssd-devel mailing list