From b816b2a6741cd41512a7102b24f278f8e297b864 Mon Sep 17 00:00:00 2001 From: David Goulet Date: Wed, 22 Aug 2012 12:12:27 -0400 Subject: [PATCH] Fix: remove set subdir call that uses bad ptr When copying a consumer object from the tracing session to a specific domain session, the original consumer of the session was modified by appending the session name and datetime once again. We ended up removing the call since at this stage in the code, the session name and date is already appended if necessary. Reported-by: Signed-off-by: David Goulet --- src/bin/lttng-sessiond/main.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c index 76d185700..a9bb2157e 100644 --- a/src/bin/lttng-sessiond/main.c +++ b/src/bin/lttng-sessiond/main.c @@ -1849,12 +1849,6 @@ static int copy_session_consumer(int domain, struct ltt_session *session) goto error; } - ret = consumer_set_subdir(session->consumer, session->name); - if (ret < 0) { - ret = LTTCOMM_FATAL; - goto error; - } - /* Append correct directory to subdir */ strncat(consumer->subdir, dir_name, sizeof(consumer->subdir) - strlen(consumer->subdir) - 1); -- 2.34.1