[virt-tools-list] [libosinfo 2/4] loader: Set GError on osinfo_loader_nodeset failures
Christophe Fergeau
cfergeau at redhat.com
Thu Dec 6 09:53:58 UTC 2012
Most of the code in OsinfoLoader check if osinfo_loader_nodeset
set a GError to see if it failed and never test if it returned -1.
Better to make sure that we actually set the GError when this
function fails.
---
osinfo/osinfo_loader.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c
index 1c27187..606e413 100644
--- a/osinfo/osinfo_loader.c
+++ b/osinfo/osinfo_loader.c
@@ -133,6 +133,8 @@ osinfo_loader_nodeset(const char *xpath,
if (obj == NULL)
return(0);
if (obj->type != XPATH_NODESET) {
+ g_set_error(err, g_quark_from_static_string("libosinfo"), 0,
+ _("Expected a nodeset in XPath query %s"), xpath);
xmlXPathFreeObject(obj);
return (-1);
}
--
1.8.0.1
More information about the virt-tools-list
mailing list