[PATCH 10/43] util: xml: Extract XPath evaluation for strings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Extract the internals of virXPathString which evaluate the XPath and
validate that the returned object is a string into a new helper named
'virXPathEvalString'.

The function will be later reused in the number XPath evaluation
functions.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 src/util/virxml.c | 39 ++++++++++++++++++++++++++++++---------
 1 file changed, 30 insertions(+), 9 deletions(-)

diff --git a/src/util/virxml.c b/src/util/virxml.c
index e771471dd7..aec475ccfd 100644
--- a/src/util/virxml.c
+++ b/src/util/virxml.c
@@ -60,6 +60,34 @@ virXMLXPathContextNew(xmlDocPtr xml)
 }


+static xmlXPathObject *
+virXPathEvalString(const char *xpath,
+                   xmlXPathContextPtr ctxt)
+{
+    g_autoptr(xmlXPathObject) obj = NULL;
+
+    if (!xpath) {
+        virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Missing XPath expression"));
+        return NULL;
+    }
+
+    if (!ctxt) {
+        virReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Missing XPath context"));
+        return NULL;
+    }
+
+    if (!(obj = xmlXPathEval(BAD_CAST xpath, ctxt)))
+        return NULL;
+
+    if (obj->type != XPATH_STRING ||
+        !obj->stringval ||
+        obj->stringval[0] == '\0')
+        return NULL;
+
+    return g_steal_pointer(&obj);
+}
+
+
 /**
  * virXPathString:
  * @xpath: the XPath string to evaluate
@@ -76,16 +104,9 @@ virXPathString(const char *xpath,
 {
     g_autoptr(xmlXPathObject) obj = NULL;

-    if ((ctxt == NULL) || (xpath == NULL)) {
-        virReportError(VIR_ERR_INTERNAL_ERROR,
-                       "%s", _("Invalid parameter to virXPathString()"));
+    if (!(obj = virXPathEvalString(xpath, ctxt)))
         return NULL;
-    }
-    obj = xmlXPathEval(BAD_CAST xpath, ctxt);
-    if ((obj == NULL) || (obj->type != XPATH_STRING) ||
-        (obj->stringval == NULL) || (obj->stringval[0] == 0)) {
-        return NULL;
-    }
+
     return g_strdup((char *)obj->stringval);
 }

-- 
2.37.3




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux