Messages in this thread Patch in this message | | | From | Riccardo Mancini <> | Subject | [PATCH 1/2] perf report: delete evlist when deleting session | Date | Tue, 22 Jun 2021 01:43:17 +0200 |
| |
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@gmail.com> --- 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
| |