diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-03-19 21:17:45 +0100 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-03-22 10:34:12 +0100 |
| commit | ff0901f8036a1586037c30a365c9666e946af0f1 (patch) | |
| tree | 0ac6bf41e2d0837dfa7107052f741d78a08a6dd7 /net/sunrpc/clnt.c | |
| parent | SUNRPC: Fix a use after free bug with the NFSv4.1 backchannel (diff) | |
| download | linux-ff0901f8036a1586037c30a365c9666e946af0f1.tar.xz linux-ff0901f8036a1586037c30a365c9666e946af0f1.zip | |
SUNRPC: Fix the return value of rpc_run_bc_task()
Currently rpc_run_bc_task() will return NULL if the task allocation failed.
However the only caller is bc_send, which assumes that the return value
will be an ERR_PTR.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/clnt.c')
| -rw-r--r-- | net/sunrpc/clnt.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/sunrpc/clnt.c b/net/sunrpc/clnt.c index 154034b675bd..19c9983d5360 100644 --- a/net/sunrpc/clnt.c +++ b/net/sunrpc/clnt.c @@ -659,6 +659,7 @@ struct rpc_task *rpc_run_bc_task(struct rpc_rqst *req, task = rpc_new_task(&task_setup_data); if (!task) { xprt_free_bc_request(req); + task = ERR_PTR(-ENOMEM); goto out; } task->tk_rqstp = req; |