diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2011-02-01 13:18:10 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2011-02-01 13:18:10 -0500 |
commit | 0015e2e101f5fd3256ab8b5a374c0e8806098871 (patch) | |
tree | 3c889612ca4ede04a5b13d1cc3499479b60c66a3 /tools/perf/builtin-stat.c | |
parent | 568bb7b8e856b9efb98a3f63259c717adc1b96b8 (diff) |
perf stat: Fix up resource release order
That was causing a SEGV on selected old distros.
Problem introduced in 7e2ed09.
Reported-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Cc: Tom Zanussi <tzanussi@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-stat.c')
-rw-r--r-- | tools/perf/builtin-stat.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c index e0f95755361b..806a9998fcd5 100644 --- a/tools/perf/builtin-stat.c +++ b/tools/perf/builtin-stat.c | |||
@@ -748,8 +748,8 @@ int cmd_stat(int argc, const char **argv, const char *prefix __used) | |||
748 | out_free_fd: | 748 | out_free_fd: |
749 | list_for_each_entry(pos, &evsel_list->entries, node) | 749 | list_for_each_entry(pos, &evsel_list->entries, node) |
750 | perf_evsel__free_stat_priv(pos); | 750 | perf_evsel__free_stat_priv(pos); |
751 | perf_evlist__delete(evsel_list); | ||
752 | out: | ||
753 | perf_evlist__delete_maps(evsel_list); | 751 | perf_evlist__delete_maps(evsel_list); |
752 | out: | ||
753 | perf_evlist__delete(evsel_list); | ||
754 | return status; | 754 | return status; |
755 | } | 755 | } |