]> xenbits.xensource.com Git - libvirt.git/commitdiff
virXMLParseHelper: Store XML parsing flags in a variable
authorMichal Privoznik <mprivozn@redhat.com>
Sat, 18 Nov 2023 03:17:47 +0000 (04:17 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 21 Nov 2023 13:43:39 +0000 (14:43 +0100)
The virXMLParseHelper() can work in two modes: either it parses a
file or a string. Either way, the same set of flags is specified
in call of corresponding function. Save flags in a local variable
instead.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
src/util/virxml.c

index 4f215a0e5947e7d07dd2827da9ad3fb2354941cd..027cdb97b995346ada18159284771ee23c5590e6 100644 (file)
@@ -1136,6 +1136,7 @@ virXMLParseHelper(int domcode,
     g_autoptr(xmlDoc) xml = NULL;
     xmlNodePtr rootnode;
     const char *docname;
+    const int parseFlags = XML_PARSE_NONET | XML_PARSE_NOWARNING;
 
     if (filename)
         docname = filename;
@@ -1154,13 +1155,9 @@ virXMLParseHelper(int domcode,
     pctxt->sax->error = catchXMLError;
 
     if (filename) {
-        xml = xmlCtxtReadFile(pctxt, filename, NULL,
-                              XML_PARSE_NONET |
-                              XML_PARSE_NOWARNING);
+        xml = xmlCtxtReadFile(pctxt, filename, NULL, parseFlags);
     } else {
-        xml = xmlCtxtReadDoc(pctxt, BAD_CAST xmlStr, url, NULL,
-                             XML_PARSE_NONET |
-                             XML_PARSE_NOWARNING);
+        xml = xmlCtxtReadDoc(pctxt, BAD_CAST xmlStr, url, NULL, parseFlags);
     }
 
     if (!xml) {