diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-03-26 02:24:35 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-03-27 11:48:17 -0400 |
commit | a0e7e3cf7932d6c77de0dc79a40dbaeb8060b544 (patch) | |
tree | fcc8ab44418afe356b683674fcd2c1589b7c6efa /fs/nfs/namespace.c | |
parent | 4d65c520fb4abed970069d18c119cfe85624f46d (diff) |
NFS: Don't leak RPC clients in NFSv4 secinfo negotiation
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/namespace.c')
-rw-r--r-- | fs/nfs/namespace.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/nfs/namespace.c b/fs/nfs/namespace.c index ad92bf731ff5..9166fcb66da2 100644 --- a/fs/nfs/namespace.c +++ b/fs/nfs/namespace.c | |||
@@ -192,13 +192,15 @@ static rpc_authflavor_t nfs_lookup_with_sec(struct nfs_server *server, struct de | |||
192 | auth = rpcauth_create(flavor, clone); | 192 | auth = rpcauth_create(flavor, clone); |
193 | if (!auth) { | 193 | if (!auth) { |
194 | flavor = -EIO; | 194 | flavor = -EIO; |
195 | goto out; | 195 | goto out_shutdown; |
196 | } | 196 | } |
197 | err = server->nfs_client->rpc_ops->lookup(clone, parent->d_inode, | 197 | err = server->nfs_client->rpc_ops->lookup(clone, parent->d_inode, |
198 | &path->dentry->d_name, | 198 | &path->dentry->d_name, |
199 | fh, fattr); | 199 | fh, fattr); |
200 | if (err < 0) | 200 | if (err < 0) |
201 | flavor = err; | 201 | flavor = err; |
202 | out_shutdown: | ||
203 | rpc_shutdown_client(clone); | ||
202 | out: | 204 | out: |
203 | return flavor; | 205 | return flavor; |
204 | } | 206 | } |