[PATCH 2/2] s3-libads: Use the configured LDAP page size.

Uri Simchoni uri at samba.org
Thu Jan 14 05:49:43 UTC 2016



On 01/14/2016 03:12 AM, Jeremy Allison wrote:
> On Wed, Jan 13, 2016 at 04:25:34PM -0800, Justin Maggard wrote:
>> We already allow the user to configure LDAP page size, and use it in pdb_ldap.
>> But then we hard-code the initial LDAP page size value to 1000 in ads_init, so
>> it doesn't take effect there.  So let's use the configured LDAP page size value
>> in ads_init also, which defaults to 1000.
> LGTM for both patches. Can I get a second Team reviewer ?
>
> Jeremy.
>
RB+ me.

I'll push later if none objects (I mean, originally the ldap_page_size() 
seems to have been there for non-AD ldap servers, so changing the 
default may have have implications for non-AD uses).

Uri.




More information about the samba-technical mailing list