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 a124b9328170..05c9310c3da1 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 | } |
