Merged in master and stable-2.11. On Fri, Sep 20, 2019 at 10:47:30AM -0400, Jonathan Rajotte wrote: > Fixes coverity #1405775 > > Signed-off-by: Jonathan Rajotte <jonathan.rajotte-jul...@efficios.com> > --- > src/common/trace-chunk.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/common/trace-chunk.c b/src/common/trace-chunk.c > index 6770bd982..bad993bea 100644 > --- a/src/common/trace-chunk.c > +++ b/src/common/trace-chunk.c > @@ -367,6 +367,8 @@ end: > return new_chunk; > error_unlock: > pthread_mutex_unlock(&source_chunk->lock); > + lttng_trace_chunk_fini(new_chunk); > + free(new_chunk);
Changed this to lttng_trace_chunk_put(), which will drop the only existing reference to `new_chunk` and perform the equivalent operations to what you proposed here. Thanks! Jérémie > return NULL; > } > > -- > 2.17.1 > _______________________________________________ lttng-dev mailing list lttng-dev@lists.lttng.org https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
