[lkml]   [2018]   [Mar]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Subject[PATCH v9 0/4] fuse: mounts from non-init user namespaces

This patchset builds on the work by Donsu Park and Seth Forshee and is
reduced to the set of patches that just affect fuse. The non-fuse
vfs patches are far enough along we can ignore them except possibly for the
question of when does FS_USERNS_MOUNT get set in fuse_fs_type.

Fuse with a block device has been left as an exercise for a later time.

Since v5 I changed the core of this patchset around as the previous
patches were showing signs of bitrot. Some important explanations were
missing, some important functionality was missing, and xattr handling
was completely absent.

Since v6 I have:
- Removed the failure case from fuse_get_req_nofail_nopages that I
- Updated fuse to always to use posix_acl_access_xattr_handler, and
posix_acl_default_xattr_handler, by teaching fuse to set

Since v7 I have:
- Rethought and reworked how I am unifying the cached and the non-cached
posix acl case so the code is cleaner and simpler.
- I have dropped enhancements to caching negative acls when
fc->no_getxattr is set.
- Removed the need to wrap forget_all_cached_acls in fuse.
- Reorder the patches so the posix acl work comes first

Since v8 I have:
- Dropped and postponed the unification of the uncached and the cached
posix acls case. The code is not hard but tricky enough it needs
to be considered on it's own on it's own merits.

Miklos can you take a look and see what you think?

Miklos if you could pick these up I would appreciate it. If not I can
merge these through the userns tree.

These changes are also available at:

git:// userns-fuse-v9

Eric W. Biederman (3):
fuse: Remove the buggy retranslation of pids in fuse_dev_do_read
fuse: Fail all requests with invalid uids or gids
fuse: Support fuse filesystems outside of init_user_ns

Seth Forshee (1):
fuse: Restrict allow_other to the superblock's namespace or a descendant

fs/fuse/acl.c | 4 ++--
fs/fuse/cuse.c | 7 ++++++-
fs/fuse/dev.c | 30 +++++++++++++++++-------------
fs/fuse/dir.c | 16 ++++++++--------
fs/fuse/fuse_i.h | 6 +++++-
fs/fuse/inode.c | 31 +++++++++++++++++++------------
kernel/user_namespace.c | 1 +
7 files changed, 58 insertions(+), 37 deletions(-)

 \ /
  Last update: 2018-03-08 22:24    [W:0.227 / U:13.412 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site