[PATCH] Add check for broken LDAP connection callbacks

[Sumit Bose]

Hi,

this patch extends the test for LDAP connection callbacks. There is a
bug in OpenLDAP 2.4.12 which makes ldap_set_option() fail. Now it is
first check if the ldap_conncb structure is available and then if
ldap_set_option() succeeds.

bye,
Sumit

[Jeff Schroeder]

On Fri, Feb 12, 2010 at 3:50 AM, Sumit Bose  wrote:
> Hi,
>
> this patch extends the test for LDAP connection callbacks. There is a
> bug in OpenLDAP 2.4.12 which makes ldap_set_option() fail. Now it is
> first check if the ldap_conncb structure is available and then if
> ldap_set_option() succeeds.

Works like a champ with openldap 2.4.12.

Tested-by: Jeff Schroeder

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/12/2010 08:34 AM, Jeff Schroeder wrote:
> On Fri, Feb 12, 2010 at 3:50 AM, Sumit Bose  wrote:
> > Hi,
> >
> > this patch extends the test for LDAP connection callbacks. There is a
> > bug in OpenLDAP 2.4.12 which makes ldap_set_option() fail. Now it is
> > first check if the ldap_conncb structure is available and then if
> > ldap_set_option() succeeds.
>
> Works like a champ with openldap 2.4.12.
>
> Tested-by: Jeff Schroeder

Ack.

The code looks good as well.

Thanks for testing, Jeff!

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkt1WUMACgkQeiVVYja6o6N45ACfWx4+JaHA6DfinIpwVu/Kt5Kr
> 2qUAoIksE6Qx3vF6n9auVZs6sr/dcEwP
> =Kmv1
> -----END PGP SIGNATURE-----

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/12/2010 08:36 AM, Stephen Gallagher wrote:
> On 02/12/2010 08:34 AM, Jeff Schroeder wrote:
> > On Fri, Feb 12, 2010 at 3:50 AM, Sumit Bose  wrote:
> > > Hi,
> > >
> > > this patch extends the test for LDAP connection callbacks. There is a
> > > bug in OpenLDAP 2.4.12 which makes ldap_set_option() fail. Now it is
> > > first check if the ldap_conncb structure is available and then if
> > > ldap_set_option() succeeds.
> > Works like a champ with openldap 2.4.12.
> > Tested-by: Jeff Schroeder
>
> Ack.
>
> The code looks good as well.
>
> Thanks for testing, Jeff!

Pushed to master.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkt1W54ACgkQeiVVYja6o6Mi4ACgp9hVtFAYzvgp3lIU0AdXfmd7
> rWwAn19EhXSy2o2ZmuPJc1lxbIBpqbcO
> =ocTb
> -----END PGP SIGNATURE-----