From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Tue, 9 Jan 2018 22:00:41 +0000 (-0500)
Subject: Fix: channel lock must be taken to check for pending notifications
X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=94a61469bd430d5906b5c976379dd16e32d73933;ds=sidebyside

Fix: channel lock must be taken to check for pending notifications

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 3d48babb7..f2daf0d50 100644
--- a/src/lib/lttng-ctl/channel.c
+++ b/src/lib/lttng-ctl/channel.c
@@ -215,6 +215,8 @@ lttng_notification_channel_get_next_notification(
 		goto end;
 	}
 
+	pthread_mutex_lock(&channel->lock);
+
 	if (channel->pending_notifications.count) {
 		struct pending_notification *pending_notification;
 
@@ -232,11 +234,9 @@ lttng_notification_channel_get_next_notification(
 		cds_list_del(&pending_notification->node);
 		channel->pending_notifications.count--;
 		free(pending_notification);
-		goto end;
+		goto end_unlock;
 	}
 
-	pthread_mutex_lock(&channel->lock);
-
 	ret = receive_message(channel);
 	if (ret) {
 		status = LTTNG_NOTIFICATION_CHANNEL_STATUS_ERROR;