diff options
| author | Christian Brauner <brauner@kernel.org> | 2024-06-24 11:49:45 -0400 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-06-28 09:53:30 +0200 |
| commit | f3107df39df123328a9d3c8f40c006834b37287d (patch) | |
| tree | 31786fcfcaefe4b6dfe644b36486fa5d651b52b0 /fs/namespace.c | |
| parent | dd7cb142f467c4660698bcaa4a48c688b443ab81 (diff) | |
| download | linux-stable-f3107df39df123328a9d3c8f40c006834b37287d.tar.gz linux-stable-f3107df39df123328a9d3c8f40c006834b37287d.tar.bz2 linux-stable-f3107df39df123328a9d3c8f40c006834b37287d.zip | |
fs: relax permissions for statmount()
It is sufficient to have capabilities in the owning user namespace of
the mount namespace to stat a mount regardless of whether it's reachable
or not.
Link: https://lore.kernel.org/r/bf5961d71ec479ba85806766b0d8d96043e67bba.1719243756.git.josef@toxicpanda.com
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/namespace.c')
| -rw-r--r-- | fs/namespace.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/namespace.c b/fs/namespace.c index 22274f74beb0..352e8c90b9f7 100644 --- a/fs/namespace.c +++ b/fs/namespace.c @@ -4930,6 +4930,7 @@ static int copy_statmount_to_user(struct kstatmount *s) static int do_statmount(struct kstatmount *s) { struct mount *m = real_mount(s->mnt); + struct mnt_namespace *ns = m->mnt_ns; int err; /* @@ -4937,7 +4938,7 @@ static int do_statmount(struct kstatmount *s) * mounts to show users. */ if (!is_path_reachable(m, m->mnt.mnt_root, &s->root) && - !ns_capable_noaudit(&init_user_ns, CAP_SYS_ADMIN)) + !ns_capable_noaudit(ns->user_ns, CAP_SYS_ADMIN)) return -EPERM; err = security_sb_statfs(s->mnt->mnt_root); |