diff options
author | Xu Kuohai <xukuohai@huawei.com> | 2024-07-19 19:00:52 +0800 |
---|---|---|
committer | Andrii Nakryiko <andrii@kernel.org> | 2024-07-29 13:09:22 -0700 |
commit | 5d99e198be279045e6ecefe220f5c52f8ce9bfd5 (patch) | |
tree | 006d24765b0fd8fdff944e4924a2f75149f78471 /tools/testing/selftests/bpf/prog_tests/assign_reuse.c | |
parent | 21c7063f6d08ab9afa088584939791bee0c177e5 (diff) |
bpf, lsm: Add check for BPF LSM return value
A bpf prog returning a positive number attached to file_alloc_security
hook makes kernel panic.
This happens because file system can not filter out the positive number
returned by the LSM prog using IS_ERR, and misinterprets this positive
number as a file pointer.
Given that hook file_alloc_security never returned positive number
before the introduction of BPF LSM, and other BPF LSM hooks may
encounter similar issues, this patch adds LSM return value check
in verifier, to ensure no unexpected value is returned.
Fixes: 520b7aa00d8c ("bpf: lsm: Initialize the BPF LSM hooks")
Reported-by: Xin Liu <liuxin350@huawei.com>
Signed-off-by: Xu Kuohai <xukuohai@huawei.com>
Acked-by: Eduard Zingerman <eddyz87@gmail.com>
Link: https://lore.kernel.org/r/20240719110059.797546-3-xukuohai@huaweicloud.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Andrii Nakryiko <andrii@kernel.org>
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/assign_reuse.c')
0 files changed, 0 insertions, 0 deletions