[libvirt] [PATCH 2/2] XML: Suppress printing "domain.xml" for parse errors on XML strings

Daniel Veillard veillard at redhat.com
Tue Sep 6 07:41:50 UTC 2011


On Mon, Sep 05, 2011 at 02:33:40PM +0200, Peter Krempa wrote:
> Patch e81778d6184f1850a10eb661eb756b50421d5ac4 introduces printing
> file name on XML errors. This corrects the URL string to be NULL and
> therefore to print an error message not containing bogus filename
> "domain.xml".
> 
> NULL is a valid parameter for the file name value, as the only usage is
> for error handlers. Functions touching the value either in libxml2 or in
> libvirt check this parameter for NULL and behave according to it.

 Well, I started that trend, and used that as a hint about what kind
of XML paring might be failing. If you have a better idea on how to
name that temporary "file" (it's actually a memory entity), but I'm
not sure setting it as NULL is such an improvement,

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list