From d2243cc136eada1b871f4767b65c4d56c98ed6ad Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Fri, 7 Nov 2014 18:07:07 -0500 Subject: [PATCH] Fix: Don't report an error when listing a session with no channels MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The LTTng sessiond daemon reports an error when the list_channel command is invoked on a session which only has a metadata channel. The error lies in assuming that a session which has an enabled domains also has (user-visible) channels. The metadata channel should be listed in a future fix. Fixes #855 Signed-off-by: Jérémie Galarneau Conflicts: src/bin/lttng/commands/list.c --- src/bin/lttng-sessiond/cmd.c | 8 ++------ src/bin/lttng-sessiond/main.c | 2 +- src/bin/lttng/commands/list.c | 2 +- 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c index 044e9eefe..79b104012 100644 --- a/src/bin/lttng-sessiond/cmd.c +++ b/src/bin/lttng-sessiond/cmd.c @@ -2404,12 +2404,12 @@ ssize_t cmd_list_channels(int domain, struct ltt_session *session, session->ust_session->domain_global.channels); } DBG3("Number of UST global channels %zd", nb_chan); - if (nb_chan <= 0) { + if (nb_chan < 0) { ret = LTTNG_ERR_UST_CHAN_NOT_FOUND; + goto error; } break; default: - *channels = NULL; ret = LTTNG_ERR_UND; goto error; } @@ -2422,10 +2422,6 @@ ssize_t cmd_list_channels(int domain, struct ltt_session *session, } list_lttng_channels(domain, session, *channels); - } else { - *channels = NULL; - /* Ret value was set in the domain switch case */ - goto error; } return nb_chan; diff --git a/src/bin/lttng-sessiond/main.c b/src/bin/lttng-sessiond/main.c index ad136a3a2..36c99cda1 100644 --- a/src/bin/lttng-sessiond/main.c +++ b/src/bin/lttng-sessiond/main.c @@ -3472,7 +3472,7 @@ skip_domain: case LTTNG_LIST_CHANNELS: { int nb_chan; - struct lttng_channel *channels; + struct lttng_channel *channels = NULL; nb_chan = cmd_list_channels(cmd_ctx->lsm->domain.type, cmd_ctx->session, &channels); diff --git a/src/bin/lttng/commands/list.c b/src/bin/lttng/commands/list.c index f56adf482..a91d30757 100644 --- a/src/bin/lttng/commands/list.c +++ b/src/bin/lttng/commands/list.c @@ -720,7 +720,7 @@ static int list_channels(const char *channel_name) goto error_channels; } - if (channel_name == NULL) { + if (count) { MSG("Channels:\n-------------"); } -- 2.34.1