Seth Forshee
2014-10-22 21:24:16 UTC
Here's another update to the patches to enable userns mounts in fuse.
The changes this time around center on xattrs and allow_other. I'm
considering the following patch to be a prerequisite for this series:
http://lkml.kernel.org/g/***@amacapital.net
With this patch I'm dropping the xattr restrictions from the v4 patches.
This leaves the situation with xattrs unchanged in init_user_ns. A fuse
userns mount could still supply xattrs which the user would normally be
unable to set, but since the mount will be inaccessible outside of that
userns or its descendants those xattrs will never be seen by the host.
I'm also in agreement with Andy that some decision regarding this patch
should be made before merging the fuse userns support as well:
http://lkml.kernel.org/g/***@amacapital.net
Changes since v4:
- Drop patch to restrict xattrs
- Update allow_other to restrict based on userns of current rather than
uid/gid
Thanks,
Seth
Seth Forshee (4):
fuse: Add support for pid namespaces
fuse: Support fuse filesystems outside of init_user_ns
fuse: Restrict allow_other to the superblock's namespace or a
descendant
fuse: Allow user namespace mounts
fs/fuse/dev.c | 13 ++++----
fs/fuse/dir.c | 91 +++++++++++++++++++++++++++++++++++++-------------------
fs/fuse/file.c | 38 ++++++++++++++++-------
fs/fuse/fuse_i.h | 16 +++++++---
fs/fuse/inode.c | 81 ++++++++++++++++++++++++++++++++++++++-----------
5 files changed, 171 insertions(+), 68 deletions(-)
The changes this time around center on xattrs and allow_other. I'm
considering the following patch to be a prerequisite for this series:
http://lkml.kernel.org/g/***@amacapital.net
With this patch I'm dropping the xattr restrictions from the v4 patches.
This leaves the situation with xattrs unchanged in init_user_ns. A fuse
userns mount could still supply xattrs which the user would normally be
unable to set, but since the mount will be inaccessible outside of that
userns or its descendants those xattrs will never be seen by the host.
I'm also in agreement with Andy that some decision regarding this patch
should be made before merging the fuse userns support as well:
http://lkml.kernel.org/g/***@amacapital.net
Changes since v4:
- Drop patch to restrict xattrs
- Update allow_other to restrict based on userns of current rather than
uid/gid
Thanks,
Seth
Seth Forshee (4):
fuse: Add support for pid namespaces
fuse: Support fuse filesystems outside of init_user_ns
fuse: Restrict allow_other to the superblock's namespace or a
descendant
fuse: Allow user namespace mounts
fs/fuse/dev.c | 13 ++++----
fs/fuse/dir.c | 91 +++++++++++++++++++++++++++++++++++++-------------------
fs/fuse/file.c | 38 ++++++++++++++++-------
fs/fuse/fuse_i.h | 16 +++++++---
fs/fuse/inode.c | 81 ++++++++++++++++++++++++++++++++++++++-----------
5 files changed, 171 insertions(+), 68 deletions(-)