Re: PR/50441 CVS commit: src/lib/libc/db/hash

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR lib/50441; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: Manuel Bouyer <bouyer%antioche.eu.org@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, lib-bug-people%netbsd.org@localhost, 
	gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Subject: Re: PR/50441 CVS commit: src/lib/libc/db/hash
Date: Wed, 18 Nov 2015 07:22:30 -0500

 On Nov 18, 12:30pm, bouyer%antioche.eu.org@localhost (Manuel Bouyer) wrote:
 -- Subject: Re: PR/50441 CVS commit: src/lib/libc/db/hash
 
 | > Heh, I know. I just fixed this.
 | 
 | sorry for bein impatient :)
 | 
 | > 
 | > |  Also, with spamassassin, I still get a corrupted/unreadable db file.
 | > |  
 | > |  I could'nt reproduce the corruption without spamassassin (maybe this requires
 | > |  non-ascii keys, which I can't do with a simple shell script).
 | > 
 | > I don't know how to debug that! test please?
 | 
 | With
 | src/lib/libc/db/hash/hash_page.c 1.27 and
 | src/lib/libc/db/hash/hash.c 1.36
 | pulled up to netbsd-7, the problem seems to be fixed.
 | With my shell script test, the db file is 256k instead of 76M for the 64k page
 | test, and spamassassin also produces a file of reasonable size, witout
 | complains.
 | 
 | I've reenabled the bayes filter on my mail server, let see how it works after
 | a few hours.
 
 This db code is complicated and broken. It is a fun brain-teaser problem.
 With the latest patches, one of the unit-tests break. I'll have to track
 that down now :-)
 
 christos