diff options
author | Ramkumar Ramachandra <artagnon@gmail.com> | 2014-03-28 00:50:19 +0100 |
---|---|---|
committer | Jiri Olsa <jolsa@redhat.com> | 2014-04-14 12:55:53 +0200 |
commit | 5673872d4afd2584e075cf2b1adb2cccec46a0f3 (patch) | |
tree | fe0e946ecbcc3747e75413b22a1e24687c9d4bbc /tools | |
parent | perf bench: Update manpage to mention numa and futex (diff) | |
download | linux-5673872d4afd2584e075cf2b1adb2cccec46a0f3.tar.xz linux-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>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/perf/builtin-bench.c | 2 |
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) { |