[PATCH] some cleanups for smbldap.c

vl at samba.org vl at samba.org
Wed Apr 19 14:47:28 UTC 2017


On Wed, Apr 19, 2017 at 05:41:48PM +0300, Alexander Bokovoy 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.

Ok, thanks for the confirmation.

> Please do not remove getter/setter for paged results, this is useful.

I think this should be hidden behind an improved smbldap_search
interface in the future.

Volker



More information about the samba-technical mailing list