The following is against the development branch of the tree If a user is using ldap result to obtain search results one message at a time and calls ldap_parse_result on one of those messages, i.e. the message chain does not have an LDAP_RES_SEARCH_RESULT message in the chain, ldap_parse_result returns a LDAP_DECODING_ERROR instead of LDAP_NO_RESULTS_RETURNED. The following patch corrects that problem: --- error.c 2000/07/04 19:49:01 1.33 +++ error.c 2000/07/27 21:06:48 @@ -231,7 +231,7 @@ if(serverctrls != NULL) *serverctrls = NULL; /* Find the next result... */ - for ( lm = r; lm->lm_chain != NULL; lm = lm->lm_chain ) { + for ( lm = r; lm != NULL; lm = lm->lm_chain ) { /* skip over entries and references */ if( lm->lm_msgtype != LDAP_RES_SEARCH_ENTRY && lm->lm_msgtype != LDAP_RES_SEARCH_REFERENCE &&
Thanks, committed, please test. Kurt At 09:19 PM 7/27/00 +0000, vtag@novell.com wrote: >The following is against the development branch of the tree > >If a user is using ldap result to obtain search results one message at a >time >and calls ldap_parse_result on one of those messages, i.e. the message >chain does not have an LDAP_RES_SEARCH_RESULT message >in the chain, ldap_parse_result returns a LDAP_DECODING_ERROR >instead of LDAP_NO_RESULTS_RETURNED. > >The following patch corrects that problem: > >--- error.c 2000/07/04 19:49:01 1.33 >+++ error.c 2000/07/27 21:06:48 >@@ -231,7 +231,7 @@ > if(serverctrls != NULL) *serverctrls = NULL; > > /* Find the next result... */ >- for ( lm = r; lm->lm_chain != NULL; lm = lm->lm_chain ) { >+ for ( lm = r; lm != NULL; lm = lm->lm_chain ) { > /* skip over entries and references */ > if( lm->lm_msgtype != LDAP_RES_SEARCH_ENTRY && > lm->lm_msgtype != LDAP_RES_SEARCH_REFERENCE &&
changed notes changed state Open to Closed moved from Incoming to Development
Fixed.