memory usage [was: Forwarded [bogoutil]]

Matthias Andree matthias.andree at gmx.de
Fri Nov 21 00:47:23 CET 2003


On Thu, 20 Nov 2003, Dan Deward wrote:

> Bogoutil has a memory leak for the -l (load file) option.
> 
> 	token = word_new(buf, len);
> 	ds_read(dsh, token, &data);
> 	data.spamcount += spamcount;
> 	data.goodcount += goodcount;
> 	ds_write(dsh, token, &data);
> 
> There should be a word_free(token); after the ds_write.  Seems it should
> also check the token against MAXTOKENLEN when reading from a file like
> dump.load.inp

Good catches! Thanks for this detailed report, the bugs are fixed now in
the CVS tree.




More information about the bogofilter-dev mailing list