Roberto Sassu a9100b6f1a ima: do not update security.ima if appraisal status is not INTEGRITY_PASS ... commit 020aae3ee5 upstream. Commit b65a9cfc2c ("Untangling ima mess, part 2: deal with counters") moved the call of ima_file_check() from may_open() to do_filp_open() at a point where the file descriptor is already opened. This breaks the assumption made by IMA that file descriptors being closed belong to files whose access was granted by ima_file_check(). The consequence is that security.ima and security.evm are updated with good values, regardless of the current appraisal status. For example, if a file does not have security.ima, IMA will create it after opening the file for writing, even if access is denied. Access to the file will be allowed afterwards. Avoid this issue by checking the appraisal status before updating security.ima. Signed-off-by: Roberto Sassu <roberto.sassu@huawei.com> Signed-off-by: Mimi Zohar <zohar@linux.vnet.ibm.com> Signed-off-by: James Morris <james.l.morris@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2017-11-24 08:32:24 +01:00
..
evm	EVM: Use crypto_memneq() for digest comparisons	2016-02-17 12:31:04 -08:00
ima	ima: do not update security.ima if appraisal status is not INTEGRITY_PASS	2017-11-24 08:32:24 +01:00
digsig.c	integrity: prevent loading untrusted certificates on the IMA trusted keyring	2015-10-09 15:31:18 -04:00
digsig_asymmetric.c	integrity: do zero padding of the key id	2014-10-06 17:33:27 +01:00
iint.c	integrity: add validity checks for 'path' parameter	2015-05-21 13:59:28 -04:00
integrity.h	integrity: add validity checks for 'path' parameter	2015-05-21 13:59:28 -04:00
integrity_audit.c	Merge git://git.infradead.org/users/eparis/audit	2014-04-12 12:38:53 -07:00
Kconfig	kconfig: use bool instead of boolean for type definition attributes	2015-01-07 13:08:04 +01:00
Makefile	integrity: make integrity files as 'integrity' module	2014-09-09 10:28:58 -04:00