From ff1a2c3965e5222769364e9b1cbef06bd32066d3 Mon Sep 17 00:00:00 2001 From: David Goulet Date: Mon, 4 Nov 2013 16:27:00 -0500 Subject: [PATCH] Fix: missing LTTNG_OK return value for snapshot After a successful kernel snapshot, the ret = LTTNG_OK was missing thus returning 0 to the caller which was expecting LTTNG_OK. Acked-by: Mathieu Desnoyers Signed-off-by: David Goulet --- src/bin/lttng-sessiond/kernel.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/bin/lttng-sessiond/kernel.c b/src/bin/lttng-sessiond/kernel.c index 5eda07758..531f65488 100644 --- a/src/bin/lttng-sessiond/kernel.c +++ b/src/bin/lttng-sessiond/kernel.c @@ -927,6 +927,8 @@ int kernel_snapshot_record(struct ltt_kernel_session *ksess, (void) kernel_consumer_destroy_metadata(socket, ksess->metadata); } + ret = LTTNG_OK; + error_consumer: /* Close newly opened metadata stream. It's now on the consumer side. */ err = close(ksess->metadata_stream_fd); -- 2.34.1