From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Fri, 2 Aug 2019 14:43:35 +0000 (-0400)
Subject: Fix: don't wait for initial statedump when 0 session active
X-Git-Tag: v2.9.6~1
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=c41cdc2abcd4cf7311dd0927fb47d1d507d5ccfc;p=lttng-ust.git

Fix: don't wait for initial statedump when 0 session active

commit eb0e6022d5e2 "Fix: wait for initial statedump before proceeding
to the main program"

introduced a regression when an application interacts with a session
daemon which has 0 session active.

An application linked against lttng-ust started with
LTTNG_UST_REGISTER_TIMEOUT=-1 hangs forever.

Fix this by decrementing the semaphore if no statedump was requested
when the registration done command is received.

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

diff --git a/liblttng-ust/lttng-ust-comm.c b/liblttng-ust/lttng-ust-comm.c
index 98f9bdb1..26c2e290 100644
--- a/liblttng-ust/lttng-ust-comm.c
+++ b/liblttng-ust/lttng-ust-comm.c
@@ -637,6 +637,10 @@ int handle_register_done(struct sock_info *sock_info)
 	sock_info->registration_done = 1;
 
 	decrement_sem_count(1);
+	if (!sock_info->statedump_pending) {
+		sock_info->initial_statedump_done = 1;
+		decrement_sem_count(1);
+	}
 
 	return 0;
 }