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 ddf8c16773331b9786b2018c4cbf2d7fb5a651e5..ae07689e239f4a44168d91b396f4a649fc68d134 100644 (file)
--- 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);
        }
 
        ret = xmlXPathRegisterNs(xpathCtx, prefix, ns);

-       xmlFree(prefix);

 end:
 end:

+       xmlFree(prefix);

        xmlFree(ns);
        return ret;
 }
        xmlFree(ns);
        return ret;
 }