diff options
| author |   Arnaldo Carvalho de Melo <acme@redhat.com> | 2025-03-10 16:45:32 -0300 |
|---|---|---|
| committer |   Namhyung Kim <namhyung@kernel.org> | 2025-03-13 00:30:08 -0700 |
| commit | cf67629f7f637fb988228abdb3aae46d0c1748fe (patch) | |
| tree | 0c70483e7a9fd12d68ef91ca9a593d3f8ec6e18b | |
| parent | libapi: Add missing header with NAME_MAX define to io_dir.h (diff) | |
| download | wireguard-linux-cf67629f7f637fb988228abdb3aae46d0c1748fe.tar.xz wireguard-linux-cf67629f7f637fb988228abdb3aae46d0c1748fe.zip | |
perf units: Fix insufficient array space
No need to specify the array size, let the compiler figure that out.
This addresses this compiler warning that was noticed while build
testing on fedora rawhide:
31 15.81 fedora:rawhide : FAIL gcc version 15.0.1 20250225 (Red Hat 15.0.1-0) (GCC)
util/units.c: In function 'unit_number__scnprintf':
util/units.c:67:24: error: initializer-string for array of 'char' is too long [-Werror=unterminated-string-initialization]
67 | char unit[4] = "BKMG";
| ^~~~~~
cc1: all warnings being treated as errors
Fixes: 9808143ba2e54818 ("perf tools: Add unit_number__scnprintf function")
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Link: https://lore.kernel.org/r/20250310194534.265487-3-acme@kernel.org
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
| -rw-r--r-- | tools/perf/util/units.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/util/units.c b/tools/perf/util/units.c index 32c39cfe209b..4c6a86e1cb54 100644 --- a/tools/perf/util/units.c +++ b/tools/perf/util/units.c @@ -64,7 +64,7 @@ unsigned long convert_unit(unsigned long value, char *unit) int unit_number__scnprintf(char *buf, size_t size, u64 n) { - char unit[4] = "BKMG"; + char unit[] = "BKMG"; int i = 0; while (((n / 1024) > 1) && (i < 3)) { |