On Mon, Oct 13, 2014 at 12:37:59PM +0200, Yegor Yefremov wrote:
> Hi Daniel,
> 
> On Mon, Oct 13, 2014 at 11:30 AM, Daniel Veillard <veill...@redhat.com> wrote:
> >     It is available in the FTP area at
> >
> >   ftp://xmlsoft.org/libxml2/
> >
> > as a signed tarball libxml2-2.9.2-rc1.tar.gz as well as
> > signed rpms built on Fedora.
> >
> >  It seems to work fine for me but more testing including on
> > windows would be appreciated. For example one of the patch
> > changes tries to cope with _vsprintf availability (or not)
> > on some of the Microsoft tool !
> >
> >  Patrick, it seems the NEWS doesn't get rebuild I will have to check
> > why and do this manually.
> >
> >   If everything looks okay I will push 2.9.2 on Thursday,
> >
> >    thanks in advance,
> 
> cppcheck finds following error: Common realloc mistake: 'out' nulled
> but not freed upon failure

  ah another one :) thanks ! Patch looks good, pushed 

> I've attached the patch. It is only compile tested.

  It's actually hard to exercise that code as it requires the memory
  debugging which is usually not activated, you need to configure
with --with-mem-debug to activate that code, but then the resulting
libraries can only be used for debug, as free() instead of xmlFree()
or related libxml2 routines will lead to crash.

  That code helped finding a lot of leaks and memory issues
a decade ago :)



-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veill...@redhat.com  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/
_______________________________________________
xml mailing list, project page  http://xmlsoft.org/
xml@gnome.org
https://mail.gnome.org/mailman/listinfo/xml

Reply via email to