diff options
author | Chris Samuel <chris@csamuel.org> | 2010-11-12 21:35:06 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-12-06 09:52:35 -0500 |
commit | ce47dc56a2241dc035160a85bc5e34283cdd622c (patch) | |
tree | d9404df9d3ca0ddacdd8faae791ece27206dece4 /tools/perf/builtin-kmem.c | |
parent | 965bb6beaf70862d3846e330ea7a14996d82c499 (diff) |
perf tools: Catch a few uncheck calloc/malloc's
There were a few stray calloc()'s and malloc()'s which were not having
their return values checked for success.
As the calling code either already coped with failure or didn't actually
care we just return -ENOMEM at that point.
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Chris Samuel <chris@csamuel.org>
LKML-Reference: <4CDDF95A.1050400@csamuel.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-kmem.c')
-rw-r--r-- | tools/perf/builtin-kmem.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tools/perf/builtin-kmem.c b/tools/perf/builtin-kmem.c index d0a652e9c708..c9620ff6496f 100644 --- a/tools/perf/builtin-kmem.c +++ b/tools/perf/builtin-kmem.c | |||
@@ -736,6 +736,9 @@ static int __cmd_record(int argc, const char **argv) | |||
736 | rec_argc = ARRAY_SIZE(record_args) + argc - 1; | 736 | rec_argc = ARRAY_SIZE(record_args) + argc - 1; |
737 | rec_argv = calloc(rec_argc + 1, sizeof(char *)); | 737 | rec_argv = calloc(rec_argc + 1, sizeof(char *)); |
738 | 738 | ||
739 | if (rec_argv == NULL) | ||
740 | return -ENOMEM; | ||
741 | |||
739 | for (i = 0; i < ARRAY_SIZE(record_args); i++) | 742 | for (i = 0; i < ARRAY_SIZE(record_args); i++) |
740 | rec_argv[i] = strdup(record_args[i]); | 743 | rec_argv[i] = strdup(record_args[i]); |
741 | 744 | ||