Re: [JunkMail] Re: RE24 testing

Quanah Gibson-Mount writes: > Please test RE24 in preparation for 2.4.8. Passes all tests on Linux - Ubuntu/x86_64 and RHEL/i386. CHANGES says: > Fixed ldapmodify exit code init (ITS#5317) ldapsearch, not ldapmodify.

And only half fixed, I asked a question and let the other half wait. The other half looks trivial, though I don't have time to test it now:

--- ldapsearch.c.~1.249.~ 2008-01-09 22:03:44.000000000 +0100 +++ ldapsearch.c 2008-02-12 16:27:04.000000000 +0100 @@ -627,3 +627,3 @@ FILE *fp = NULL; - int rc, i, first; + int rc, rc1, i, first; LDAP *ld = NULL; @@ -992,7 +992,9 @@ } - rc = dosearch( ld, base, scope, filtpattern, line, + rc1 = dosearch( ld, base, scope, filtpattern, line, attrs, attrsonly, NULL, NULL, NULL, -1 ); - if ( rc != 0&& !contoper ) { - break; + if ( rc1 != 0 ) { + rc = rc1; + if ( !contoper ) + break; } > Fixed slapd indexing with component matching (ITS#4112) Disabled the broken code, not fixed it, according to the cvs log. (So the ITS should mention it's been disabled, I guess.)