diff mbox

rxrpc: fix ptr_ret.cocci warnings

Message ID 20160604231719.GA37147@lkp-hsx03
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

kbuild test robot June 4, 2016, 11:17 p.m. UTC
net/rxrpc/rxkad.c:1165:1-3: WARNING: PTR_ERR_OR_ZERO can be used


 Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR

Generated by: scripts/coccinelle/api/ptr_ret.cocci

CC: David Howells <dhowells@redhat.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---

 rxkad.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

David Miller June 7, 2016, 10:30 p.m. UTC | #1
From: kbuild test robot <fengguang.wu@intel.com>
Date: Sun, 5 Jun 2016 07:17:19 +0800

> net/rxrpc/rxkad.c:1165:1-3: WARNING: PTR_ERR_OR_ZERO can be used
> 
> 
>  Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
> 
> Generated by: scripts/coccinelle/api/ptr_ret.cocci
> 
> CC: David Howells <dhowells@redhat.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Applied.
David Howells June 9, 2016, 2:38 p.m. UTC | #2
kbuild test robot <fengguang.wu@intel.com> wrote:

> net/rxrpc/rxkad.c:1165:1-3: WARNING: PTR_ERR_OR_ZERO can be used
> 
> 
>  Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
> 
> Generated by: scripts/coccinelle/api/ptr_ret.cocci
> 
> CC: David Howells <dhowells@redhat.com>
> Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>

Acked-by: David Howells <dhowells@redhat.com>
diff mbox

Patch

--- a/net/rxrpc/rxkad.c
+++ b/net/rxrpc/rxkad.c
@@ -1162,9 +1162,7 @@  static int rxkad_init(void)
 	/* pin the cipher we need so that the crypto layer doesn't invoke
 	 * keventd to go get it */
 	rxkad_ci = crypto_alloc_skcipher("pcbc(fcrypt)", 0, CRYPTO_ALG_ASYNC);
-	if (IS_ERR(rxkad_ci))
-		return PTR_ERR(rxkad_ci);
-	return 0;
+	return PTR_ERR_OR_ZERO(rxkad_ci);
 }
 
 /*