diff options
| author | Han Pingtian <phan@redhat.com> | 2011-01-15 07:00:50 +0800 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2011-01-22 19:56:30 -0200 |
| commit | 98d77b78504a423fca911a26a17bee00ef2fdda2 (patch) | |
| tree | fbb246e981c5358fa28e1eaf3cf92dc753cc4afe /tools/perf/builtin-test.c | |
| parent | d2af9687c96f3864178de1860e6d83873aeef224 (diff) | |
| download | lwn-98d77b78504a423fca911a26a17bee00ef2fdda2.tar.gz lwn-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.c | 6 |
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; } |