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.3.0-rc1~38
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=03b4fdcf623dd767bb03bc6d62fa813d360b4269;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 b2faa41b1..636450782 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:
 	{