From: David Goulet <dgoulet@efficios.com>
Date: Fri, 12 Jul 2013 14:23:28 +0000 (-0400)
Subject: Fix: RCU read side lock unbalanced
X-Git-Tag: v2.2.2~6
X-Git-Url: http://git.lttng.org./?a=commitdiff_plain;h=46d0fbbc30ab58c8b70e8689d1a0bfe6872286a1;hp=f0e1c78570c3a9f98489696672e1286fe79091fd;p=lttng-tools.git

Fix: RCU read side lock unbalanced

Signed-off-by: David Goulet <dgoulet@efficios.com>
---

diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c
index bcf349f92..d8571aa02 100644
--- a/src/bin/lttng-sessiond/cmd.c
+++ b/src/bin/lttng-sessiond/cmd.c
@@ -857,6 +857,8 @@ int cmd_enable_channel(struct ltt_session *session,
 
 	DBG("Enabling channel %s for session %s", attr->name, session->name);
 
+	rcu_read_lock();
+
 	/*
 	 * Don't try to enable a channel if the session has been started at
 	 * some point in time before. The tracer does not allow it.
@@ -866,8 +868,6 @@ int cmd_enable_channel(struct ltt_session *session,
 		goto error;
 	}
 
-	rcu_read_lock();
-
 	switch (domain->type) {
 	case LTTNG_DOMAIN_KERNEL:
 	{