[Freeipa-devel] PATCH: fixes LDAP driver searches

Stephen Gallagher sgallagh at redhat.com
Mon Aug 3 17:34:32 UTC 2009


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

On 08/03/2009 01:18 PM, Simo Sorce wrote:
> Stupid typo was making all searches just vanish in thin air.
> Should work fine with this patch.
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

Nack.

You're testing if(ret==EOK) below the switch statement, but it's not
initialized, and it's only set for
    case LDAP_RES_BIND:
    case LDAP_RES_SEARCH_RESULT:
    case LDAP_RES_MODIFY:
    case LDAP_RES_ADD:
    case LDAP_RES_DELETE:
    case LDAP_RES_MODDN:
    case LDAP_RES_COMPARE:
    case LDAP_RES_EXTENDED:
    case LDAP_RES_INTERMEDIATE:

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkp3H5oACgkQeiVVYja6o6MO2gCglSqE5aHp9WSpT/U26BzzUFz8
0+sAoIz4hnJDYOIu/aj4gNyQLY/LnMH5
=k9em
-----END PGP SIGNATURE-----




More information about the Freeipa-devel mailing list