diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-05-31 15:26:38 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-06-22 11:36:54 -0700 |
| commit | 5f1d81e9fb6c05e8ab89a610bb63cdc8f0f66d81 (patch) | |
| tree | 9c962613c1c1f124f9d0f8e132368c7060f05a47 /net/sunrpc | |
| parent | c7aa88da6b435fb45b01fe6076caf73ee6e529ae (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: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/sunrpc')
| -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 4153846984a..234ee39000a 100644 --- a/net/sunrpc/svc.c +++ b/net/sunrpc/svc.c @@ -1379,7 +1379,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; } } |