Re: [PATCH 2/6] perf mem: Free the allocated sort string

From: Arnaldo Carvalho de Melo
Date: Thu Aug 01 2024 - 11:20:29 EST


On Wed, Jul 31, 2024 at 04:55:01PM -0700, Namhyung Kim wrote:
> The get_sort_order() returns either a new string (from strdup) or NULL
> but it never gets freed.

Applied and added:

Fixes: 2e7f545096f954a9 ("perf mem: Factor out a function to generate sort order")

- Arnaldo

> Signed-off-by: Namhyung Kim <namhyung@xxxxxxxxxx>
> ---
> tools/perf/builtin-mem.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/perf/builtin-mem.c b/tools/perf/builtin-mem.c
> index 863fcd735dae..93413cfcd585 100644
> --- a/tools/perf/builtin-mem.c
> +++ b/tools/perf/builtin-mem.c
> @@ -372,6 +372,7 @@ static int report_events(int argc, const char **argv, struct perf_mem *mem)
> rep_argv[i] = argv[j];
>
> ret = cmd_report(i, rep_argv);
> + free(new_sort_order);
> free(rep_argv);
> return ret;
> }
> --
> 2.46.0.rc1.232.g9752f9e123-goog