diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-05-31 15:26:38 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2012-06-19 23:18:08 +0100 |
| commit | 0997627ec97c5159f383df2b64f4c8e7fbe71a7b (patch) | |
| tree | 9abe84b531ea45b9040fd86e03a648655d1a51c2 /net/sunrpc/svc.c | |
| parent | 3fed6d5841445d9d3a4266de48e9047a153d029e (diff) | |
NFSv4.1: Fix a request leak on the back channel
commit b3b02ae5865c2dcd506322e0fc6def59a042e72f upstream.
If the call to svc_process_common() fails, then the request
needs to be freed before we can exit bc_svc_process.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'net/sunrpc/svc.c')
| -rw-r--r-- | net/sunrpc/svc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/sunrpc/svc.c b/net/sunrpc/svc.c index d4ad50ece9b..c80c16248a3 100644 --- a/net/sunrpc/svc.c +++ b/net/sunrpc/svc.c @@ -1358,7 +1358,8 @@ bc_svc_process(struct svc_serv *serv, struct rpc_rqst *req, sizeof(req->rq_snd_buf)); return bc_send(req); } else { - /* Nothing to do to drop request */ + /* drop request */ + xprt_free_bc_request(req); return 0; } } |