From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Wed, 25 Sep 2019 20:55:35 +0000 (-0400)
Subject: Fix: tests: leak of prefix on error to register lttng namespace
X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=0916f272a91b4e8840bc11b4c0efb883c089e16a

Fix: tests: leak of prefix on error to register lttng namespace

prefix should always be free'd in this function.

1402045 Resource leak
The system resource will not be reclaimed and reused, reducing the
future availability of the resource.

In register_lttng_namespace: Leak of memory or pointers to system
resources (CWE-404)

Reported-by: Coverity Scan
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/tests/regression/tools/mi/extract_xml.c b/tests/regression/tools/mi/extract_xml.c
index ddf8c1677..ae07689e2 100644
--- a/tests/regression/tools/mi/extract_xml.c
+++ b/tests/regression/tools/mi/extract_xml.c
@@ -155,8 +155,8 @@ static int register_lttng_namespace(xmlXPathContextPtr xpathCtx)
 	}
 
 	ret = xmlXPathRegisterNs(xpathCtx, prefix, ns);
-	xmlFree(prefix);
 end:
+	xmlFree(prefix);
 	xmlFree(ns);
 	return ret;
 }