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 <[email protected]> Closes: https://lore.kernel.org/all/[email protected]/ Fixes: b4b7a40 ("selftests/bpf: Factor out get_xlated_program() helper") Signed-off-by: Hao Ge <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Alexei Starovoitov <[email protected]>
1 parent 4961d8f commit c264487

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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)