diff options
author | Namhyung Kim <namhyung.kim@lge.com> | 2012-09-05 01:02:48 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2012-09-05 16:46:34 -0400 |
commit | 618a3f1d30ea0ee2ff3a88661b8d6a4035123211 (patch) | |
tree | b8576de28fbe9eab9844b67528dacf40a2f10913 | |
parent | 74ba9e11f02a4ce0c7708dc77d1756b93065e440 (diff) |
perf header: Set tracepoint event name only if not set
The event name can be set already by processing a event_desc data.
So check it before setting to prevent possible leak.
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1346821373-31621-3-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/util/header.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c index a124b932817..05c9310c3da 100644 --- a/tools/perf/util/header.c +++ b/tools/perf/util/header.c | |||
@@ -2314,7 +2314,7 @@ static int perf_evlist__set_tracepoint_names(struct perf_evlist *evlist, | |||
2314 | struct perf_evsel *pos; | 2314 | struct perf_evsel *pos; |
2315 | 2315 | ||
2316 | list_for_each_entry(pos, &evlist->entries, node) { | 2316 | list_for_each_entry(pos, &evlist->entries, node) { |
2317 | if (pos->attr.type == PERF_TYPE_TRACEPOINT && | 2317 | if (pos->attr.type == PERF_TYPE_TRACEPOINT && !pos->name && |
2318 | perf_evsel__set_tracepoint_name(pos, pevent)) | 2318 | perf_evsel__set_tracepoint_name(pos, pevent)) |
2319 | return -1; | 2319 | return -1; |
2320 | } | 2320 | } |