| From 0f16ee3b21dc62324d2604f5484fe423e3bbb862 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Thu, 15 Jul 2021 18:07:08 +0200 |
| Subject: perf test session_topology: Delete session->evlist |
| |
| From: Riccardo Mancini <rickyman7@gmail.com> |
| |
| [ Upstream commit 233f2dc1c284337286f9a64c0152236779a42f6c ] |
| |
| ASan reports a memory leak related to session->evlist while running: |
| |
| # perf test "41: Session topology". |
| |
| When perf_data is in write mode, session->evlist is owned by the caller, |
| which should also take care of deleting it. |
| |
| This patch adds the missing evlist__delete(). |
| |
| Signed-off-by: Riccardo Mancini <rickyman7@gmail.com> |
| Fixes: c84974ed9fb67293 ("perf test: Add entry to test cpu topology") |
| Cc: Ian Rogers <irogers@google.com> |
| Cc: Jiri Olsa <jolsa@redhat.com> |
| Cc: Kan Liang <kan.liang@intel.com> |
| Cc: Mark Rutland <mark.rutland@arm.com> |
| Cc: Namhyung Kim <namhyung@kernel.org> |
| Cc: Peter Zijlstra <peterz@infradead.org> |
| Link: http://lore.kernel.org/lkml/822f741f06eb25250fb60686cf30a35f447e9e91.1626343282.git.rickyman7@gmail.com |
| Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| tools/perf/tests/topology.c | 1 + |
| 1 file changed, 1 insertion(+) |
| |
| diff --git a/tools/perf/tests/topology.c b/tools/perf/tests/topology.c |
| index ec4e3b21b831..b5efe675b321 100644 |
| --- a/tools/perf/tests/topology.c |
| +++ b/tools/perf/tests/topology.c |
| @@ -61,6 +61,7 @@ static int session_write_header(char *path) |
| TEST_ASSERT_VAL("failed to write header", |
| !perf_session__write_header(session, session->evlist, data.file.fd, true)); |
| |
| + evlist__delete(session->evlist); |
| perf_session__delete(session); |
| |
| return 0; |
| -- |
| 2.30.2 |
| |