From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Wed, 14 Nov 2018 22:45:38 +0000 (-0500)
Subject: Fix: channel_ust_create: remove channel on agent error
X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=f86e086cd553c4944c4772dac43cc5f2a6863fc0

Fix: channel_ust_create: remove channel on agent error

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/bin/lttng-sessiond/channel.c b/src/bin/lttng-sessiond/channel.c
index 5aa0cd82b..f19dbec37 100644
--- a/src/bin/lttng-sessiond/channel.c
+++ b/src/bin/lttng-sessiond/channel.c
@@ -342,6 +342,7 @@ int channel_ust_create(struct ltt_ust_session *usess,
 	struct ltt_ust_channel *uchan = NULL;
 	struct lttng_channel *defattr = NULL;
 	enum lttng_domain_type domain = LTTNG_DOMAIN_UST;
+	bool chan_published = false;
 
 	assert(usess);
 
@@ -476,6 +477,7 @@ int channel_ust_create(struct ltt_ust_session *usess,
 	if (strncmp(uchan->name, DEFAULT_METADATA_NAME,
 				sizeof(uchan->name))) {
 		lttng_ht_add_unique_str(usess->domain_global.channels, &uchan->node);
+		chan_published = true;
 	} else {
 		/*
 		 * Copy channel attribute to session if this is metadata so if NO
@@ -495,7 +497,7 @@ int channel_ust_create(struct ltt_ust_session *usess,
 			agt = agent_create(domain);
 			if (!agt) {
 				ret = LTTNG_ERR_NOMEM;
-				goto error_free_chan;
+				goto error_remove_chan;
 			}
 			agent_add(agt, usess->agents);
 		}
@@ -504,11 +506,11 @@ int channel_ust_create(struct ltt_ust_session *usess,
 	channel_attr_destroy(defattr);
 	return LTTNG_OK;
 
+error_remove_chan:
+	if (chan_published) {
+		trace_ust_delete_channel(usess->domain_global.channels, uchan);
+	}
 error_free_chan:
-	/*
-	 * No need to remove the channel from the hash table because at this point
-	 * it was not added hence the direct call and no call_rcu().
-	 */
 	trace_ust_destroy_channel(uchan);
 error:
 	channel_attr_destroy(defattr);