diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2011-11-16 16:12:14 -0500 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-11-16 16:12:14 -0500 |
| commit | c13344958780b4046305ee6235d686c846535529 (patch) | |
| tree | aed2958283867030aa9e9f742c3fb94c895b2d5c /fs/btrfs/super.c | |
| parent | 8d514bbf37eecf0a3e309284728637816a36764b (diff) | |
| download | op-kernel-dev-c13344958780b4046305ee6235d686c846535529.zip op-kernel-dev-c13344958780b4046305ee6235d686c846535529.tar.gz | |
switch create_mnt_ns() to saner calling conventions, fix double mntput() in nfs
Life is much saner if create_mnt_ns(mnt) drops mnt in case of error...
Switch it to such calling conventions, switch callers, fix double mntput() in
fs/nfs/super.c one.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/btrfs/super.c')
| -rw-r--r-- | fs/btrfs/super.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 969a774..cfbedd7 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -843,10 +843,8 @@ static struct dentry *mount_subvol(const char *subvol_name, int flags, return ERR_CAST(mnt); ns_private = create_mnt_ns(mnt); - if (IS_ERR(ns_private)) { - mntput(mnt); + if (IS_ERR(ns_private)) return ERR_CAST(ns_private); - } /* * This will trigger the automount of the subvol so we can just |
