diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-03-19 16:17:45 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-03-22 05:34:12 -0400 |
commit | ff0901f8036a1586037c30a365c9666e946af0f1 (patch) | |
tree | 0ac6bf41e2d0837dfa7107052f741d78a08a6dd7 /net/sunrpc/clnt.c | |
parent | c9acb42ef1904d15d0fb315061cefbe638f67f3a (diff) | |
download | op-kernel-dev-ff0901f8036a1586037c30a365c9666e946af0f1.zip op-kernel-dev-ff0901f8036a1586037c30a365c9666e946af0f1.tar.gz |
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 154034b..19c9983 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; |