diff options
| author | Christian Brauner <brauner@kernel.org> | 2024-08-06 19:02:30 +0300 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2024-08-30 09:22:12 +0300 |
| commit | 3836b31c3e717fcaa2b63c69e4bcf8ff7b82e6e4 (patch) | |
| tree | e7c3f9780c5cf6e540b4fa2d1f57ed72f2b998ec | |
| parent | 32a0a965b8a5b7b505babae2ee10c513ede376d8 (diff) | |
| download | linux-3836b31c3e717fcaa2b63c69e4bcf8ff7b82e6e4.tar.xz | |
proc: block mounting on top of /proc/<pid>/map_files/*
Entries under /proc/<pid>/map_files/* are ephemeral and may go away
before the process dies. As such allowing them to be used as mount
points creates the ability to leak mounts that linger until the process
dies with no ability to unmount them until then. Don't allow using them
as mountpoints.
Link: https://lore.kernel.org/r/20240806-work-procfs-v1-4-fb04e1d09f0c@kernel.org
Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
| -rw-r--r-- | fs/proc/base.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/proc/base.c b/fs/proc/base.c index 72a1acd03675..fce3d377b826 100644 --- a/fs/proc/base.c +++ b/fs/proc/base.c @@ -2276,8 +2276,8 @@ proc_map_files_instantiate(struct dentry *dentry, inode->i_op = &proc_map_files_link_inode_operations; inode->i_size = 64; - d_set_d_op(dentry, &tid_map_files_dentry_operations); - return d_splice_alias(inode, dentry); + return proc_splice_unmountable(inode, dentry, + &tid_map_files_dentry_operations); } static struct dentry *proc_map_files_lookup(struct inode *dir, |