selinux: fall back to ref-walk if audit is required
authorStephen Smalley <sds@tycho.nsa.gov>
Fri, 22 Nov 2019 17:22:45 +0000 (12:22 -0500)
committerPaul Moore <paul@paul-moore.com>
Mon, 9 Dec 2019 23:37:47 +0000 (18:37 -0500)
commit0188d5c025ca8fe756ba3193bd7d150139af5a88
treed0c0095e8765270013f691f0cf5df3626137e8f4
parent1a37079c236d55fb31ebbf4b59945dab8ec8764c
selinux: fall back to ref-walk if audit is required

commit bda0be7ad994 ("security: make inode_follow_link RCU-walk aware")
passed down the rcu flag to the SELinux AVC, but failed to adjust the
test in slow_avc_audit() to also return -ECHILD on LSM_AUDIT_DATA_DENTRY.
Previously, we only returned -ECHILD if generating an audit record with
LSM_AUDIT_DATA_INODE since this was only relevant from inode_permission.
Move the handling of MAY_NOT_BLOCK to avc_audit() and its inlined
equivalent in selinux_inode_permission() immediately after we determine
that audit is required, and always fall back to ref-walk in this case.

Fixes: bda0be7ad994 ("security: make inode_follow_link RCU-walk aware")
Reported-by: Will Deacon <will@kernel.org>
Suggested-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: Paul Moore <paul@paul-moore.com>
security/selinux/avc.c
security/selinux/hooks.c
security/selinux/include/avc.h