|
|
Subscribe / Log in / New account

Add seccomp notifier ioctl that enables adding fds

From:  Kees Cook <keescook-AT-chromium.org>
To:  linux-kernel-AT-vger.kernel.org
Subject:  [PATCH v5 0/7] Add seccomp notifier ioctl that enables adding fds
Date:  Wed, 17 Jun 2020 15:03:20 -0700
Message-ID:  <20200617220327.3731559-1-keescook@chromium.org>
Cc:  Kees Cook <keescook-AT-chromium.org>, Sargun Dhillon <sargun-AT-sargun.me>, Christian Brauner <christian-AT-brauner.io>, Tycho Andersen <tycho-AT-tycho.ws>, David Laight <David.Laight-AT-ACULAB.COM>, Christoph Hellwig <hch-AT-lst.de>, "David S. Miller" <davem-AT-davemloft.net>, Jakub Kicinski <kuba-AT-kernel.org>, Alexander Viro <viro-AT-zeniv.linux.org.uk>, Aleksa Sarai <cyphar-AT-cyphar.com>, Matt Denton <mpdenton-AT-google.com>, Jann Horn <jannh-AT-google.com>, Chris Palmer <palmer-AT-google.com>, Robert Sesek <rsesek-AT-google.com>, Giuseppe Scrivano <gscrivan-AT-redhat.com>, Greg Kroah-Hartman <gregkh-AT-linuxfoundation.org>, Andy Lutomirski <luto-AT-amacapital.net>, Will Drewry <wad-AT-chromium.org>, Shuah Khan <shuah-AT-kernel.org>, netdev-AT-vger.kernel.org, containers-AT-lists.linux-foundation.org, linux-api-AT-vger.kernel.org, linux-fsdevel-AT-vger.kernel.org, linux-kselftest-AT-vger.kernel.org
Archive-link:  Article

Hello!

v5:
- merge ioctl fixes into Sargun's patches directly
- adjust new API to avoid "ufd_required" argument
- drop general clean up patches now present in for-next/seccomp
v4: https://lore.kernel.org/lkml/20200616032524.460144-1-kees...

This continues the thread-merge between [1] and [2]. tl;dr: add a way for
a seccomp user_notif process manager to inject files into the managed
process in order to handle emulation of various fd-returning syscalls
across security boundaries. Containers folks and Chrome are in need
of the feature, and investigating this solution uncovered (and fixed)
implementation issues with existing file sending routines.

I intend to carry this in the seccomp tree, unless someone has objections.
:) Please review and test!

-Kees

[1] https://lore.kernel.org/lkml/20200603011044.7972-1-sargun...
[2] https://lore.kernel.org/lkml/20200610045214.1175600-1-kee...


Kees Cook (5):
  net/scm: Regularize compat handling of scm_detach_fds()
  fs: Move __scm_install_fd() to __fd_install_received()
  fs: Add fd_install_received() wrapper for __fd_install_received()
  pidfd: Replace open-coded partial fd_install_received()
  fs: Expand __fd_install_received() to accept fd

Sargun Dhillon (2):
  seccomp: Introduce addfd ioctl to seccomp user notifier
  selftests/seccomp: Test SECCOMP_IOCTL_NOTIF_ADDFD

 fs/file.c                                     |  63 +++++
 include/linux/file.h                          |  19 ++
 include/uapi/linux/seccomp.h                  |  22 ++
 kernel/pid.c                                  |  11 +-
 kernel/seccomp.c                              | 172 ++++++++++++-
 net/compat.c                                  |  55 ++---
 net/core/scm.c                                |  50 +---
 tools/testing/selftests/seccomp/seccomp_bpf.c | 229 ++++++++++++++++++
 8 files changed, 540 insertions(+), 81 deletions(-)

-- 
2.25.1



Copyright © 2020, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds