I noticed that the current code calls xmlFreeNode on attributes which
looks wrong, surely this should be xmlFreeProp?

        Jon

--- ext/xml/ruby_xml_attr.c~	2007-05-25 00:13:43.000000000 +0100
+++ ext/xml/ruby_xml_attr.c	2007-05-25 00:13:56.000000000 +0100
@@ -14,7 +14,7 @@
       rxa->attr->doc == NULL) {       // No document? (otherwise, freed with doc)
     if (rxa->attr->_private <= (void*)1) {
       // is null or last reference, 
-      xmlFreeNode((xmlNodePtr)rxa->attr);  
+      xmlFreeProp(rxa->attr);
     } else {
       // other pointers remain
       rxa->attr->_private--;    
_______________________________________________
libxml-devel mailing list
libxml-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/libxml-devel

Reply via email to