summaryrefslogtreecommitdiffstats
path: root/security/smack
diff options
context:
space:
mode:
authorCasey Schaufler <casey@schaufler-ca.com>2009-01-27 19:56:30 -0800
committerLinus Torvalds <torvalds@linux-foundation.org>2009-01-27 20:13:32 -0800
commit152a649b647a8ef47bb74ff9e11850fa6001bedc (patch)
treeea626697e2cbf07f1cba973158b99125e98344ae /security/smack
parente4a7ca29039e615ce13a61b9c6abfb2aa394e9a1 (diff)
downloadop-kernel-dev-152a649b647a8ef47bb74ff9e11850fa6001bedc.zip
op-kernel-dev-152a649b647a8ef47bb74ff9e11850fa6001bedc.tar.gz
smackfs load append mode fix
Given just how hard it is to find the code that uses MAY_APPEND it's probably not a big surprise that this went unnoticed for so long. The Smack rules loading code is incorrectly setting the MAY_READ bit when MAY_APPEND is requested. Signed-off-by: Casey Schaufler <casey@schaufler-ca.com> Reviewed-by: James Morris <jmorris@namei.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'security/smack')
-rw-r--r--security/smack/smackfs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/security/smack/smackfs.c b/security/smack/smackfs.c
index 71e2b91..8e42800 100644
--- a/security/smack/smackfs.c
+++ b/security/smack/smackfs.c
@@ -334,7 +334,7 @@ static ssize_t smk_write_load(struct file *file, const char __user *buf,
break;
case 'a':
case 'A':
- rule.smk_access |= MAY_READ;
+ rule.smk_access |= MAY_APPEND;
break;
default:
goto out;
OpenPOWER on IntegriCloud