diff options
| author | Mimi Zohar <zohar@linux.vnet.ibm.com> | 2012-01-17 22:11:28 -0500 |
|---|---|---|
| committer | James Morris <jmorris@namei.org> | 2012-01-19 15:59:11 +1100 |
| commit | 3db59dd93309710c40aaf1571c607cb0feef3ecb (patch) | |
| tree | 6a224a855aad0e5207abae573456b2d2ec381f7c /security/integrity | |
| parent | 4bf1924c008dffdc154f82507b4052e49263a6f4 (diff) | |
| download | lwn-3db59dd93309710c40aaf1571c607cb0feef3ecb.tar.gz lwn-3db59dd93309710c40aaf1571c607cb0feef3ecb.zip | |
ima: fix cred sparse warning
Fix ima_policy.c sparse "warning: dereference of noderef expression"
message, by accessing cred->uid using current_cred().
Changelog v1:
- Change __cred to just cred (based on David Howell's comment)
Signed-off-by: Mimi Zohar <zohar@us.ibm.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/integrity')
| -rw-r--r-- | security/integrity/ima/ima_policy.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/security/integrity/ima/ima_policy.c b/security/integrity/ima/ima_policy.c index d661afbe474c..d45061d02fee 100644 --- a/security/integrity/ima/ima_policy.c +++ b/security/integrity/ima/ima_policy.c @@ -99,6 +99,7 @@ static bool ima_match_rules(struct ima_measure_rule_entry *rule, struct inode *inode, enum ima_hooks func, int mask) { struct task_struct *tsk = current; + const struct cred *cred = current_cred(); int i; if ((rule->flags & IMA_FUNC) && rule->func != func) @@ -108,7 +109,7 @@ static bool ima_match_rules(struct ima_measure_rule_entry *rule, if ((rule->flags & IMA_FSMAGIC) && rule->fsmagic != inode->i_sb->s_magic) return false; - if ((rule->flags & IMA_UID) && rule->uid != tsk->cred->uid) + if ((rule->flags & IMA_UID) && rule->uid != cred->uid) return false; for (i = 0; i < MAX_LSM_RULES; i++) { int rc = 0; |