diff options
author | Geliang Tang <geliangtang@163.com> | 2015-10-21 17:44:27 -0400 |
---|---|---|
committer | Paul Moore <pmoore@redhat.com> | 2015-10-21 17:44:27 -0400 |
commit | 1d2a168a085f1c65b895f258ee11a52813d25af6 (patch) | |
tree | 5d3d0cbb40dfbf61d9a78c79b61bb16c5a2050c3 | |
parent | 9529c7886c0741847eeb85cf2b0e0730eebe4fa5 (diff) |
selinux: ioctl_has_perm should be static
Fixes the following sparse warning:
security/selinux/hooks.c:3242:5: warning: symbol 'ioctl_has_perm' was
not declared. Should it be static?
Signed-off-by: Geliang Tang <geliangtang@163.com>
Acked-by: Jeff Vander Stoep <jeffv@google.com>
Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
Signed-off-by: Paul Moore <pmoore@redhat.com>
-rw-r--r-- | security/selinux/hooks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 1530f661ef85..799d15aa35f9 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c | |||
@@ -3235,7 +3235,7 @@ static void selinux_file_free_security(struct file *file) | |||
3235 | * Check whether a task has the ioctl permission and cmd | 3235 | * Check whether a task has the ioctl permission and cmd |
3236 | * operation to an inode. | 3236 | * operation to an inode. |
3237 | */ | 3237 | */ |
3238 | int ioctl_has_perm(const struct cred *cred, struct file *file, | 3238 | static int ioctl_has_perm(const struct cred *cred, struct file *file, |
3239 | u32 requested, u16 cmd) | 3239 | u32 requested, u16 cmd) |
3240 | { | 3240 | { |
3241 | struct common_audit_data ad; | 3241 | struct common_audit_data ad; |