]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
mnt: Don't propagate umounts in __detach_mounts
authorEric W. Biederman <ebiederm@xmission.com>
Wed, 24 Dec 2014 13:35:10 +0000 (07:35 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 6 May 2015 20:03:50 +0000 (22:03 +0200)
commit 8318e667f176f7ea34451a1a530634e293f216ac upstream.

Invoking mount propagation from __detach_mounts is inefficient and
wrong.

It is inefficient because __detach_mounts already walks the list of
mounts that where something needs to be done, and mount propagation
walks some subset of those mounts again.

It is actively wrong because if the dentry that is passed to
__detach_mounts is not part of the path to a mount that mount should
not be affected.

change_mnt_propagation(p,MS_PRIVATE) modifies the mount propagation
tree of a master mount so it's slaves are connected to another master
if possible.  Which means even removing a mount from the middle of a
mount tree with __detach_mounts will not deprive any mount propagated
mount events.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/namespace.c

index 712b3c5bddd6d7cb06a398bc035f5fd8367267d2..616a694391d68116a13888ad0a0f5a3120d16ac9 100644 (file)
@@ -1487,7 +1487,7 @@ void __detach_mounts(struct dentry *dentry)
        lock_mount_hash();
        while (!hlist_empty(&mp->m_list)) {
                mnt = hlist_entry(mp->m_list.first, struct mount, mnt_mp_list);
-               umount_tree(mnt, UMOUNT_PROPAGATE);
+               umount_tree(mnt, 0);
        }
        unlock_mount_hash();
        put_mountpoint(mp);