[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 <sbose(a)redhat.com&gt; 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 <jeffschroeder(a)computer.org&gt;

-- 
Jeff Schroeder

Don't drink and derive, alcohol and analysis don't mix.
http://www.digitalprognosis.com

[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 <sbose(a)redhat.com&gt;
> 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 <jeffschroeder(a)computer.org&gt;
>   
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 <sbose(a)redhat.com&gt; 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 <jeffschroeder(a)computer.org&gt;
>  
>  
>  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-----