diff mbox

crypto: fix semicolon.cocci warnings

Message ID 20160615111325.GA68889@ivytown2 (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show

Commit Message

Fengguang Wu June 15, 2016, 11:13 a.m. UTC
crypto/drbg.c:1637:39-40: Unneeded semicolon


 Remove unneeded semicolon.

Generated by: scripts/coccinelle/misc/semicolon.cocci

CC: Stephan Mueller <smueller@chronox.de>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

 drbg.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Stephan Mueller June 15, 2016, 11:40 a.m. UTC | #1
Am Mittwoch, 15. Juni 2016, 19:13:25 schrieb kbuild test robot:

Hi Fengguang,

> crypto/drbg.c:1637:39-40: Unneeded semicolon
> 
> 
>  Remove unneeded semicolon.

Thank you!

> 
> Generated by: scripts/coccinelle/misc/semicolon.cocci
> 
> CC: Stephan Mueller <smueller@chronox.de>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: Stephan Mueller <smueller@chronox.de>
> ---
> 
>  drbg.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- a/crypto/drbg.c
> +++ b/crypto/drbg.c
> @@ -1634,7 +1634,7 @@ static int drbg_fini_sym_kernel(struct d
>  	drbg->ctr_handle = NULL;
> 
>  	if (drbg->ctr_req)
> -		skcipher_request_free(drbg->ctr_req);;
> +		skcipher_request_free(drbg->ctr_req);
>  	drbg->ctr_req = NULL;
> 
>  	kfree(drbg->ctr_null_value_buf);


Ciao
Stephan
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Herbert Xu June 20, 2016, 11:33 a.m. UTC | #2
On Wed, Jun 15, 2016 at 07:13:25PM +0800, kbuild test robot wrote:
> crypto/drbg.c:1637:39-40: Unneeded semicolon
> 
> 
>  Remove unneeded semicolon.
> 
> Generated by: scripts/coccinelle/misc/semicolon.cocci
> 
> CC: Stephan Mueller <smueller@chronox.de>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Patch applied.  Thanks.
diff mbox

Patch

--- a/crypto/drbg.c
+++ b/crypto/drbg.c
@@ -1634,7 +1634,7 @@  static int drbg_fini_sym_kernel(struct d
 	drbg->ctr_handle = NULL;
 
 	if (drbg->ctr_req)
-		skcipher_request_free(drbg->ctr_req);;
+		skcipher_request_free(drbg->ctr_req);
 	drbg->ctr_req = NULL;
 
 	kfree(drbg->ctr_null_value_buf);