From: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Date: Wed, 26 Jul 2017 14:29:17 +0000 (-0400)
Subject: Fix: use error code path instead of break when errors happen before execl
X-Git-Tag: v2.9.6~5
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=5b49a21f8fcf7a179a86c76669c405b82b6439fb;p=lttng-tools.git

Fix: use error code path instead of break when errors happen before execl

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c
index 44b88f39f..80ca3d7c0 100644
--- a/src/bin/lttng-sessiond/main.c
+++ b/src/bin/lttng-sessiond/main.c
@@ -2526,7 +2526,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);
 			ret = execl(consumer_to_use,