Fix some XPath relative node resets

* src/util/xml.c: The virXPath... function take extra care to preserve
  the XPath context node (ctxt->node) but in the case of virXPathString
  and virXPathBoolean they forgot to do this on the error path. This
  patch fixes this and move all ctxt->node = relnode instuctions just
  after the xmlXPathEval() to make sure this doesn't happen if this code
  is modified.

diff --git a/src/util/xml.c b/src/util/xml.c
index 575d14e7707820fbd0479872e455267522ea9cbf..4118d2a1dc45c0c96a00c6fc31145b6795cb90e6 100644 (file)
--- a/src/util/xml.c
+++ b/src/util/xml.c
@@ -62,6 +62,7 @@ virXPathString(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj == NULL) || (obj->type != XPATH_STRING) ||
         (obj->stringval == NULL) || (obj->stringval[0] == 0)) {
         xmlXPathFreeObject(obj);
@@ -72,7 +73,6 @@ virXPathString(virConnectPtr conn,
     if (ret == NULL) {
         virReportOOMError(conn);
     }
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -133,16 +133,15 @@ virXPathNumber(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj == NULL) || (obj->type != XPATH_NUMBER) ||
         (isnan(obj->floatval))) {
         xmlXPathFreeObject(obj);
-        ctxt->node = relnode;
         return (-1);
     }
 
     *value = obj->floatval;
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (0);
 }
 
@@ -164,6 +163,7 @@ virXPathLongBase(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj != NULL) && (obj->type == XPATH_STRING) &&
         (obj->stringval != NULL) && (obj->stringval[0] != 0)) {
         char *conv = NULL;
@@ -186,7 +186,6 @@ virXPathLongBase(virConnectPtr conn,
     }
 
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -251,6 +250,7 @@ virXPathULongBase(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj != NULL) && (obj->type == XPATH_STRING) &&
         (obj->stringval != NULL) && (obj->stringval[0] != 0)) {
         char *conv = NULL;
@@ -273,7 +273,6 @@ virXPathULongBase(virConnectPtr conn,
     }
 
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -349,6 +348,7 @@ virXPathULongLong(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj != NULL) && (obj->type == XPATH_STRING) &&
         (obj->stringval != NULL) && (obj->stringval[0] != 0)) {
         char *conv = NULL;
@@ -371,7 +371,6 @@ virXPathULongLong(virConnectPtr conn,
     }
 
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -407,6 +406,7 @@ virXPathBoolean(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj == NULL) || (obj->type != XPATH_BOOLEAN) ||
         (obj->boolval < 0) || (obj->boolval > 1)) {
         xmlXPathFreeObject(obj);
@@ -415,7 +415,6 @@ virXPathBoolean(virConnectPtr conn,
     ret = obj->boolval;
 
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -445,17 +444,16 @@ virXPathNode(virConnectPtr conn,
     }
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj == NULL) || (obj->type != XPATH_NODESET) ||
         (obj->nodesetval == NULL) || (obj->nodesetval->nodeNr <= 0) ||
         (obj->nodesetval->nodeTab == NULL)) {
         xmlXPathFreeObject(obj);
-        ctxt->node = relnode;
         return (NULL);
     }
 
     ret = obj->nodesetval->nodeTab[0];
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }
 
@@ -491,10 +489,10 @@ virXPathNodeSet(virConnectPtr conn,
 
     relnode = ctxt->node;
     obj = xmlXPathEval(BAD_CAST xpath, ctxt);
+    ctxt->node = relnode;
     if ((obj == NULL) || (obj->type != XPATH_NODESET) ||
         (obj->nodesetval == NULL) || (obj->nodesetval->nodeNr < 0)) {
         xmlXPathFreeObject(obj);
-        ctxt->node = relnode;
         return (-1);
     }
 
@@ -509,6 +507,5 @@ virXPathNodeSet(virConnectPtr conn,
         }
     }
     xmlXPathFreeObject(obj);
-    ctxt->node = relnode;
     return (ret);
 }