Hello Daniel Veillard,

>  That doesn't seems the right approach, by freeing cur, there you
>change the semantic of the API, that doesn't look good to me. What should
>be done instead is check the return value of xmlAddChild() and if NULL
>then free the node in the caller. I think that call is done in SAX2.c
>at the end of xmlSAX2Reference() . I made that change and the leak is indeed
>fixed, I will commit this to SVN later.

Did you already commit your fix to SVN? I did not see it when I checked the 
last time.

Many thanks,

Ralf  

_______________________________________________
xml mailing list, project page  http://xmlsoft.org/
[email protected]
http://mail.gnome.org/mailman/listinfo/xml

Reply via email to