[PATCH] some cleanups for smbldap.c

vl at samba.org vl at samba.org
Wed Apr 19 15:16:29 UTC 2017


On Wed, Apr 19, 2017 at 05:41:48PM +0300, Alexander Bokovoy via samba-technical wrote:
> On ke, 19 huhti 2017, vl--- via samba-technical wrote:
> > Hi!
> > 
> > The main focus is to make "struct smbldap_state" private to smbldap.c
> > for better encapsulation.
> > 
> > Review appreciated!
> Looks good to me. This will break ipasam in FreeIPA but I already
> started moving it to own private struct as 'struct ldapsam_privates' is
> not accessible anymore and I neeed anyway to implement some more of a
> backend code that needs own proper private struct. Updating it to use
> your wrappers is not a problem.

I've pushed the "privatize smbldap_struct" now. This renders the
bind_callback inaccessible too. What are you using it for?  Looking at
ipa_sam.c I don't get the full picture.

Volker



More information about the samba-technical mailing list