aboutsummaryrefslogtreecommitdiffstats
path: root/security
diff options
context:
space:
mode:
authorCasey Schaufler <casey@schaufler-ca.com>2011-02-09 22:58:42 -0500
committerCasey Schaufler <casey@schaufler-ca.com>2011-02-09 22:58:42 -0500
commit75a25637bf8a1b8fbed2368c0a3ec15c66a534f1 (patch)
tree038d52827d9a285fed1bb384f06d7adabf4ef674 /security
parentdb904aa8147440b750a35d58befed38155a1abb9 (diff)
Smack: correct final mmap check comparison
The mmap policy enforcement checks the access of the SMACK64MMAP subject against the current subject incorrectly. The check as written works correctly only if the access rules involved have the same access. This is the common case, so initial testing did not find a problem. Signed-off-by: Casey Schaufler <casey@schaufler-ca.com>
Diffstat (limited to 'security')
-rw-r--r--security/smack/smack_lsm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
index 92cb71507f5b..5ab3f39442f2 100644
--- a/security/smack/smack_lsm.c
+++ b/security/smack/smack_lsm.c
@@ -1218,7 +1218,7 @@ static int smack_file_mmap(struct file *file,
1218 * not available to a SMACK64MMAP subject 1218 * not available to a SMACK64MMAP subject
1219 * deny access. 1219 * deny access.
1220 */ 1220 */
1221 if ((may | mmay) != may) { 1221 if ((may | mmay) != mmay) {
1222 rc = -EACCES; 1222 rc = -EACCES;
1223 break; 1223 break;
1224 } 1224 }