summaryrefslogtreecommitdiffstats
path: root/security
diff options
context:
space:
mode:
authorStephen Smalley <sds@tycho.nsa.gov>2008-03-17 08:55:18 -0400
committerJames Morris <jmorris@namei.org>2008-04-02 16:05:52 +1100
commit0794c66d49885a2f615618ce4940434b5b067d84 (patch)
treeb01be53c424c7d4793f5673539c11d09fbbe2b5a /security
parent0e81a8ae37687845f7cdfa2adce14ea6a5f1dd34 (diff)
downloadkernel-crypto-0794c66d49885a2f615618ce4940434b5b067d84.tar.gz
kernel-crypto-0794c66d49885a2f615618ce4940434b5b067d84.tar.xz
kernel-crypto-0794c66d49885a2f615618ce4940434b5b067d84.zip
selinux: handle files opened with flags 3 by checking ioctl permission
Handle files opened with flags 3 by checking ioctl permission. Default to returning FILE__IOCTL from file_to_av() if the f_mode has neither FMODE_READ nor FMODE_WRITE, and thus check ioctl permission on exec or transfer, thereby validating such descriptors early as with normal r/w descriptors and catching leaks of them prior to attempted usage. Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov> Acked-by: Eric Paris <eparis@redhat.com> Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security')
-rw-r--r--security/selinux/hooks.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
index 41a049f50f5..c2fef7b12dc 100644
--- a/security/selinux/hooks.c
+++ b/security/selinux/hooks.c
@@ -1630,6 +1630,12 @@ static inline u32 file_to_av(struct file *file)
else
av |= FILE__WRITE;
}
+ if (!av) {
+ /*
+ * Special file opened with flags 3 for ioctl-only use.
+ */
+ av = FILE__IOCTL;
+ }
return av;
}