[PATCH 1/2] ima: simplify function ima_store_template

Lakshmi Ramasubramanian nramas at linux.microsoft.com
Tue Apr 14 15:17:14 UTC 2020


On 4/14/20 4:48 AM, Tianjia Zhang wrote:

> The 'result' here is not necessary, remove redundant code,
> the code is more concise.
> 
> Signed-off-by: Tianjia Zhang <tianjia.zhang at linux.alibaba.com>
> ---
>   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);
>   }
>   
>   /*
> 


Reviewed-by: Lakshmi Ramasubramanian <nramas at linux.microsoft.com>



More information about the Linux-security-module-archive mailing list