diff options
author | Ondrej Mosnacek <omosnace@redhat.com> | 2023-10-31 13:32:07 +0100 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2023-11-08 21:39:42 +0100 |
commit | b36995b8609a5a8fe5cf259a1ee768fcaed919f8 (patch) | |
tree | 85eb5e04fd0a67f184ad34c125c61815e7b950c8 /lib/842 | |
parent | lsm: fix default return value for vm_enough_memory (diff) | |
download | linux-b36995b8609a5a8fe5cf259a1ee768fcaed919f8.tar.xz linux-b36995b8609a5a8fe5cf259a1ee768fcaed919f8.zip |
lsm: fix default return value for inode_getsecctx
-EOPNOTSUPP is the return value that implements a "no-op" hook, not 0.
Without this fix having only the BPF LSM enabled (with no programs
attached) can cause uninitialized variable reads in
nfsd4_encode_fattr(), because the BPF hook returns 0 without touching
the 'ctxlen' variable and the corresponding 'contextlen' variable in
nfsd4_encode_fattr() remains uninitialized, yet being treated as valid
based on the 0 return value.
Cc: stable@vger.kernel.org
Fixes: 98e828a0650f ("security: Refactor declaration of LSM hooks")
Reported-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'lib/842')
0 files changed, 0 insertions, 0 deletions