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

[Daniel Veillard <veillard redhat com>]

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/