On Thu, Mar 23, 2006 at 12:50:57AM +0000, David Pashley wrote:
> Package: libxml-libxml-perl
> Version: 1.58-3
> Severity: normal
> Tags: patch
> 
> There is a memory leak in libxml-libxml-perl-1.58.  More info at
> http://rt.cpan.org/Public/Bug/Display.html?id=12765
> 
> I've included a patch which fixes this problem.

Thank you very much for your patch, David.


While your patch looks reasonable and correct to me, I must admit that I
don't have too much insights in the XML::LibXML internals. Therefor I
want to ask you if you maybe could provide a test that fails with the
current implementation and works with your fix? I think it would fit
into t/11memory.t pretty well.


Regards,
Flo

-- 
BOFH excuse #244:
Your cat tried to eat the mouse.

Attachment: signature.asc
Description: Digital signature

Reply via email to