[virt-tools-list] [libosinfo PATCH] Fix libxml error reporting

Christophe Fergeau cfergeau at redhat.com
Fri Nov 25 09:32:41 UTC 2011


On Fri, Nov 25, 2011 at 09:30:27AM +0100, Guido Günther wrote:
> On Fri, Nov 18, 2011 at 11:03:41PM +0100, Guido Günther wrote:
> > The error code might be set in catchXMLError so make sure we don't miss
> > it. Otherwise this results in errors like:
> > 
> > /usr/lib/python2.7/dist-packages/gi/types.py:43: Warning: GError set over the top of a previous GError or uninitialized memory.
> > This indicates a bug in someone's code. You must ensure an error is NULL before it's set.
> > The overwriting error message was: Incorrect root element
> 
> Can this be appplied?

ACK, do you have commit access or should I push it. Fwiw, the test should
be if (!xml || (err && *err)) to handle err being NULL when the caller is
not interested in the error, but none of the code in osinfo_loader does
this, so better to fix all of these in one go.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20111125/84f66c89/attachment.sig>


More information about the virt-tools-list mailing list