summaryrefslogtreecommitdiffstats
path: root/tools/perf/builtin-test.c
diff options
context:
space:
mode:
authorHan Pingtian <phan@redhat.com>2011-01-15 00:00:50 +0100
committerArnaldo Carvalho de Melo <acme@redhat.com>2011-01-22 22:56:30 +0100
commit98d77b78504a423fca911a26a17bee00ef2fdda2 (patch)
treefbb246e981c5358fa28e1eaf3cf92dc753cc4afe /tools/perf/builtin-test.c
parentperf test: Check counts on all cpus in test__open_syscall_event_on_all_cpus (diff)
downloadlinux-98d77b78504a423fca911a26a17bee00ef2fdda2.tar.xz
linux-98d77b78504a423fca911a26a17bee00ef2fdda2.zip
perf test: check if cpu_map__new() return NULL
It looks like we should check if cpus is NULL after cpus = cpu_map__new(NULL); in test__open_syscall_event_on_all_cpus(). LKML-Reference: <20110114230050.GA7011@localhost> Signed-off-by: Han Pingtian <phan@redhat.com> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-test.c')
-rw-r--r--tools/perf/builtin-test.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/perf/builtin-test.c b/tools/perf/builtin-test.c
index 7cc6b2086947..5a50e4755e6c 100644
--- a/tools/perf/builtin-test.c
+++ b/tools/perf/builtin-test.c
@@ -347,9 +347,9 @@ static int test__open_syscall_event_on_all_cpus(void)
}
cpus = cpu_map__new(NULL);
- if (threads == NULL) {
- pr_debug("thread_map__new\n");
- return -1;
+ if (cpus == NULL) {
+ pr_debug("cpu_map__new\n");
+ goto out_thread_map_delete;
}