Re: (ITS#8997) openldap-nssov/back ldap segfault

[hyc@symas.com]

This is a multi-part message in MIME format.
--------------23850F02105CA62A388BCB06
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

Pallissard, Matthew wrote:
>> Actually now that I'm looking more closely at this output; the last line below looks interesting.
>>
>> (I probably should have tacked this on the backtrace)
>>
>> Thread 4 "slapd" received signal SIGSEGV, Segmentation fault.
>> [Switching to Thread 0x7ffff0c7c700 (LWP 341)]
>> nssov_dn2uid (op=0x7ffff0c7b730, ni=0x555555a24420, dn=0x7fffe811eab0, uid=0x7fffe8002450) at passwd.c:137
>> 137     passwd.c: No such file or directory.
>>
> 
> Sorry, that was from the wrong output, see below for the correct snippet.
> 
> Thread 3 "slapd" received signal SIGSEGV, Segmentation fault.
> [Switching to Thread 0x7ffff277f700 (LWP 170)]
> nssov_dn2uid (op=0x7ffff277e730, ni=0x555555a24420, dn=0x7fffe4128430, uid=0x7fffe4002430) at passwd.c:137
> 137     passwd.c: No such file or directory.
> 
> Matt Pallissard
> 
Thanks that was a crucial piece.

Please try this patch.

-- 
  -- Howard Chu
  CTO, Symas Corp.           http://www.symas.com
  Director, Highland Sun     http://highlandsun.com/hyc/
  Chief Architect, OpenLDAP  http://www.openldap.org/project/

--------------23850F02105CA62A388BCB06
Content-Type: text/plain; charset=UTF-8;
 name="dif.txt"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename="dif.txt"

diff --git a/servers/slapd/back-ldap/search.c b/servers/slapd/back-ldap/search.c
index 63345aed22..82e72aa8a7 100644
--- a/servers/slapd/back-ldap/search.c
+++ b/servers/slapd/back-ldap/search.c
@@ -1006,6 +1006,7 @@ retry:
 	e = ldap_first_entry( lc->lc_ld, result );
 	if ( e == NULL ) {
 		/* the entry exists, but it doesn't match the filter? */
+		rc = LDAP_NO_RESULTS_RETURNED;
 		goto cleanup;
 	}
 

--------------23850F02105CA62A388BCB06--