I also found a bunch of similar leaks, plus one different kind which was 
leaking on every xpath find.  I communicated once with Dan Janowski and sent 
him this set of patches, then never heard back from him again, so not sure what 
happened.  Here are the leaks I fixed, if you have others, please make a diff 
and post them!  I could really use it!

-----Original Message-----
From: [EMAIL PROTECTED] on behalf of Charlie Savage
Sent: Tue 7/1/2008 12:07 AM
To: libxml-devel@rubyforge.org
Subject: Re: [libxml-devel] patch for memory leak
 
Hi Tom,


> I found a memory leak when the value of an attribute is accessed.  Fix 
> is simple, in function ruby_xml_attr_value in ruby_xml_attr.c, need to 
> call xmlFree:
> 
> 356,357c356,360
> 
> <     if (value != NULL)
> 
> <       return(rb_str_new2((const char*)value));
> 
> ---
> 
>>     if (value != NULL) {
> 
>>       VALUE retVal = rb_str_new2((const char*)value);
> 
>>       xmlFree(value);
> 
>>       return(retVal);
> 

Thanks for the patch.  I've applied it, and found about 6 other places 
similar issues were happening.

Charlie



<<winmail.dat>>

_______________________________________________
libxml-devel mailing list
libxml-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/libxml-devel

Reply via email to