From: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Date: Wed, 14 Mar 2018 18:54:21 +0000 (-0400)
Subject: Fix: use signed variable for refcounting of consumer_relayd_sock_pair
X-Git-Tag: v2.9.10~5
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=17977dd023f0ee2ac06aabe9cb848ec279eec746;p=lttng-tools.git

Fix: use signed variable for refcounting of consumer_relayd_sock_pair

Otherwise refcount check after decreasing have no meaning as in
consumer_stream_relayd_close function.

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/common/consumer/consumer.h b/src/common/consumer/consumer.h
index e1a4b7e9e..fc4723879 100644
--- a/src/common/consumer/consumer.h
+++ b/src/common/consumer/consumer.h
@@ -423,7 +423,7 @@ struct consumer_relayd_sock_pair {
 	/* Network sequence number. */
 	uint64_t net_seq_idx;
 	/* Number of stream associated with this relayd */
-	unsigned int refcount;
+	int refcount;
 
 	/*
 	 * This flag indicates whether or not we should destroy this object. The