diff options
author | Florent Revest <revest@chromium.org> | 2021-05-17 11:28:30 +0200 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2021-05-20 23:48:38 +0200 |
commit | 0af02eb2a7d76ca85a1ecaf4b3775e2c86408fab (patch) | |
tree | 258cbac9f9e49aca8f960e3eb45f200189b2ffd2 /ipc | |
parent | 8afcc19fbf083a8459284d9a29b4b5ac1cb2396c (diff) | |
download | linux-stable-0af02eb2a7d76ca85a1ecaf4b3775e2c86408fab.tar.gz linux-stable-0af02eb2a7d76ca85a1ecaf4b3775e2c86408fab.tar.bz2 linux-stable-0af02eb2a7d76ca85a1ecaf4b3775e2c86408fab.zip |
bpf: Avoid using ARRAY_SIZE on an uninitialized pointer
The cppcheck static code analysis reported the following error:
if (WARN_ON_ONCE(nest_level > ARRAY_SIZE(bufs->tmp_bufs))) {
^
ARRAY_SIZE is a macro that expands to sizeofs, so bufs is not actually
dereferenced at runtime, and the code is actually safe. But to keep
things tidy, this patch removes the need for a call to ARRAY_SIZE by
extracting the size of the array into a macro. Cppcheck should no longer
be confused and the code ends up being a bit cleaner.
Fixes: e2d5b2bb769f ("bpf: Fix nested bpf_bprintf_prepare with more per-cpu buffers")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Florent Revest <revest@chromium.org>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Song Liu <song@kernel.org>
Link: https://lore.kernel.org/bpf/20210517092830.1026418-2-revest@chromium.org
Diffstat (limited to 'ipc')
0 files changed, 0 insertions, 0 deletions