diff options
| author | Chuck Lever | 2007-07-01 18:12:35 +0200 |
|---|---|---|
| committer | Trond Myklebust | 2007-07-11 05:40:44 +0200 |
| commit | 0655960f76922a720ad14a510ed91a51395e742b (patch) | |
| tree | a1dee04fc1ebaddb2a700627f3c9eb57df2938c7 /fs/nfs/super.c | |
| parent | NFS: Clean-up: Replace nfs_copy_user_string with strndup_user (diff) | |
| download | kernel-qcow2-linux-0655960f76922a720ad14a510ed91a51395e742b.tar.gz kernel-qcow2-linux-0655960f76922a720ad14a510ed91a51395e742b.tar.xz kernel-qcow2-linux-0655960f76922a720ad14a510ed91a51395e742b.zip | |
NFS: Clean up error handling in nfs_get_sb
The error return logic in nfs_get_sb now matches nfs4_get_sb, and is more maintainable.
A subsequent patch will take advantage of this simplification.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/super.c')
| -rw-r--r-- | fs/nfs/super.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/fs/nfs/super.c b/fs/nfs/super.c index 04ad881eac77..aab5cd61725d 100644 --- a/fs/nfs/super.c +++ b/fs/nfs/super.c @@ -633,13 +633,13 @@ static int nfs_get_sb(struct file_system_type *fs_type, /* Validate the mount data */ error = nfs_validate_mount_data(data, &mntfh); if (error < 0) - return error; + goto out; /* Get a volume representation */ server = nfs_create_server(data, &mntfh); if (IS_ERR(server)) { error = PTR_ERR(server); - goto out_err_noserver; + goto out; } /* Get a superblock - note that we may end up sharing one that already exists */ @@ -669,17 +669,19 @@ static int nfs_get_sb(struct file_system_type *fs_type, s->s_flags |= MS_ACTIVE; mnt->mnt_sb = s; mnt->mnt_root = mntroot; - return 0; + error = 0; + +out: + return error; out_err_nosb: nfs_free_server(server); -out_err_noserver: - return error; + goto out; error_splat_super: up_write(&s->s_umount); deactivate_super(s); - return error; + goto out; } /* |