Commit 6ee28442 authored Apr 27, 2020 by Krzysztof Struczynski Committed by Mimi Zohar May 07, 2020

ima: Remove redundant policy rule set in add_rules()

Function ima_appraise_flag() returns the flag to be set in
temp_ima_appraise depending on the hook identifier passed as an argument.
It is not necessary to set the flag again for the POLICY_CHECK hook.

Signed-off-by: Krzysztof Struczynski <krzysztof.struczynski@huawei.com>
Signed-off-by: Mimi Zohar <zohar@linux.ibm.com>

parent 1129d31b

security/integrity/ima/ima_policy.c

+1 −4

Original line number	Diff line number	Diff line
		@@ -643,11 +643,8 @@ static void add_rules(struct ima_rule_entry *entries, int count,

		list_add_tail(&entry->list, &ima_policy_rules);
		}
		if (entries[i].action == APPRAISE) {
		if (entries[i].action == APPRAISE)
		temp_ima_appraise \|= ima_appraise_flag(entries[i].func);
		if (entries[i].func == POLICY_CHECK)
		temp_ima_appraise \|= IMA_APPRAISE_POLICY;
		}
		}
		}