diff options
author | Adrian Bunk <bunk@kernel.org> | 2007-11-14 20:00:00 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-14 21:45:41 -0500 |
commit | d5cd97872dca9b79c31224ca014bcea7ca01f5f1 (patch) | |
tree | 1d2252ebddd62fb8d6669385e58e27b489c30dba | |
parent | e02f5f52cafbea013817d81f1acc4baf50d6324b (diff) |
sunrpc/xprtrdma/transport.c: fix use-after-free
Fix an obvious use-after-free spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@kernel.org>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Neil Brown <neilb@suse.de>
Cc: "David S. Miller" <davem@davemloft.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | net/sunrpc/xprtrdma/transport.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/xprtrdma/transport.c b/net/sunrpc/xprtrdma/transport.c index dc55cc974c90..1afeb3eb8e4c 100644 --- a/net/sunrpc/xprtrdma/transport.c +++ b/net/sunrpc/xprtrdma/transport.c | |||
@@ -320,9 +320,9 @@ xprt_setup_rdma(struct xprt_create *args) | |||
320 | xprt->slot = kcalloc(xprt->max_reqs, | 320 | xprt->slot = kcalloc(xprt->max_reqs, |
321 | sizeof(struct rpc_rqst), GFP_KERNEL); | 321 | sizeof(struct rpc_rqst), GFP_KERNEL); |
322 | if (xprt->slot == NULL) { | 322 | if (xprt->slot == NULL) { |
323 | kfree(xprt); | ||
324 | dprintk("RPC: %s: couldn't allocate %d slots\n", | 323 | dprintk("RPC: %s: couldn't allocate %d slots\n", |
325 | __func__, xprt->max_reqs); | 324 | __func__, xprt->max_reqs); |
325 | kfree(xprt); | ||
326 | return ERR_PTR(-ENOMEM); | 326 | return ERR_PTR(-ENOMEM); |
327 | } | 327 | } |
328 | 328 | ||