diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 09:04:18 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-04-02 15:28:31 -0400 |
commit | ad5b217b152d99ca3922153500c619d9758dd87a (patch) | |
tree | e0377f6f1de3394d29babe49c24a0152df18000d /tools/perf/builtin-report.c | |
parent | b9fb93047756c5e4129dfda7591612de61b0e877 (diff) |
perf session: Remove one more exit() call from library code
Return NULL instead and make the caller propagate the error.
LKML-Reference: <new-submission>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-report.c')
-rw-r--r-- | tools/perf/builtin-report.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 1fb13e5fd1f9..6767f10615ea 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -89,9 +89,12 @@ static int perf_session__add_hist_entry(struct perf_session *self, | |||
89 | struct event_stat_id *stats; | 89 | struct event_stat_id *stats; |
90 | struct perf_event_attr *attr; | 90 | struct perf_event_attr *attr; |
91 | 91 | ||
92 | if ((sort__has_parent || symbol_conf.use_callchain) && data->callchain) | 92 | if ((sort__has_parent || symbol_conf.use_callchain) && data->callchain) { |
93 | syms = perf_session__resolve_callchain(self, al->thread, | 93 | syms = perf_session__resolve_callchain(self, al->thread, |
94 | data->callchain, &parent); | 94 | data->callchain, &parent); |
95 | if (syms == NULL) | ||
96 | return -ENOMEM; | ||
97 | } | ||
95 | 98 | ||
96 | attr = perf_header__find_attr(data->id, &self->header); | 99 | attr = perf_header__find_attr(data->id, &self->header); |
97 | if (attr) | 100 | if (attr) |