diff options
| author | Andreas Schwab <schwab@suse.de> | 2022-07-25 18:12:18 +0300 |
|---|---|---|
| committer | Steven Rostedt (Google) <rostedt@goodmis.org> | 2022-08-01 00:03:38 +0300 |
| commit | 4f753c3be52c1d930afc0fe3169baa605dbaf611 (patch) | |
| tree | 8cd193d81bd146364bc1e8696748f3c13877b9b9 /tools/tracing/rtla | |
| parent | c7d8a598c5b1e21a0957f5dec2ef4139d2d1a23a (diff) | |
| download | linux-4f753c3be52c1d930afc0fe3169baa605dbaf611.tar.xz | |
rtla: Fix double free
Avoid double free by making trace_instance_destroy indempotent. When
trace_instance_init fails, it calls trace_instance_destroy, but its only
caller osnoise_destroy_tool calls it again.
Link: https://lkml.kernel.org/r/mvmilnlkyzx.fsf_-_@suse.de
Fixes: 0605bf009f18 ("rtla: Add osnoise tool")
Signed-off-by: Andreas Schwab <schwab@suse.de>
Acked-by: Daniel Bristot de Oliveira <bristot@kernel.org>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Diffstat (limited to 'tools/tracing/rtla')
| -rw-r--r-- | tools/tracing/rtla/src/trace.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tools/tracing/rtla/src/trace.c b/tools/tracing/rtla/src/trace.c index 5784c9f9e570..e1ba6d9f4265 100644 --- a/tools/tracing/rtla/src/trace.c +++ b/tools/tracing/rtla/src/trace.c @@ -134,13 +134,18 @@ void trace_instance_destroy(struct trace_instance *trace) if (trace->inst) { disable_tracer(trace->inst); destroy_instance(trace->inst); + trace->inst = NULL; } - if (trace->seq) + if (trace->seq) { free(trace->seq); + trace->seq = NULL; + } - if (trace->tep) + if (trace->tep) { tep_free(trace->tep); + trace->tep = NULL; + } } /* |