On Mon, 2009-08-03 at 13:34 -0400, Stephen Gallagher wrote:
> 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.

> 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:

Right, that check is actually not really needed anymore.
Attached patch that changes the code so that the useless check is
removed.

Simo.


Attachment: 0001-Fix-search-replies-getting-ignored.patch
Description: application/mbox

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to