[PATCH 1/2] perf report: delete evlist when deleting session

From: Riccardo Mancini
Date: Mon Jun 21 2021 - 19:44:15 EST


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 <rickyman7@xxxxxxxxx>
---
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