aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRamkumar Ramachandra <artagnon@gmail.com>2014-03-27 19:50:19 -0400
committerJiri Olsa <jolsa@redhat.com>2014-04-14 12:55:53 +0200
commit5673872d4afd2584e075cf2b1adb2cccec46a0f3 (patch)
treefe0e946ecbcc3747e75413b22a1e24687c9d4bbc
parentperf bench: Update manpage to mention numa and futex (diff)
downloadlinux-dev-5673872d4afd2584e075cf2b1adb2cccec46a0f3.tar.xz
linux-dev-5673872d4afd2584e075cf2b1adb2cccec46a0f3.zip
perf bench: Fix segfault at the end of an 'all' execution
At the end of $ perf bench all the program segfaults because it attempts to dereference a NULL pointer. Fix this fault. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: David Ahern <dsahern@gmail.com> Cc: Jiri Olsa <jolsa@redhat.com> Link: http://lkml.kernel.org/r/1395964219-22173-4-git-send-email-artagnon@gmail.com Signed-off-by: Jiri Olsa <jolsa@redhat.com>
-rw-r--r--tools/perf/builtin-bench.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/builtin-bench.c b/tools/perf/builtin-bench.c
index f600b74216da..1e6e77710545 100644
--- a/tools/perf/builtin-bench.c
+++ b/tools/perf/builtin-bench.c
@@ -86,7 +86,7 @@ static struct collection collections[] = {
/* Iterate over all benchmarks within a collection: */
#define for_each_bench(coll, bench) \
- for (bench = coll->benchmarks; bench->name; bench++)
+ for (bench = coll->benchmarks; bench && bench->name; bench++)
static void dump_benchmarks(struct collection *coll)
{