Em Tue, Oct 07, 2014 at 05:15:41PM +0900, Namhyung Kim escreveu: > Hi Yasser, > > On Sat, 4 Oct 2014 11:37:57 -0400, Yasser Shalabi wrote: > > Fix for double free bug in tools/perf due to dangling thread_map pointer in > > perf_evlist struct. > > Code path excercised when perf stat -C switch is used but not set and is > > followed by another switch. > > Example: perf stat -C -e. > > Signed-off-by: Yasser Shalabi <[email protected]> > > > > --- > > tools/perf/util/evlist.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c > > index 814e954..09af633 100644 > > --- a/tools/perf/util/evlist.c > > +++ b/tools/perf/util/evlist.c > > @@ -845,6 +845,7 @@ int perf_evlist__create_maps(struct perf_evlist > > *evlist, struct target *target) > > > > out_delete_threads: > > thread_map__delete(evlist->threads); > > + evlist->threads = 0; > > Nitpick - I think NULL is preferred rather than 0.
yeah, I fixed that. > Other than that, > > Acked-by: Namhyung Kim <[email protected]> Thanks, applied! > Thanks, > Namhyung > > > > return -1; > > } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
