diff options
| author | Arnaldo Carvalho de Melo | 2009-12-18 19:35:58 +0100 |
|---|---|---|
| committer | Ingo Molnar | 2009-12-18 20:01:52 +0100 |
| commit | 0422a4fc2c6a41c3d4184984797512af2650014e (patch) | |
| tree | 9360c59733c31e1bbf160c4d089942c04082cfb4 /tools/perf/builtin-diff.c | |
| parent | perf session: Make events_stats u64 to avoid overflow on 32-bit arches (diff) | |
| download | kernel-qcow2-linux-0422a4fc2c6a41c3d4184984797512af2650014e.tar.gz kernel-qcow2-linux-0422a4fc2c6a41c3d4184984797512af2650014e.tar.xz kernel-qcow2-linux-0422a4fc2c6a41c3d4184984797512af2650014e.zip | |
perf diff: Fix usage array, it must end with a NULL entry
Fixing this:
[acme@doppio linux-2.6-tip]$ perf diff --hell
Error: unknown option `hell'
usage: perf diff [<options>] [old_file] [new_file]
Segmentation fault
[acme@doppio linux-2.6-tip]$
Also go over the other such arrays to check if they all were OK,
they are, but there were some minor changes to do like making
one static and renaming another to match the command it refers
to.
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1261161358-23959-1-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-diff.c')
| -rw-r--r-- | tools/perf/builtin-diff.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tools/perf/builtin-diff.c b/tools/perf/builtin-diff.c index 4d33b55d5584..bd71b8ceafb7 100644 --- a/tools/perf/builtin-diff.c +++ b/tools/perf/builtin-diff.c @@ -189,8 +189,9 @@ out_delete: return ret; } -static const char *const diff_usage[] = { +static const char * const diff_usage[] = { "perf diff [<options>] [old_file] [new_file]", + NULL, }; static const struct option options[] = { |