From: Greg Kroah-Hartman Date: Sat, 2 May 2020 06:18:27 +0000 (+0200) Subject: 4.14-stable patches X-Git-Tag: v5.4.37~2 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=0d7ec76bc033e74621967ba02001ba711490a8d1;p=thirdparty%2Fkernel%2Fstable-queue.git 4.14-stable patches added patches: propagate_one-mnt_set_mountpoint-needs-mount_lock.patch --- diff --git a/queue-4.14/propagate_one-mnt_set_mountpoint-needs-mount_lock.patch b/queue-4.14/propagate_one-mnt_set_mountpoint-needs-mount_lock.patch new file mode 100644 index 00000000000..86078d97b8d --- /dev/null +++ b/queue-4.14/propagate_one-mnt_set_mountpoint-needs-mount_lock.patch @@ -0,0 +1,46 @@ +From b0d3869ce9eeacbb1bbd541909beeef4126426d5 Mon Sep 17 00:00:00 2001 +From: Al Viro +Date: Mon, 27 Apr 2020 10:26:22 -0400 +Subject: propagate_one(): mnt_set_mountpoint() needs mount_lock + +From: Al Viro + +commit b0d3869ce9eeacbb1bbd541909beeef4126426d5 upstream. + +... to protect the modification of mp->m_count done by it. Most of +the places that modify that thing also have namespace_lock held, +but not all of them can do so, so we really need mount_lock here. +Kudos to Piotr Krysiuk , who'd spotted a related +bug in pivot_root(2) (fixed unnoticed in 5.3); search for other +similar turds has caught out this one. + +Cc: stable@kernel.org +Signed-off-by: Al Viro +Signed-off-by: Piotr Krysiuk +Signed-off-by: Greg Kroah-Hartman + +--- + fs/pnode.c | 9 ++++----- + 1 file changed, 4 insertions(+), 5 deletions(-) + +--- a/fs/pnode.c ++++ b/fs/pnode.c +@@ -266,14 +266,13 @@ static int propagate_one(struct mount *m + if (IS_ERR(child)) + return PTR_ERR(child); + child->mnt.mnt_flags &= ~MNT_LOCKED; ++ read_seqlock_excl(&mount_lock); + mnt_set_mountpoint(m, mp, child); ++ if (m->mnt_master != dest_master) ++ SET_MNT_MARK(m->mnt_master); ++ read_sequnlock_excl(&mount_lock); + last_dest = m; + last_source = child; +- if (m->mnt_master != dest_master) { +- read_seqlock_excl(&mount_lock); +- SET_MNT_MARK(m->mnt_master); +- read_sequnlock_excl(&mount_lock); +- } + hlist_add_head(&child->mnt_hash, list); + return count_mounts(m->mnt_ns, child); + } diff --git a/queue-4.14/series b/queue-4.14/series index 4e4c6b236f8..a924cfcb9b1 100644 --- a/queue-4.14/series +++ b/queue-4.14/series @@ -114,3 +114,4 @@ ext4-fix-block-validity-checks-for-journal-inodes-using-indirect-blocks.patch ext4-unsigned-int-compared-against-zero.patch qed-fix-use-after-free-in-qed_chain_free.patch ext4-check-for-non-zero-journal-inum-in-ext4_calculate_overhead.patch +propagate_one-mnt_set_mountpoint-needs-mount_lock.patch