diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-11-05 03:39:52 -0500 |
---|---|---|
committer | Anna Schumaker <Anna.Schumaker@Netapp.com> | 2015-12-18 15:34:33 -0500 |
commit | abfb689711aaebd14d893236c6ea4bcdfb61e74c (patch) | |
tree | e1f39a31fd6b8115f7202de5c7ee97dd9b890028 | |
parent | 38b95bcf122545db7035a06d79ec9e851be2e011 (diff) |
xprtrdma: checking for NULL instead of IS_ERR()
The rpcrdma_create_req() function returns error pointers or success. It
never returns NULL.
Fixes: f531a5dbc451 ('xprtrdma: Pre-allocate backward rpc_rqst and send/receive buffers')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
-rw-r--r-- | net/sunrpc/xprtrdma/backchannel.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sunrpc/xprtrdma/backchannel.c b/net/sunrpc/xprtrdma/backchannel.c index 2dcb44f69e53..97554ca68191 100644 --- a/net/sunrpc/xprtrdma/backchannel.c +++ b/net/sunrpc/xprtrdma/backchannel.c | |||
@@ -42,8 +42,8 @@ static int rpcrdma_bc_setup_rqst(struct rpcrdma_xprt *r_xprt, | |||
42 | size_t size; | 42 | size_t size; |
43 | 43 | ||
44 | req = rpcrdma_create_req(r_xprt); | 44 | req = rpcrdma_create_req(r_xprt); |
45 | if (!req) | 45 | if (IS_ERR(req)) |
46 | return -ENOMEM; | 46 | return PTR_ERR(req); |
47 | req->rl_backchannel = true; | 47 | req->rl_backchannel = true; |
48 | 48 | ||
49 | size = RPCRDMA_INLINE_WRITE_THRESHOLD(rqst); | 49 | size = RPCRDMA_INLINE_WRITE_THRESHOLD(rqst); |