The 'result' here is not necessary, remove redundant code, the code is more concise. Signed-off-by: Tianjia Zhang <tianjia.zhang@xxxxxxxxxxxxxxxxx> --- security/integrity/ima/ima_api.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/security/integrity/ima/ima_api.c b/security/integrity/ima/ima_api.c index f6bc00914aa5..9121257c9dc6 100644 --- a/security/integrity/ima/ima_api.c +++ b/security/integrity/ima/ima_api.c @@ -118,8 +118,7 @@ int ima_store_template(struct ima_template_entry *entry, memcpy(entry->digest, hash.hdr.digest, hash.hdr.length); } entry->pcr = pcr; - result = ima_add_template_entry(entry, violation, op, inode, filename); - return result; + return ima_add_template_entry(entry, violation, op, inode, filename); } /* -- 2.17.1