[PATCH 1/2] perf report: delete evlist when deleting session
From: Riccardo Mancini <hidden>
Date: 2021-06-21 23:44:15
Also in:
lkml
Subsystem:
performance events subsystem, the rest · Maintainers:
Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo, Namhyung Kim, Linus Torvalds
ASan reports a memory leak related to session->evlist never being deleted. The evlist member is not deleted in perf_session__delete, so it should be deleted separately. This patch adds the missing deletion in perf-report. Signed-off-by: Riccardo Mancini <redacted> --- tools/perf/builtin-report.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 36f9ccfeb38a..07d7693d44b8 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c@@ -1618,6 +1618,7 @@ int cmd_report(int argc, const char **argv) ret = __cmd_report(&report); if (ret == K_SWITCH_INPUT_DATA || ret == K_RELOAD) { + evlist__delete(session->evlist); perf_session__delete(session); last_key = K_SWITCH_INPUT_DATA; goto repeat;
@@ -1637,6 +1638,7 @@ int cmd_report(int argc, const char **argv) } zstd_fini(&(session->zstd_data)); + evlist__delete(session->evlist); perf_session__delete(session); return ret; }
--
2.31.1