diff options
| author | Jiri Olsa <[email protected]> | 2017-08-24 18:27:34 +0200 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <[email protected]> | 2017-08-28 16:44:43 -0300 |
| commit | f4ef3b7c184c4c269f953f226f7158347d007622 (patch) | |
| tree | 78291f6aea342310a15826b002483c19fc976039 | |
| parent | 64eed1deb6d87f4c0efe03297f50367a3689eb56 (diff) | |
perf values: Fix allocation check
Bailing out in case the allocation failed, not the other way round.
Signed-off-by: Jiri Olsa <[email protected]>
Cc: Alexander Shishkin <[email protected]>
Cc: Andi Kleen <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Mark Rutland <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
| -rw-r--r-- | tools/perf/util/values.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/values.c b/tools/perf/util/values.c index 9ac36bf2c438..2c4af02f08cd 100644 --- a/tools/perf/util/values.c +++ b/tools/perf/util/values.c @@ -131,7 +131,7 @@ static int perf_read_values__enlarge_counters(struct perf_read_values *values) for (i = 0; i < values->threads; i++) { u64 *value = realloc(values->value[i], counters_max * sizeof(**values->value)); - if (value) { + if (!value) { pr_debug("failed to enlarge read_values ->values array"); goto out_free_name; } |