On Wed, Dec 28, 2005 at 01:28:36PM -0500, Sreeni Nair wrote: > Possibly a bug: > > In the function xmlRegExecPushString2(...) in xmlregexp.c, the last > couple of lines are: > > if (str != buf) > xmlFree(buf); > > Since str is the dynamically allocated entity, I think it should have > been xmlFree(str).
Right, nasty, thanks for spotting this ! Suprizingly this is never exercized by the test suite :-\ > PS: Is this the correct list to raise such flags? Also, other than > posting it here, is there anything that mere users of the library can > do? That's fine. Making a bugzilla entry and or adding patches (contextual diff) help making sure the error is not lost and to know precisely how you think it should be fixed. In that case it's simple enough though. thanks, change applied and commited to CVS ! Daniel -- Daniel Veillard | Red Hat http://redhat.com/ [EMAIL PROTECTED] | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ _______________________________________________ xml mailing list, project page http://xmlsoft.org/ [email protected] http://mail.gnome.org/mailman/listinfo/xml
