From 2a9f757b25bbd1e8e50304a789c898fd0bf3a168 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Mon, 20 Aug 2018 15:22:53 -0400 Subject: [PATCH] Fix: unchecked return value in sessiond path configuration MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The return values of the functions setting the sessiond's base paths are not checked. Reported-by: Coverity Scan (1382768 and 1382769, Unused value) Signed-off-by: Jérémie Galarneau --- src/bin/lttng-sessiond/sessiond-config.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/bin/lttng-sessiond/sessiond-config.c b/src/bin/lttng-sessiond/sessiond-config.c index 0b5408656..d514aefc7 100644 --- a/src/bin/lttng-sessiond/sessiond-config.c +++ b/src/bin/lttng-sessiond/sessiond-config.c @@ -269,6 +269,9 @@ int sessiond_config_init(struct sessiond_config *config) } else { ret = config_set_paths_non_root(config); } + if (ret < 0) { + goto end; + } /* 32 bits consumerd path setup */ ret = asprintf(&str, DEFAULT_USTCONSUMERD32_PATH, -- 2.34.1