diff options
author | Yonghong Song <yhs@fb.com> | 2022-01-26 10:19:40 -0800 |
---|---|---|
committer | Andrii Nakryiko <andrii@kernel.org> | 2022-01-26 12:07:01 -0800 |
commit | e5465a9027e9703d8f5ec9c0387e32fb3a02b58f (patch) | |
tree | ea007147350265458590ae044d97100e7ceb29bc /net | |
parent | c446fdacb10dcb3b9a9ed3b91d91e72d71d94b03 (diff) | |
download | linux-e5465a9027e9703d8f5ec9c0387e32fb3a02b58f.tar.gz linux-e5465a9027e9703d8f5ec9c0387e32fb3a02b58f.tar.bz2 linux-e5465a9027e9703d8f5ec9c0387e32fb3a02b58f.zip |
selftests/bpf: Fix a clang compilation error
Compiling kernel and selftests/bpf with latest llvm like blow:
make -j LLVM=1
make -C tools/testing/selftests/bpf -j LLVM=1
I hit the following compilation error:
/.../prog_tests/log_buf.c:215:6: error: variable 'log_buf' is used uninitialized whenever 'if' condition is true [-Werror,-Wsometimes-uninitialized]
if (!ASSERT_OK_PTR(raw_btf_data, "raw_btf_data_good"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/.../prog_tests/log_buf.c:264:7: note: uninitialized use occurs here
free(log_buf);
^~~~~~~
/.../prog_tests/log_buf.c:215:2: note: remove the 'if' if its condition is always false
if (!ASSERT_OK_PTR(raw_btf_data, "raw_btf_data_good"))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/.../prog_tests/log_buf.c:205:15: note: initialize the variable 'log_buf' to silence this warning
char *log_buf;
^
= NULL
1 error generated.
Compiler rightfully detected that log_buf is uninitialized in one of failure path as indicated
in the above.
Proper initialization of 'log_buf' variable fixed the issue.
Signed-off-by: Yonghong Song <yhs@fb.com>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Link: https://lore.kernel.org/bpf/20220126181940.4105997-1-yhs@fb.com
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions