aboutsummaryrefslogtreecommitdiffstats
path: root/security/integrity
diff options
context:
space:
mode:
authorDmitry Kasatkin <d.kasatkin@samsung.com>2014-08-20 05:37:57 -0400
committerMimi Zohar <zohar@linux.vnet.ibm.com>2014-09-09 10:28:51 -0400
commit86f2bc024966d962d4d7575468e226e2269d198c (patch)
tree85f59bd2444364d5e722ba71dd3ca73074757063 /security/integrity
parente4a9c5196566bd47ac92f6e5ef7f48412ded7176 (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')
-rw-r--r--security/integrity/ima/ima_appraise.c2
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