[SAMBA4][PATCH] Handle search_options control in LDB

Andrew Bartlett abartlet at samba.org
Tue Feb 20 21:43:09 GMT 2007


On Tue, 2007-02-20 at 08:24 -0500, simo wrote:
> On Tue, 2007-02-20 at 18:00 +1100, Andrew Bartlett wrote:
> > Attached please find a patch to implement the server_side of the
> > search_options control, for the 'phantom root' bit (only).
> 
> Can you also implement the other bit?

I'm not entirely clear on it's behaviour.

> > This also includes code to modify the LDAP server to set this flag on
> > the global catalog port, and tests to verify the behaviour.
> 
> Seems ok for now. I still think we will have to create a separate ldb
> for the global catalog in future, but I am not pushing that right now.
> 
> > I would like comments on the ldb changes, required because I needed a
> > way to handle controls from ejs, and from the LDAP server.
> 
> Why did you put ldb_parse_control_strings() in ldb_private.h ?

I was squeamish about extending the public API.  

> It should be put in ldb.h, the private one is for functions that are
> used internally only. Please move it there. Also please fix the wrong
> comments on the OIDs not just remove them.
> 
> Over all I like the patch, please fix details and commit it.

OK, thanks.

Andrew Bartlett
-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Red Hat Inc.                  http://redhat.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20070221/2793a0ec/attachment.bin


More information about the samba-technical mailing list