[PATCH] tools/crypto:Remove unused variable

Zhu Jun posted 1 patch 1 year, 5 months ago
tools/crypto/ccp/dbc.c | 1 -
1 file changed, 1 deletion(-)
[PATCH] tools/crypto:Remove unused variable
Posted by Zhu Jun 1 year, 5 months ago
the variable is never referenced in the code, just remove them.

Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>
---
 tools/crypto/ccp/dbc.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/crypto/ccp/dbc.c b/tools/crypto/ccp/dbc.c
index a807df0f0597..80248d3d3a5a 100644
--- a/tools/crypto/ccp/dbc.c
+++ b/tools/crypto/ccp/dbc.c
@@ -57,7 +57,6 @@ int process_param(int fd, int msg_index, __u8 *signature, int *data)
 		.msg_index = msg_index,
 		.param = *data,
 	};
-	int ret;
 
 	assert(signature);
 	assert(data);
-- 
2.17.1
Re: [PATCH] tools/crypto:Remove unused variable
Posted by Mario Limonciello 1 year, 5 months ago
On 8/20/2024 02:42, Zhu Jun wrote:
> the variable is never referenced in the code, just remove them.

nit on the sentence:

s,the variable,the ret variable,
s,them,it,

> 
> Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>

Thank you!

Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>

> ---
>   tools/crypto/ccp/dbc.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/tools/crypto/ccp/dbc.c b/tools/crypto/ccp/dbc.c
> index a807df0f0597..80248d3d3a5a 100644
> --- a/tools/crypto/ccp/dbc.c
> +++ b/tools/crypto/ccp/dbc.c
> @@ -57,7 +57,6 @@ int process_param(int fd, int msg_index, __u8 *signature, int *data)
>   		.msg_index = msg_index,
>   		.param = *data,
>   	};
> -	int ret;
>   
>   	assert(signature);
>   	assert(data);