diff options
author | Anna Schumaker <Anna.Schumaker@Netapp.com> | 2017-01-11 22:30:08 +0100 |
---|---|---|
committer | Anna Schumaker <Anna.Schumaker@Netapp.com> | 2017-01-30 19:14:50 +0100 |
commit | 9df1336ca42c16622f1cae5dff246a0c9c644514 (patch) | |
tree | f98be7f1e17798d3cac4c3bc86477e1065821962 | |
parent | NFS: Remove nfs4_recover_expired_lease() (diff) | |
download | linux-9df1336ca42c16622f1cae5dff246a0c9c644514.tar.xz linux-9df1336ca42c16622f1cae5dff246a0c9c644514.zip |
NFS: Remove unnecessary goto in nfs4_lookup_root_sec()
Once again, it's easier and cleaner just to return the error directly.
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
-rw-r--r-- | fs/nfs/nfs4proc.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index baf968990475..6c40944cb83d 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c @@ -3416,16 +3416,11 @@ static int nfs4_lookup_root_sec(struct nfs_server *server, struct nfs_fh *fhandl .pseudoflavor = flavor, }; struct rpc_auth *auth; - int ret; auth = rpcauth_create(&auth_args, server->client); - if (IS_ERR(auth)) { - ret = -EACCES; - goto out; - } - ret = nfs4_lookup_root(server, fhandle, info); -out: - return ret; + if (IS_ERR(auth)) + return -EACCES; + return nfs4_lookup_root(server, fhandle, info); } /* |