Fix: add missing goto error in UST consumer
authorDavid Goulet <dgoulet@efficios.com>
Mon, 29 Apr 2013 21:58:51 +0000 (17:58 -0400)
committerDavid Goulet <dgoulet@efficios.com>
Mon, 29 Apr 2013 21:58:51 +0000 (17:58 -0400)
On error, a channel set to NULL can be used causing a segfault when
receiving the metadata.

Reported-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
src/common/ust-consumer/ust-consumer.c

index 2bbc3f5e4fab1ba7b89fd0820ddf93080229b15d..44a8240b055e55dd798e98874f642f1efc72c414 100644 (file)
@@ -1049,6 +1049,7 @@ int lttng_ustconsumer_recv_cmd(struct lttng_consumer_local_data *ctx,
                if (!channel) {
                        ERR("UST consumer push metadata %" PRIu64 " not found", key);
                        ret_code = LTTNG_ERR_UST_CHAN_NOT_FOUND;
                if (!channel) {
                        ERR("UST consumer push metadata %" PRIu64 " not found", key);
                        ret_code = LTTNG_ERR_UST_CHAN_NOT_FOUND;
+                       goto end_msg_sessiond;
                }
 
                /* Tell session daemon we are ready to receive the metadata. */
                }
 
                /* Tell session daemon we are ready to receive the metadata. */
This page took 0.030483 seconds and 5 git commands to generate.