From 822abc2fe108d5eec368de0f458e8d172a3e5d80 Mon Sep 17 00:00:00 2001 From: Mathieu Desnoyers Date: Tue, 17 May 2016 12:31:23 -0400 Subject: [PATCH] Fix: remove logically dead code in send_channel_uid_to_ust MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Found by Coverity: at_most: At condition ret < 0, the value of ret must be at most -1. cannot_set: At condition ret < 0, the value of ret cannot be equal to any of {-1030, -32}. dead_error_condition: The condition ret < 0 must be true. 2825 } else if (ret < 0) { 2826 goto error_stream_unlock; 2827 } CID 1323135 (#1 of 1): Logically dead code (DEADCODE)dead_error_line: Execution cannot reach this statement: goto error_stream_unlock;. Signed-off-by: Mathieu Desnoyers Signed-off-by: Jérémie Galarneau --- src/bin/lttng-sessiond/ust-app.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/bin/lttng-sessiond/ust-app.c b/src/bin/lttng-sessiond/ust-app.c index 1bb183d5c..17e33a108 100644 --- a/src/bin/lttng-sessiond/ust-app.c +++ b/src/bin/lttng-sessiond/ust-app.c @@ -2821,9 +2821,6 @@ static int send_channel_uid_to_ust(struct buffer_reg_channel *reg_chan, (void) release_ust_app_stream(-1, &stream, app); if (ret == -EPIPE || ret == -LTTNG_UST_ERR_EXITING) { ret = -ENOTCONN; /* Caused by app exiting. */ - goto error_stream_unlock; - } else if (ret < 0) { - goto error_stream_unlock; } goto error_stream_unlock; } -- 2.34.1