summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCasey Schaufler <casey@schaufler-ca.com>2013-12-30 17:37:45 -0800
committerCasey Schaufler <casey@schaufler-ca.com>2013-12-31 13:35:27 -0800
commit4482a44f6a3221cd0076eb6af65672a7e198d8da (patch)
tree5caa99f7179486a64399de14496fba56a88cb7e9
parent24ea1b6efcd8fc3b465fb74964e1a0cbe9979730 (diff)
downloadop-kernel-dev-4482a44f6a3221cd0076eb6af65672a7e198d8da.zip
op-kernel-dev-4482a44f6a3221cd0076eb6af65672a7e198d8da.tar.gz
Smack: File receive audit correction
Eric Paris politely points out: Inside smack_file_receive() it seems like you are initting the audit field with LSM_AUDIT_DATA_TASK. And then use smk_ad_setfield_u_fs_path(). Seems like LSM_AUDIT_DATA_PATH would make more sense. (and depending on how it's used fix a crash...) He is correct. This puts things in order. Targeted for git://git.gitorious.org/smack-next/kernel.git Signed-off-by: Casey Schaufler <casey@schaufler-ca.com>
-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 d552832..d814e35 100644
--- a/security/smack/smack_lsm.c
+++ b/security/smack/smack_lsm.c
@@ -1351,7 +1351,7 @@ static int smack_file_receive(struct file *file)
int may = 0;
struct smk_audit_info ad;
- smk_ad_init(&ad, __func__, LSM_AUDIT_DATA_TASK);
+ smk_ad_init(&ad, __func__, LSM_AUDIT_DATA_PATH);
smk_ad_setfield_u_fs_path(&ad, file->f_path);
/*
* This code relies on bitmasks.
OpenPOWER on IntegriCloud