From mboxrd@z Thu Jan 1 00:00:00 1970 From: mathieu.desnoyers@efficios.com (Mathieu Desnoyers) Date: Sat, 24 Aug 2013 14:55:10 -0400 Subject: [lttng-dev] [PATCH lttng-tools] Fix: snapshot should fail if no successful snapshot is taken In-Reply-To: <20130822204658.GA4260@Krystal> References: <20130822204658.GA4260@Krystal> Message-ID: <20130824185510.GC13216@Krystal> * Mathieu Desnoyers (mathieu.desnoyers at efficios.com) wrote: > Signed-off-by: Mathieu Desnoyers I notice this patch is merged too. Thanks, Mathieu > --- > diff --git a/src/bin/lttng-sessiond/cmd.c b/src/bin/lttng-sessiond/cmd.c > index 64fa5c0..17dcfc4 100644 > --- a/src/bin/lttng-sessiond/cmd.c > +++ b/src/bin/lttng-sessiond/cmd.c > @@ -2836,6 +2836,8 @@ int cmd_snapshot_record(struct ltt_session *session, > > if (snapshot_success) { > session->snapshot.nb_snapshot++; > + } else { > + ret = LTTNG_ERR_SNAPSHOT_FAIL; > } > > error: > > -- > Mathieu Desnoyers > EfficiOS Inc. > http://www.efficios.com > > _______________________________________________ > lttng-dev mailing list > lttng-dev at lists.lttng.org > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com