Skip to content

Commit c264487

Browse files
GeHao01994Alexei Starovoitov
authored andcommitted
selftests/bpf: Fix incorrect parameters in NULL pointer checking
Smatch reported the following warning: ./tools/testing/selftests/bpf/testing_helpers.c:455 get_xlated_program() warn: variable dereferenced before check 'buf' (see line 454) It seems correct,so let's modify it based on it's suggestion. Actually,commit b23ed4d ("selftests/bpf: Fix invalid pointer check in get_xlated_program()") fixed an issue in the test_verifier.c once,but it was reverted this time. Let's solve this issue with the minimal changes possible. Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Closes: https://lore.kernel.org/all/1eb3732f-605a-479d-ba64-cd14250cbf91@stanley.mountain/ Fixes: b4b7a40 ("selftests/bpf: Factor out get_xlated_program() helper") Signed-off-by: Hao Ge <gehao@kylinos.cn> Link: https://lore.kernel.org/r/20240820023622.29190-1-hao.ge@linux.dev Signed-off-by: Alexei Starovoitov <ast@kernel.org>
1 parent 4961d8f commit c264487

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tools/testing/selftests/bpf/testing_helpers.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -452,7 +452,7 @@ int get_xlated_program(int fd_prog, struct bpf_insn **buf, __u32 *cnt)
452452

453453
*cnt = xlated_prog_len / buf_element_size;
454454
*buf = calloc(*cnt, buf_element_size);
455-
if (!buf) {
455+
if (!*buf) {
456456
perror("can't allocate xlated program buffer");
457457
return -ENOMEM;
458458
}

0 commit comments

Comments
 (0)