On Sat, May 09, 2009 at 08:16:05PM +0100, Jeremy Henty wrote:
On Sat, May 09, 2009 at 12:19:17PM -0400, Jorge Arellano Cid wrote:
I've found this:
==9830== 1,420 (352 direct, 1,068 indirect) bytes in 11 blocks are definitely lost in loss record 153 of 192 [...] in www.lacuarta.cl
I can reproduce this with a vanilla Dillo, but the leak disappears if I apply the imgref patch that was suggested to me a while back. (Patch attached.)
Good! Image reference counting looks fishy inside the code. I'll review the patch.
Please check whether the reported errors also happen in the vanilla repo.
OK, I'll back out the image patches and see what the logs show then. (It may take a few days before I can be sure if it's made a difference.) I'd still suggest applying the imgref patch to the mainline as it fixes your leak above. It looks as though DilloImages aren't getting unrefed enough, and the patch fixes that.
I still don't know what kind of valgrind problems are happening in vanilla dillo. This is useful information, but getting logs of a patched dillo is certainly confusing. -- Cheers Jorge.-