X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=tests%2Fregression%2Ftools%2Fmi%2Fextract_xml.c;h=32c57e60a93c3f462069623f994ae2056ecf8e14;hb=51704c8eb4792d8ff748ffdbc2118471a9a67e6c;hp=ae07689e239f4a44168d91b396f4a649fc68d134;hpb=0916f272a91b4e8840bc11b4c0efb883c089e16a;p=lttng-tools.git

diff --git a/tests/regression/tools/mi/extract_xml.c b/tests/regression/tools/mi/extract_xml.c
index ae07689e2..32c57e60a 100644
--- a/tests/regression/tools/mi/extract_xml.c
+++ b/tests/regression/tools/mi/extract_xml.c
@@ -1,17 +1,8 @@
 /*
- * Copyright (C) 2014 - Jonathan Rajotte <jonathan.r.julien@gmail.com>
+ * Copyright (C) 2014 Jonathan Rajotte <jonathan.r.julien@gmail.com>
  *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License, version 2 only, as
- * published by the Free Software Foundation.
+ * SPDX-License-Identifier: GPL-2.0-only
  *
- * This program is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
- * more details.
- *
- * You should have received a copy of the GNU General Public License along with
- * this program; if not, write to the Free Software Foundation, Inc., 51
  */
 
 /*
@@ -33,10 +24,11 @@
  *     node;b;
  *     node;c;
  */
-#include <stdlib.h>
+#include <assert.h>
+#include <stdbool.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
-#include <assert.h>
 #include <unistd.h>
 
 #include <libxml/tree.h>
@@ -47,9 +39,9 @@
 
 #if defined(LIBXML_XPATH_ENABLED)
 
-
-int opt_verbose;
-int node_exist;
+static int opt_verbose;
+static int node_exist;
+static bool result = false;
 
 /**
  * print_xpath_nodes:
@@ -86,7 +78,7 @@ static int print_xpath_nodes(xmlDocPtr doc, xmlNodeSetPtr nodes, FILE *output)
 					node_child_value_string = xmlNodeListGetString(doc,
 							cur->children, 1);
 					if (node_exist) {
-						fprintf(output, "true\n");
+						result = true;
 					} else if (opt_verbose) {
 						fprintf(output, "%s;%s;\n", cur->name,
 								node_child_value_string);
@@ -98,7 +90,7 @@ static int print_xpath_nodes(xmlDocPtr doc, xmlNodeSetPtr nodes, FILE *output)
 				} else {
 					/* We don't want to print non-final element */
 					if (node_exist) {
-						fprintf(output, "true\n");
+						result = true;
 					} else {
 						fprintf(stderr, "ERR:%s\n",
 								"Xpath expression return non-final xml element");
@@ -108,7 +100,7 @@ static int print_xpath_nodes(xmlDocPtr doc, xmlNodeSetPtr nodes, FILE *output)
 				}
 			} else {
 				if (node_exist) {
-					fprintf(output, "true\n");
+					result = true;
 				} else {
 					/* We don't want to print non-final element */
 					fprintf(stderr, "ERR:%s\n",
@@ -121,7 +113,7 @@ static int print_xpath_nodes(xmlDocPtr doc, xmlNodeSetPtr nodes, FILE *output)
 		} else {
 			cur = nodes->nodeTab[i];
 			if (node_exist) {
-				fprintf(output, "true\n");
+				result = true;
 			} else if (opt_verbose) {
 				fprintf(output, "%s;%s;\n", cur->parent->name, cur->content);
 			} else {
@@ -221,6 +213,9 @@ static int extract_xpath(const char *xml_path, const xmlChar *xpath)
 		xmlFreeDoc(doc);
 		return -1;
 	}
+	if (node_exist && result) {
+		fprintf(stdout, "true\n");
+	}
 
 	/* Cleanup */
 	xmlXPathFreeObject(xpathObj);