diff mbox series

[net-next,v1] sunrpc: xprtrdma: Use ERR_CAST() to return

Message ID 20240830014356.3465470-1-yanzhen@vivo.com (mailing list archive)
State New
Headers show
Series [net-next,v1] sunrpc: xprtrdma: Use ERR_CAST() to return | expand

Commit Message

Yan Zhen Aug. 30, 2024, 1:43 a.m. UTC 
Using ERR_CAST() is more reasonable and safer, When it is necessary
to convert the type of an error pointer and return it.

Signed-off-by: Yan Zhen <yanzhen@vivo.com>
---
 net/sunrpc/xprtrdma/svc_rdma_transport.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Chuck Lever Aug. 30, 2024, 2:33 p.m. UTC | #1 
On Fri, 30 Aug 2024 09:43:56 +0800, Yan Zhen wrote:
> Using ERR_CAST() is more reasonable and safer, When it is necessary
> to convert the type of an error pointer and return it.
> 
> 

Applied to nfsd-next for v6.12, thanks!

[1/1] sunrpc: xprtrdma: Use ERR_CAST() to return
      commit: a1499a6890cd3551e9b04c3ec55774236188cc62
diff mbox series

Patch

diff --git a/net/sunrpc/xprtrdma/svc_rdma_transport.c b/net/sunrpc/xprtrdma/svc_rdma_transport.c
index 581cc5ed7c0c..c3fbf0779d4a 100644
--- a/net/sunrpc/xprtrdma/svc_rdma_transport.c
+++ b/net/sunrpc/xprtrdma/svc_rdma_transport.c
@@ -369,7 +369,7 @@  static struct svc_xprt *svc_rdma_create(struct svc_serv *serv,
 	listen_id = svc_rdma_create_listen_id(net, sa, cma_xprt);
 	if (IS_ERR(listen_id)) {
 		kfree(cma_xprt);
-		return (struct svc_xprt *)listen_id;
+		return ERR_CAST(listen_id);
 	}
 	cma_xprt->sc_cm_id = listen_id;