From: Francis Deslauriers <francis.deslauriers@efficios.com>
Date: Thu, 22 Apr 2021 21:24:57 +0000 (-0400)
Subject: Fix: tests: gen-ust-events-ns: Uninitialized argument value
X-Git-Tag: v2.12.4~9
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=032a3d09c1834efc9e4112cce5f5d58082db08ba;p=lttng-tools.git

Fix: tests: gen-ust-events-ns: Uninitialized argument value

If both `if (snprintf(...` of the `get_ns_inum()` function fail, the
function will not uninitialize the `ns_inum` output parameter and still
return 0. Leading to the argument `ns1` of debug_printf() being used
uninitialized.

Reported-by: scan-build.
Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I090e57d5c896fbac7381989e887ae7281697b4de
---

diff --git a/tests/utils/testapp/gen-ust-events-ns/gen-ust-events-ns.c b/tests/utils/testapp/gen-ust-events-ns/gen-ust-events-ns.c
index c5f25dd7b..2a3c7a45f 100644
--- a/tests/utils/testapp/gen-ust-events-ns/gen-ust-events-ns.c
+++ b/tests/utils/testapp/gen-ust-events-ns/gen-ust-events-ns.c
@@ -88,7 +88,7 @@ static void debug_printf(const char *format, ...)
 
 static int get_ns_inum(const char *ns, ino_t *ns_inum)
 {
-	int ret = 0;
+	int ret = -1;
 	struct stat sb;
 	char proc_ns_path[LTTNG_PROC_NS_PATH_MAX];
 
@@ -99,8 +99,7 @@ static int get_ns_inum(const char *ns, ino_t *ns_inum)
 			"/proc/thread-self/ns/%s", ns) >= 0) {
 		if (stat(proc_ns_path, &sb) == 0) {
 			*ns_inum = sb.st_ino;
-		} else {
-			ret = -1;
+			ret = 0;
 		}
 		goto end;
 	}
@@ -109,8 +108,7 @@ static int get_ns_inum(const char *ns, ino_t *ns_inum)
 			"/proc/self/task/%d/%s/net", lttng_gettid(), ns) >= 0) {
 		if (stat(proc_ns_path, &sb) == 0) {
 			*ns_inum = sb.st_ino;
-		} else {
-			ret = -1;
+			ret = 0;
 		}
 		goto end;
 	}