diff options
| author | Anton Protopopov <[email protected]> | 2024-12-10 11:42:45 +0000 |
|---|---|---|
| committer | Alexei Starovoitov <[email protected]> | 2024-12-10 11:17:53 -0800 |
| commit | c4441ca86afe4814039ee1b32c39d833c1a16bbc (patch) | |
| tree | 3112465a5fa03bfe57455d23b72a67771819dc28 /tools/testing/selftests/bpf/prog_tests/prog_array_init.c | |
| parent | cf8b876363da4fccdcc4ba209d4d098ec0f1ffac (diff) | |
bpf: fix potential error return
The bpf_remove_insns() function returns WARN_ON_ONCE(error), where
error is a result of bpf_adj_branches(), and thus should be always 0
However, if for any reason it is not 0, then it will be converted to
boolean by WARN_ON_ONCE and returned to user space as 1, not an actual
error value. Fix this by returning the original err after the WARN check.
Signed-off-by: Anton Protopopov <[email protected]>
Acked-by: Jiri Olsa <[email protected]>
Acked-by: Andrii Nakryiko <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Alexei Starovoitov <[email protected]>
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/prog_array_init.c')
0 files changed, 0 insertions, 0 deletions