From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Fri, 5 Oct 2018 16:06:37 +0000 (-0400)
Subject: Remove unnecessary check of output parameter
X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=6aa2fd29e34ccb014f337d6d633b4f8cad50c746

Remove unnecessary check of output parameter

It is not necessary to check for `_notification != NULL` as it
is done at the beginning of the function. Moreover, it confuses
Coverity which warns that `notification` will be leaked if the
output parameter is NULL.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/lib/lttng-ctl/channel.c b/src/lib/lttng-ctl/channel.c
index f2daf0d50..b399ea23b 100644
--- a/src/lib/lttng-ctl/channel.c
+++ b/src/lib/lttng-ctl/channel.c
@@ -264,10 +264,8 @@ lttng_notification_channel_get_next_notification(
 
 end_unlock:
 	pthread_mutex_unlock(&channel->lock);
+	*_notification = notification;
 end:
-	if (_notification) {
-		*_notification = notification;
-	}
 	return status;
 }