diff options
author | Dmitry Kasatkin <d.kasatkin@samsung.com> | 2014-08-20 05:37:57 -0400 |
---|---|---|
committer | Mimi Zohar <zohar@linux.vnet.ibm.com> | 2014-09-09 10:28:51 -0400 |
commit | 86f2bc024966d962d4d7575468e226e2269d198c (patch) | |
tree | 85f59bd2444364d5e722ba71dd3ca73074757063 /security/integrity/ima | |
parent | e4a9c5196566bd47ac92f6e5ef7f48412ded7176 (diff) |
ima: remove unnecessary appraisal test
ima_get_action() sets the "action" flags based on policy.
Before collecting, measuring, appraising, or auditing the
file, the "action" flag is updated based on the cached
iint->flags.
This patch removes the subsequent unnecessary appraisal
test in ima_appraise_measurement().
Signed-off-by: Dmitry Kasatkin <d.kasatkin@samsung.com>
Signed-off-by: Mimi Zohar <zohar@linux.vnet.ibm.com>
Diffstat (limited to 'security/integrity/ima')
-rw-r--r-- | security/integrity/ima/ima_appraise.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/security/integrity/ima/ima_appraise.c b/security/integrity/ima/ima_appraise.c index 225fd944a4ef..013ec3f0e42d 100644 --- a/security/integrity/ima/ima_appraise.c +++ b/security/integrity/ima/ima_appraise.c | |||
@@ -192,8 +192,6 @@ int ima_appraise_measurement(int func, struct integrity_iint_cache *iint, | |||
192 | enum integrity_status status = INTEGRITY_UNKNOWN; | 192 | enum integrity_status status = INTEGRITY_UNKNOWN; |
193 | int rc = xattr_len, hash_start = 0; | 193 | int rc = xattr_len, hash_start = 0; |
194 | 194 | ||
195 | if (!ima_appraise) | ||
196 | return 0; | ||
197 | if (!inode->i_op->getxattr) | 195 | if (!inode->i_op->getxattr) |
198 | return INTEGRITY_UNKNOWN; | 196 | return INTEGRITY_UNKNOWN; |
199 | 197 | ||