Messages in this thread Patch in this message | | | Date | Fri, 19 Jul 2013 00:45:03 -0700 | From | tip-bot for David Ahern <> | Subject | [tip:perf/core] perf tools: Don' t free list head in parse_events__free_terms |
| |
Commit-ID: 0142dab01cd690f6e376f1fb4d4462beb054dfaf Gitweb: http://git.kernel.org/tip/0142dab01cd690f6e376f1fb4d4462beb054dfaf Author: David Ahern <dsahern@gmail.com> AuthorDate: Tue, 2 Jul 2013 13:27:23 -0600 Committer: Arnaldo Carvalho de Melo <acme@redhat.com> CommitDate: Fri, 12 Jul 2013 13:46:14 -0300
perf tools: Don't free list head in parse_events__free_terms
Function should only be freeing the entries in the list in case of failure, as those were allocated there, not the list_head itself.
Signed-off-by: David Ahern <dsahern@gmail.com> Cc: Adrian Hunter <adrian.hunter@intel.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Ingo Molnar <mingo@kernel.org> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/r/1372793245-4136-5-git-send-email-dsahern@gmail.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> --- tools/perf/util/parse-events.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c index ef72e98..bcf83ee 100644 --- a/tools/perf/util/parse-events.c +++ b/tools/perf/util/parse-events.c @@ -1260,6 +1260,4 @@ void parse_events__free_terms(struct list_head *terms) list_for_each_entry_safe(term, h, terms, list) free(term); - - free(terms); }
| |