From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Fri, 2 Sep 2016 19:20:41 +0000 (-0400)
Subject: Fix: remove unlock in getcpu
X-Git-Tag: v2.9.0-rc1~20
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=40745074477e85361af72620a85ccf7945079d68;p=lttng-ust.git

Fix: remove unlock in getcpu

Causes unbalanced RCU read-side lock if nesting safety net is triggered.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
---

diff --git a/libringbuffer/frontend_api.h b/libringbuffer/frontend_api.h
index 77a26918..93f6760b 100644
--- a/libringbuffer/frontend_api.h
+++ b/libringbuffer/frontend_api.h
@@ -60,7 +60,6 @@ int lib_ring_buffer_get_cpu(const struct lttng_ust_lib_ring_buffer_config *confi
 	if (caa_unlikely(nesting > 4)) {
 		WARN_ON_ONCE(1);
 		URCU_TLS(lib_ring_buffer_nesting)--;
-		rcu_read_unlock();
 		return -EPERM;
 	} else
 		return cpu;