Issue 5362 - syncrepl provider kills consumer by sending truncated cookie
Summary: syncrepl provider kills consumer by sending truncated cookie
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: slapd (show other issues)
Version: 2.4.7
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-09 16:25 UTC by Quanah Gibson-Mount
Modified: 2014-08-01 21:04 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Quanah Gibson-Mount 2008-02-09 16:25:08 UTC
Full_Name: Quanah Gibson-Mount
Version: 2.4.7
OS: 
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (24.23.156.219)


As reported in the Debian BTS, and Howard has seen this as well:

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=464024

Comment 1 Howard Chu 2008-02-10 01:08:54 UTC
changed notes
changed state Open to Test
moved from Incoming to Software Bugs
Comment 2 Howard Chu 2008-02-10 05:06:33 UTC
quanah@OpenLDAP.org wrote:
> Full_Name: Quanah Gibson-Mount
> Version: 2.4.7
> OS:
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (24.23.156.219)
>
>
> As reported in the Debian BTS, and Howard has seen this as well:
>
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=464024

I saw a similar problem while investigating ITS#5332. While ldapadd'ing 1000 
entries I saw several instances where a cookie was sent without a CSN. It 
didn't result in a crash though. After sprinkling some assert's in the code I 
think I've identified the problem, and the current code in HEAD no longer 
triggers any of the assert's that I used. But I'm not certain that the 
behavior I was chasing is the same as in this bug report. Please test and 
report your results.
-- 
   -- Howard Chu
   Chief Architect, Symas Corp.  http://www.symas.com
   Director, Highland Sun        http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP     http://www.openldap.org/project/

Comment 3 Howard Chu 2008-02-10 05:12:32 UTC
quanah@OpenLDAP.org wrote:
> Full_Name: Quanah Gibson-Mount
> Version: 2.4.7
> OS:
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (24.23.156.219)
>
>
> As reported in the Debian BTS, and Howard has seen this as well:
>
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=464024

Also, as usual, a GDB stack trace from a slapd with debug symbols would be 
useful. strace is generally useless since it only traces system calls, not 
application code. I wish whoever is telling people to provide strace output 
would quit that.

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

Comment 4 rra@debian.org 2008-02-10 05:22:09 UTC
hyc@symas.com writes:
> quanah@OpenLDAP.org wrote:

>> Full_Name: Quanah Gibson-Mount
>> Version: 2.4.7
>> OS:
>> URL: ftp://ftp.openldap.org/incoming/
>> Submission from: (NULL) (24.23.156.219)

>> As reported in the Debian BTS, and Howard has seen this as well:

>> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=464024

> Also, as usual, a GDB stack trace from a slapd with debug symbols would
> be useful. strace is generally useless since it only traces system
> calls, not application code. I wish whoever is telling people to provide
> strace output would quit that.

Just for reference, if you ever need to get real stack traces from people
who are using Debian packages, tell them to install slapd-dbg first.  That
package contains the detached debugging information for slapd, which gdb
will pick up automatically if it's installed.

(It's still an optimized build, so the resulting backtrace may leave
something to be desired, but it's at least not completely useless.)

-- 
Russ Allbery (rra@stanford.edu)             <http://www.eyrie.org/~eagle/>

Comment 5 Quanah Gibson-Mount 2008-02-12 20:18:55 UTC
changed notes
changed state Test to Release
Comment 6 Quanah Gibson-Mount 2008-02-20 02:35:56 UTC
changed notes
changed state Release to Closed
Comment 7 OpenLDAP project 2014-08-01 21:04:13 UTC
fixed in HEAD
fixed in 2.4.8