From cb98e7dd0dccc3b79b4f54d64be67f6d548f8933 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Wed, 2 Oct 2019 20:28:39 -0400 Subject: [PATCH] Fix: liblttng-ctl: wrong variable used during argument validation MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Local 'handle' variable is used to check for NULL arguments while the provided argument is named '_handle'. This results in failures to destroy a session. Rename the variable used in the argument check. Signed-off-by: Jérémie Galarneau --- src/lib/lttng-ctl/destruction-handle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/lttng-ctl/destruction-handle.c b/src/lib/lttng-ctl/destruction-handle.c index d2559646f..55e160436 100644 --- a/src/lib/lttng-ctl/destruction-handle.c +++ b/src/lib/lttng-ctl/destruction-handle.c @@ -404,7 +404,7 @@ enum lttng_error_code lttng_destroy_session_ext(const char *session_name, int sessiond_socket = -1; struct lttng_destruction_handle *handle = NULL; - if (!session_name || !handle) { + if (!session_name || !_handle) { ret_code = LTTNG_ERR_INVALID; goto error; } -- 2.34.1