Skip to content

Commit 65dfbbb

Browse files
CKI Backport BotAbhi Das
authored andcommitted
clone_private_mnt(): make sure that caller has CAP_SYS_ADMIN in the right userns
JIRA: https://issues.redhat.com/browse/RHEL-129257 CVE: CVE-2025-38499 commit c28f922 Author: Al Viro <viro@zeniv.linux.org.uk> Date: Sun Jun 1 20:11:06 2025 -0400 clone_private_mnt(): make sure that caller has CAP_SYS_ADMIN in the right userns What we want is to verify there is that clone won't expose something hidden by a mount we wouldn't be able to undo. "Wouldn't be able to undo" may be a result of MNT_LOCKED on a child, but it may also come from lacking admin rights in the userns of the namespace mount belongs to. clone_private_mnt() checks the former, but not the latter. There's a number of rather confusing CAP_SYS_ADMIN checks in various userns during the mount, especially with the new mount API; they serve different purposes and in case of clone_private_mnt() they usually, but not always end up covering the missing check mentioned above. Reviewed-by: Christian Brauner <brauner@kernel.org> Reported-by: "Orlando, Noah" <Noah.Orlando@deshaw.com> Fixes: 427215d ("ovl: prevent private clone if bind mount is not allowed") Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: CKI Backport Bot <cki-ci-bot+cki-gitlab-backport-bot@redhat.com> Signed-off-by: Abhi Das <adas@redhat.com>
1 parent 2c141fa commit 65dfbbb

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

fs/namespace.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2004,6 +2004,11 @@ struct vfsmount *clone_private_mount(const struct path *path)
20042004
if (!check_mnt(old_mnt))
20052005
goto invalid;
20062006

2007+
if (!ns_capable(old_mnt->mnt_ns->user_ns, CAP_SYS_ADMIN)) {
2008+
up_read(&namespace_sem);
2009+
return ERR_PTR(-EPERM);
2010+
}
2011+
20072012
if (__has_locked_children(old_mnt, path->dentry))
20082013
goto invalid;
20092014

0 commit comments

Comments
 (0)