Re: [xml] FW: Linking to xml2.so



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/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]