diff options
author | Dan Carpenter <error27@gmail.com> | 2010-04-22 05:28:39 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-04-28 13:45:12 -0400 |
commit | acf82b85a70f39786e3cbb1ffed8655bcc972424 (patch) | |
tree | 99e1a7b0815aa1ca1d9196a323c9c28df185d2aa | |
parent | ba8b06e67ed7a560b0e7c80091bcadda4f4727a5 (diff) |
nfs: fix some issues in nfs41_proc_reclaim_complete()
The original code passed an ERR_PTR() to rpc_put_task() and instead of
returning zero on success it returned -ENOMEM.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r-- | fs/nfs/nfs4proc.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 638067007c65..071fcedd517c 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c | |||
@@ -5218,9 +5218,12 @@ static int nfs41_proc_reclaim_complete(struct nfs_client *clp) | |||
5218 | msg.rpc_resp = &calldata->res; | 5218 | msg.rpc_resp = &calldata->res; |
5219 | task_setup_data.callback_data = calldata; | 5219 | task_setup_data.callback_data = calldata; |
5220 | task = rpc_run_task(&task_setup_data); | 5220 | task = rpc_run_task(&task_setup_data); |
5221 | if (IS_ERR(task)) | 5221 | if (IS_ERR(task)) { |
5222 | status = PTR_ERR(task); | 5222 | status = PTR_ERR(task); |
5223 | goto out; | ||
5224 | } | ||
5223 | rpc_put_task(task); | 5225 | rpc_put_task(task); |
5226 | return 0; | ||
5224 | out: | 5227 | out: |
5225 | dprintk("<-- %s status=%d\n", __func__, status); | 5228 | dprintk("<-- %s status=%d\n", __func__, status); |
5226 | return status; | 5229 | return status; |