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