Introduce a new mount bind mount property to allow idmapping mounts. The
MOUNT_ATTR_IDMAP flag can be set via the new mount_setattr() syscall
together with a file descriptor referring to a user namespace.

The user namespace referenced by the namespace file descriptor will be
attached to the bind mount. All interactions with the filesystem going
through that mount will be idmapped according to the mapping specified in
the user namespace attached to it.

Using user namespaces to mark mounts means we can reuse all the existing
infrastructure in the kernel that already exists to handle idmappings and can
also use this for permission checking to allow unprivileged user to create
idmapped mounts in the future.

Idmapping a mount is decoupled from the caller's user and mount namespace.
This means idmapped mounts can be created in the initial user namespace
which is an important use-case for systemd-homed, portable usb-sticks
between systems, sharing data between the initial user namespace and
unprivileged containers, and other use-cases that have been brought up. For
example, assume a home directory where all files are owned by uid and gid
1000 and the home directory is brought to a new laptop where the user has
id 12345. The system administrator can simply create a mount of this home
directory with a mapping of 1000:12345:1 other mappings to indicate the ids
should be kept. (With this it is e.g. also possible to create idmapped
mounts on the host with an identity mapping 1:1:100000 where the root user
is not mapped. A user with root access that e.g. has been pivot rooted into
such a mount on the host will be not be able to execute, read, write, or
create files as root.)

Given that idmapping a mount is decoupled from the caller's user namespace
a sufficiently privileged process such as a container manager can set up a
shifted mount for the container and the container can simply pivot root to
it. There's no need for the container to do anything. The mount will appear
correctly mapped independent of the user namespace the container uses. This
means we don't need to mark a mount as idmappable.

In order to create an idmapped mount the caller must currently be privileged in
the user namespace of the superblock the mount belongs to. Currently, once a
mount has been idmapped we don't allow it to change its mapping. This can be
changed in the future if the use-cases arises.

Cc: Christoph Hellwig <h...@lst.de>
Cc: David Howells <dhowe...@redhat.com>
Cc: Mauricio Vásquez Bernal <mauri...@kinvolk.io>
Cc: Al Viro <v...@zeniv.linux.org.uk>
Cc: linux-fsde...@vger.kernel.org
Signed-off-by: Christian Brauner <christian.brau...@ubuntu.com>
---
/* v2 */
- Christoph Hellwig <h...@lst.de>:
  - Drop kconfig option to make vfs idmappings unconditional.
  - Move introduction of MOUNT_ATTR_IDMAP to the end of the series after all
    internal changes have been done.
  - Move MOUNT_ATTR_IDMAP handling from build_mount_kattr() to separate
    build_mount_idmapped() helper.
  - Move MNT_IDMAPPED handling from do_mount_setattr() into separate
    do_mount_idmap() helper.
  - Use more helpers instead of one big function for mount attribute changes.
- Mauricio Vásquez Bernal <mauri...@kinvolk.io>:
  - Recalculate flags before checking can_change_locked_flags().
---
 fs/internal.h              |   1 +
 fs/namespace.c             | 122 ++++++++++++++++++++++++++++++++++++-
 fs/proc_namespace.c        |   1 +
 include/linux/fs.h         |   2 +-
 include/linux/mount.h      |   2 +-
 include/uapi/linux/mount.h |   5 +-
 6 files changed, 128 insertions(+), 5 deletions(-)

diff --git a/fs/internal.h b/fs/internal.h
index a5a6c470dc07..b0978274155f 100644
--- a/fs/internal.h
+++ b/fs/internal.h
@@ -88,6 +88,7 @@ struct mount_kattr {
        unsigned int propagation;
        unsigned int lookup_flags;
        bool recurse;
+       struct user_namespace *mnt_user_ns;
 };
 
 extern struct vfsmount *lookup_mnt(const struct path *);
diff --git a/fs/namespace.c b/fs/namespace.c
index 15fb0ae3f01f..f76292294dbd 100644
--- a/fs/namespace.c
+++ b/fs/namespace.c
@@ -25,6 +25,7 @@
 #include <linux/proc_ns.h>
 #include <linux/magic.h>
 #include <linux/memblock.h>
+#include <linux/proc_fs.h>
 #include <linux/task_work.h>
 #include <linux/sched/task.h>
 #include <uapi/linux/mount.h>
@@ -3465,7 +3466,8 @@ static int build_attr_flags(unsigned int attr_flags, 
unsigned int *flags)
                           MOUNT_ATTR_NODEV |
                           MOUNT_ATTR_NOEXEC |
                           MOUNT_ATTR__ATIME |
-                          MOUNT_ATTR_NODIRATIME))
+                          MOUNT_ATTR_NODIRATIME |
+                          MOUNT_ATTR_IDMAP))
                return -EINVAL;
 
        if (attr_flags & MOUNT_ATTR_RDONLY)
@@ -3478,6 +3480,8 @@ static int build_attr_flags(unsigned int attr_flags, 
unsigned int *flags)
                aflags |= MNT_NOEXEC;
        if (attr_flags & MOUNT_ATTR_NODIRATIME)
                aflags |= MNT_NODIRATIME;
+       if (attr_flags & MOUNT_ATTR_IDMAP)
+               aflags |= MNT_IDMAPPED;
 
        *flags = aflags;
        return 0;
@@ -3505,6 +3509,14 @@ SYSCALL_DEFINE3(fsmount, int, fs_fd, unsigned int, flags,
        if ((flags & ~(FSMOUNT_CLOEXEC)) != 0)
                return -EINVAL;
 
+       if (attr_flags & ~(MOUNT_ATTR_RDONLY |
+                          MOUNT_ATTR_NOSUID |
+                          MOUNT_ATTR_NODEV |
+                          MOUNT_ATTR_NOEXEC |
+                          MOUNT_ATTR__ATIME |
+                          MOUNT_ATTR_NODIRATIME))
+               return -EINVAL;
+
        ret = build_attr_flags(attr_flags, &mnt_flags);
        if (ret)
                return ret;
@@ -3827,6 +3839,32 @@ static unsigned int recalc_flags(struct mount_kattr 
*kattr, struct mount *mnt)
        return flags;
 }
 
+static int can_idmap_mount(const struct mount_kattr *kattr, struct mount *mnt)
+{
+       struct vfsmount *m = &mnt->mnt;
+
+       if (!(kattr->attr_set & MNT_IDMAPPED))
+               return 0;
+
+       /*
+        * Once a mount has been idmapped we don't allow it to change its
+        * mapping. It makes things simpler and callers can just create
+        * another bind-mount they can idmap if they want to.
+        */
+       if (mnt_idmapped(m))
+               return -EPERM;
+
+       /* The underlying filesystem doesn't support idmapped mounts yet. */
+       if (!(m->mnt_sb->s_type->fs_flags & FS_ALLOW_IDMAP))
+               return -EINVAL;
+
+       /* We're controlling the superblock. */
+       if (!ns_capable(m->mnt_sb->s_user_ns, CAP_SYS_ADMIN))
+               return -EPERM;
+
+       return 0;
+}
+
 static struct mount *mount_setattr_prepare(struct mount_kattr *kattr,
                                           struct mount *mnt, int *err)
 {
@@ -3851,6 +3889,10 @@ static struct mount *mount_setattr_prepare(struct 
mount_kattr *kattr,
                        goto out;
                }
 
+               *err = can_idmap_mount(kattr, m);
+               if (*err)
+                       goto out;
+
                last = m;
 
                if ((kattr->attr_set & MNT_READONLY) &&
@@ -3865,6 +3907,17 @@ static struct mount *mount_setattr_prepare(struct 
mount_kattr *kattr,
        return last;
 }
 
+static void do_idmap_mount(const struct mount_kattr *kattr, struct mount *mnt)
+{
+       struct user_namespace *user_ns;
+
+       if (!(kattr->attr_set & MNT_IDMAPPED))
+               return;
+
+       user_ns = get_user_ns(kattr->mnt_user_ns);
+       WRITE_ONCE(mnt->mnt.mnt_user_ns, user_ns);
+}
+
 static void mount_setattr_commit(struct mount_kattr *kattr, struct mount *mnt,
                                 struct mount *last, int err)
 {
@@ -3874,6 +3927,7 @@ static void mount_setattr_commit(struct mount_kattr 
*kattr, struct mount *mnt,
                if (!err) {
                        unsigned int flags;
 
+                       do_idmap_mount(kattr, m);
                        flags = recalc_flags(kattr, m);
                        WRITE_ONCE(m->mnt.mnt_flags, flags);
                }
@@ -3946,6 +4000,61 @@ static int do_mount_setattr(struct path *path, struct 
mount_kattr *kattr)
        return err;
 }
 
+static int build_mount_idmapped(const struct mount_attr *attr,
+                               struct mount_kattr *kattr, unsigned int flags)
+{
+       int err = 0;
+       struct ns_common *ns;
+       struct user_namespace *user_ns;
+       struct file *file;
+
+       if (!((attr->attr_set | attr->attr_clr) & MOUNT_ATTR_IDMAP))
+               return 0;
+
+       /*
+        * We currently do not support clearing an idmapped mount. If this ever
+        * is a use-case we can revisit this but for now let's keep it simple
+        * and not allow it.
+        */
+       if (attr->attr_clr & MOUNT_ATTR_IDMAP)
+               return -EINVAL;
+
+       if (attr->userns_fd > INT_MAX)
+               return -EINVAL;
+
+       file = fget(attr->userns_fd);
+       if (!file)
+               return -EBADF;
+
+       if (!proc_ns_file(file)) {
+               err = -EINVAL;
+               goto out_fput;
+       }
+
+       ns = get_proc_ns(file_inode(file));
+       if (ns->ops->type != CLONE_NEWUSER) {
+               err = -EINVAL;
+               goto out_fput;
+       }
+
+       /*
+        * The init_user_ns is used to indicate that a vfsmount is not idmapped.
+        * This is simpler than just having to treat NULL as unmapped. Users
+        * wanting to idmap a mount to init_user_ns can just use a namespace
+        * with an identity mapping.
+        */
+       user_ns = container_of(ns, struct user_namespace, ns);
+       if (user_ns == &init_user_ns) {
+               err = -EPERM;
+               goto out_fput;
+       }
+       kattr->mnt_user_ns = get_user_ns(user_ns);
+
+out_fput:
+       fput(file);
+       return err;
+}
+
 static int build_mount_kattr(const struct mount_attr *attr,
                             struct mount_kattr *kattr, unsigned int flags)
 {
@@ -4025,7 +4134,15 @@ static int build_mount_kattr(const struct mount_attr 
*attr,
                }
        }
 
-       return 0;
+       return build_mount_idmapped(attr, kattr, flags);
+}
+
+static void finish_mount_kattr(struct mount_kattr *kattr)
+{
+       if (kattr->attr_set & MNT_IDMAPPED) {
+               put_user_ns(kattr->mnt_user_ns);
+               kattr->mnt_user_ns = NULL;
+       }
 }
 
 SYSCALL_DEFINE5(mount_setattr, int, dfd, const char __user *, path, unsigned 
int, flags,
@@ -4064,6 +4181,7 @@ SYSCALL_DEFINE5(mount_setattr, int, dfd, const char 
__user *, path, unsigned int
                return err;
 
        err = do_mount_setattr(&target, &kattr);
+       finish_mount_kattr(&kattr);
        path_put(&target);
        return err;
 }
diff --git a/fs/proc_namespace.c b/fs/proc_namespace.c
index e59d4bb3a89e..5ebc337de8a7 100644
--- a/fs/proc_namespace.c
+++ b/fs/proc_namespace.c
@@ -71,6 +71,7 @@ static void show_mnt_opts(struct seq_file *m, struct vfsmount 
*mnt)
                { MNT_NODIRATIME, ",nodiratime" },
                { MNT_RELATIME, ",relatime" },
                { MNT_NOSYMFOLLOW, ",nosymfollow" },
+               { MNT_IDMAPPED, ",idmapped" },
                { 0, NULL }
        };
        const struct proc_fs_opts *fs_infop;
diff --git a/include/linux/fs.h b/include/linux/fs.h
index d83647b5a299..b89fc633ccc3 100644
--- a/include/linux/fs.h
+++ b/include/linux/fs.h
@@ -2277,7 +2277,7 @@ struct file_system_type {
 #define FS_HAS_SUBTYPE         4
 #define FS_USERNS_MOUNT                8       /* Can be mounted by userns 
root */
 #define FS_DISALLOW_NOTIFY_PERM        16      /* Disable fanotify permission 
events */
-#define FS_ALLOW_IDMAP         32      /* FS has been updated to handle vfs 
idmappings. */
+#define FS_ALLOW_IDMAP         32      /* FS has been updated to handle vfs 
idmappings. */
 #define FS_THP_SUPPORT         8192    /* Remove once all fs converted */
 #define FS_RENAME_DOES_D_MOVE  32768   /* FS will handle d_move() during 
rename() internally. */
        int (*init_fs_context)(struct fs_context *);
diff --git a/include/linux/mount.h b/include/linux/mount.h
index 3c7ba1bd4a21..d6745a0b90db 100644
--- a/include/linux/mount.h
+++ b/include/linux/mount.h
@@ -48,7 +48,7 @@ struct fs_context;
 #define MNT_SHARED_MASK        (MNT_UNBINDABLE)
 #define MNT_USER_SETTABLE_MASK  (MNT_NOSUID | MNT_NODEV | MNT_NOEXEC \
                                 | MNT_NOATIME | MNT_NODIRATIME | MNT_RELATIME \
-                                | MNT_READONLY | MNT_NOSYMFOLLOW)
+                                | MNT_READONLY | MNT_NOSYMFOLLOW | 
MNT_IDMAPPED)
 #define MNT_ATIME_MASK (MNT_NOATIME | MNT_NODIRATIME | MNT_RELATIME )
 
 #define MNT_INTERNAL_FLAGS (MNT_SHARED | MNT_WRITE_HOLD | MNT_INTERNAL | \
diff --git a/include/uapi/linux/mount.h b/include/uapi/linux/mount.h
index fb3ad26fdebf..525cc67363f8 100644
--- a/include/uapi/linux/mount.h
+++ b/include/uapi/linux/mount.h
@@ -117,6 +117,7 @@ enum fsconfig_command {
 #define MOUNT_ATTR_NOATIME     0x00000010 /* - Do not update access times. */
 #define MOUNT_ATTR_STRICTATIME 0x00000020 /* - Always perform atime updates */
 #define MOUNT_ATTR_NODIRATIME  0x00000080 /* Do not update directory access 
times */
+#define MOUNT_ATTR_IDMAP       0x00100000 /* Idmap this mount to @userns in 
mount_attr. */
 
 /*
  * mount_setattr()
@@ -125,6 +126,7 @@ struct mount_attr {
        __u64 attr_set;
        __u64 attr_clr;
        __u64 propagation;
+       __u64 userns_fd;
 };
 
 /* Change propagation through mount_setattr(). */
@@ -138,6 +140,7 @@ enum propagation_type {
 
 /* List of all mount_attr versions. */
 #define MOUNT_ATTR_SIZE_VER0   24 /* sizeof first published struct */
-#define MOUNT_ATTR_SIZE_LATEST MOUNT_ATTR_SIZE_VER0
+#define MOUNT_ATTR_SIZE_VER1   32 /* sizeof second published struct */
+#define MOUNT_ATTR_SIZE_LATEST MOUNT_ATTR_SIZE_VER1
 
 #endif /* _UAPI_LINUX_MOUNT_H */
-- 
2.29.2

--
Linux-audit mailing list
Linux-audit@redhat.com
https://www.redhat.com/mailman/listinfo/linux-audit

Reply via email to