Em Wed, Nov 06, 2019 at 03:06:50PM +0100, Jiri Olsa escreveu: > On Wed, Oct 30, 2019 at 03:34:39PM -0700, Ian Rogers wrote: > > Parse event error handling may overwrite one error string with another > > creating memory leaks. Introduce a helper routine that warns about > > multiple error messages as well as avoiding the memory leak. > > > > A reproduction of this problem can be seen with: > > perf stat -e c/c/ > > After this change this produces: > > WARNING: multiple event parsing errors > > event syntax error: 'c/c/' > > \___ unknown term > > > > valid terms: event,filter_rem,filter_opc0,edge,filter_isoc,filter_tid,filter_loc,filter_nc,inv,umask,filter_opc1,tid_en,thresh,filter_all_op,filter_not_nm,filter_state,filter_nm,config,config1,config2,name,period,percore > > Run 'perf list' for a list of valid events > > > > Usage: perf stat [<options>] [<command>] > > > > -e, --event <event> event selector. use 'perf list' to list available events > > > > Signed-off-by: Ian Rogers <irogers@xxxxxxxxxx> > > Acked-by: Jiri Olsa <jolsa@xxxxxxxxxx> Thanks, applied. - Arnaldo