Commit 5ac118ee authored by Eric W. Biederman's avatar Eric W. Biederman Committed by Luis Henriques

mnt: Update fs_fully_visible to test for permanently empty directories

commit 7236c85e upstream.

fs_fully_visible attempts to make fresh mounts of proc and sysfs give
the mounter no more access to proc and sysfs than if they could have
by creating a bind mount.  One aspect of proc and sysfs that makes
this particularly tricky is that there are other filesystems that
typically mount on top of proc and sysfs.  As those filesystems are
mounted on empty directories in practice it is safe to ignore them.
However testing to ensure filesystems are mounted on empty directories
has not been something the in kernel data structures have supported so
the current test for an empty directory which checks to see
if nlink <= 2 is a bit lacking.

proc and sysfs have recently been modified to use the new empty_dir
infrastructure to create all of their dedicated mount points.  Instead
of testing for S_ISDIR(inode->i_mode) && i_nlink <= 2 to see if a
directory is empty, test for is_empty_dir_inode(inode).  That small
change guaranteess mounts found on proc and sysfs really are safe to
ignore, because the directories are not only empty but nothing can
ever be added to them.  This guarantees there is nothing to worry
about when mounting proc and sysfs.
Signed-off-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent ccf34007
...@@ -3073,9 +3073,8 @@ static bool fs_fully_visible(struct file_system_type *type, int *new_mnt_flags) ...@@ -3073,9 +3073,8 @@ static bool fs_fully_visible(struct file_system_type *type, int *new_mnt_flags)
/* Only worry about locked mounts */ /* Only worry about locked mounts */
if (!(mnt->mnt.mnt_flags & MNT_LOCKED)) if (!(mnt->mnt.mnt_flags & MNT_LOCKED))
continue; continue;
if (!S_ISDIR(inode->i_mode)) /* Is the directory permanetly empty? */
goto next; if (!is_empty_dir_inode(inode))
if (inode->i_nlink > 2)
goto next; goto next;
} }
/* Preserve the locked attributes */ /* Preserve the locked attributes */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment