From 3d678709de990ea765cbc7699719e2862bfccd1e Mon Sep 17 00:00:00 2001 From: Jonathan Rajotte Date: Wed, 26 Jul 2017 10:29:17 -0400 Subject: [PATCH 1/1] Fix: use error code path instead of break when errors happen before execl MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Jonathan Rajotte Signed-off-by: Jérémie Galarneau --- src/bin/lttng-sessiond/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c index 7c51f34a2..485a75134 100644 --- a/src/bin/lttng-sessiond/main.c +++ b/src/bin/lttng-sessiond/main.c @@ -2592,7 +2592,7 @@ static pid_t spawn_consumerd(struct consumer_data *consumer_data) } else { DBG("Could not find any valid consumerd executable"); ret = -EINVAL; - break; + goto error; } DBG("Using kernel consumer at: %s", consumer_to_use); (void) execl(consumer_to_use, -- 2.34.1