diff options
| author | Ramkumar Ramachandra <artagnon@gmail.com> | 2014-03-27 19:50:19 -0400 |
|---|---|---|
| committer | Jiri Olsa <jolsa@redhat.com> | 2014-04-14 12:55:53 +0200 |
| commit | 5673872d4afd2584e075cf2b1adb2cccec46a0f3 (patch) | |
| tree | fe0e946ecbcc3747e75413b22a1e24687c9d4bbc | |
| parent | 95a2b3c0a9e2a8038edbf3f9776d5a3d76146459 (diff) | |
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.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) { |