Re: [PATCH 1/1] perf/tool: fix read in event parsing

From: Jiri Olsa
Date: Sat Mar 07 2020 - 05:45:46 EST


On Fri, Mar 06, 2020 at 11:31:21PM -0800, Ian Rogers wrote:
> ADD_CONFIG_TERM accesses term->weak, however, in get_config_chgs this
> value is accessed outside of the list_for_each_entry and references
> invalid memory. Add an argument for ADD_CONFIG_TERM for weak and set it
> to false in the get_config_chgs case.
> This bug was cause by clang's address sanitizer and libfuzzer. It can be
> reproduced with a command line of:
> perf stat -a -e i/bs,tsc,L2/o
>
> Signed-off-by: Ian Rogers <irogers@xxxxxxxxxx>

nice catch

Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx>

thanks,
jirka