perf hists browser: Fix NULL deref in hists browsing code
authorJiri Olsa <jolsa@redhat.com>
Wed, 4 Apr 2012 20:21:31 +0000 (22:21 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 5 Apr 2012 21:58:45 +0000 (18:58 -0300)
commit8493fe1daf15324eb13a4cc2f94e258716daa568
tree31cdd2296154a6288b643cda10af61b78235f4a1
parent63fa471dd49e9c9ce029d910d1024330d9b1b145
perf hists browser: Fix NULL deref in hists browsing code

If there's an event with no samples in data file, the perf report
command can segfault after entering the event details menu.

Following steps reproduce the issue:

 # ./perf record -e syscalls:sys_enter_kexec_load,syscalls:sys_enter_mmap ls
 # ./perf report
 # enter '0 syscalls:sys_enter_kexec_load' menu
 # pres ENTER twice

Above steps are valid assuming ls wont run kexec.. ;)

The check for sellection to be NULL is missing. The fix makes sure it's
being check. Above steps now endup with menu being displayed allowing
'Exit' as the only option.

Signed-off-by: Jiri Olsa <jolsa@redhat.com>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1333570898-10505-2-git-send-email-jolsa@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/ui/browsers/hists.c