| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <20211209221545.2333249-1-pcc@google.com>
Date: Thu, 9 Dec 2021 14:15:37 -0800
From: Peter Collingbourne <pcc@...gle.com>
To: Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will@...nel.org>, Ingo Molnar <mingo@...hat.com>,
Peter Zijlstra <peterz@...radead.org>,
Juri Lelli <juri.lelli@...hat.com>,
Vincent Guittot <vincent.guittot@...aro.org>,
Dietmar Eggemann <dietmar.eggemann@....com>,
Steven Rostedt <rostedt@...dmis.org>,
Ben Segall <bsegall@...gle.com>, Mel Gorman <mgorman@...e.de>,
Daniel Bristot de Oliveira <bristot@...hat.com>,
Thomas Gleixner <tglx@...utronix.de>,
Andy Lutomirski <luto@...nel.org>,
Kees Cook <keescook@...omium.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Masahiro Yamada <masahiroy@...nel.org>,
Sami Tolvanen <samitolvanen@...gle.com>,
YiFei Zhu <yifeifz2@...inois.edu>,
Mark Rutland <mark.rutland@....com>,
Frederic Weisbecker <frederic@...nel.org>,
Viresh Kumar <viresh.kumar@...aro.org>,
Andrey Konovalov <andreyknvl@...il.com>,
Peter Collingbourne <pcc@...gle.com>,
Gabriel Krisman Bertazi <krisman@...labora.com>,
Chris Hyser <chris.hyser@...cle.com>,
Daniel Vetter <daniel.vetter@...ll.ch>,
Chris Wilson <chris@...is-wilson.co.uk>,
Arnd Bergmann <arnd@...db.de>,
Dmitry Vyukov <dvyukov@...gle.com>,
Christian Brauner <christian.brauner@...ntu.com>,
"Eric W. Biederman" <ebiederm@...ssion.com>,
Alexey Gladkov <legion@...nel.org>,
Ran Xiaokai <ran.xiaokai@....com.cn>,
David Hildenbrand <david@...hat.com>,
Xiaofeng Cao <caoxiaofeng@...ong.com>,
Cyrill Gorcunov <gorcunov@...il.com>,
Thomas Cedeno <thomascedeno@...gle.com>,
Marco Elver <elver@...gle.com>,
Alexander Potapenko <glider@...gle.com>
Cc: linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
Evgenii Stepanov <eugenis@...gle.com>
Subject: [PATCH v4 0/7] kernel: introduce uaccess logging
This patch series introduces a kernel feature known as uaccess
logging, which allows userspace programs to be made aware of the
address and size of uaccesses performed by the kernel during
the servicing of a syscall. More details on the motivation
for and interface to this feature are available in the file
Documentation/admin-guide/uaccess-logging.rst added by the final
patch in the series.
Because we don't have a common kernel entry/exit code path that is used
on all architectures, uaccess logging is only implemented for arm64
and architectures that use CONFIG_GENERIC_ENTRY, i.e. x86 and s390.
The proposed interface is the result of numerous iterations and
prototyping and is based on a proposal by Dmitry Vyukov. The interface
preserves the correspondence between uaccess log identity and syscall
identity while tolerating incoming asynchronous signals in the interval
between setting up the logging and the actual syscall. We considered
a number of alternative designs but rejected them for various reasons:
- The design from v1 of this patch [1] proposed notifying the kernel
of the address and size of the uaccess buffer via a prctl that
would also automatically mask and unmask asynchronous signals as
needed, but this would require multiple syscalls per "real" syscall,
harming performance.
- We considered extending the syscall calling convention to
designate currently-unused registers to be used to pass the
location of the uaccess buffer, but this was rejected for being
architecture-specific.
- One idea that we considered involved using the stack pointer address
as a unique identifier for the syscall, but this currently would
need to be arch-specific as we currently do not appear to have an
arch-generic way of retrieving the stack pointer; the userspace
side would also need some arch-specific code for this to work. It's
also possible that a longjmp() past the signal handler would make
the stack pointer address not unique enough for this purpose.
We also evaluated implementing this on top of the existing tracepoint
facility, but concluded that it is not suitable for this purpose:
- Tracepoints have a per-task granularity at best, whereas we really want
to trace per-syscall. This is so that we can exclude syscalls that
should not be traced, such as syscalls that make up part of the
sanitizer implementation (to avoid infinite recursion when e.g. printing
an error report).
- Tracing would need to be synchronous in order to produce useful
stack traces. For example this could be achieved using the new SIGTRAP
on perf events mechanism. However, this would require logging each
access to the stack (in the form of a sigcontext) and this is more
likely to overflow the stack due to being much larger than a uaccess
buffer entry as well as being unbounded, in contrast to the bounded
buffer size passed to prctl(). An approach based on signal handlers is
also likely to fall foul of the asynchronous signal issues mentioned
previously, together with needing sigreturn to be handled specially
(because it copies a sigcontext from userspace) otherwise we could
never return from the signal handler. Furthermore, arguments to the
trace events are not available to SIGTRAP. (This on its own wouldn't
be insurmountable though -- we could add the arguments as fields
to siginfo.)
- The API in https://www.kernel.org/doc/Documentation/trace/ftrace.txt
-- e.g. trace_pipe_raw gives access to the internal ring buffer, but
I don't think it's usable because it's per-CPU and not per-task.
- Tracepoints can be used by eBPF programs, but eBPF programs may
only be loaded as root, among other potential headaches.
[1] https://lore.kernel.org/all/20210922061809.736124-1-pcc@google.com/
Peter Collingbourne (7):
include: split out uaccess instrumentation into a separate header
uaccess-buffer: add core code
fs: use copy_from_user_nolog() to copy mount() data
uaccess-buffer: add CONFIG_GENERIC_ENTRY support
arm64: add support for uaccess logging
Documentation: document uaccess logging
selftests: test uaccess logging
Documentation/admin-guide/index.rst | 1 +
Documentation/admin-guide/uaccess-logging.rst | 151 +++++++++++++++++
arch/Kconfig | 14 ++
arch/arm64/Kconfig | 1 +
arch/arm64/include/asm/thread_info.h | 7 +-
arch/arm64/kernel/ptrace.c | 7 +
arch/arm64/kernel/signal.c | 5 +
fs/exec.c | 3 +
fs/namespace.c | 8 +-
include/linux/entry-common.h | 2 +
include/linux/instrumented-uaccess.h | 53 ++++++
include/linux/instrumented.h | 34 ----
include/linux/sched.h | 5 +
include/linux/thread_info.h | 4 +
include/linux/uaccess-buffer-info.h | 46 ++++++
include/linux/uaccess-buffer.h | 152 ++++++++++++++++++
include/linux/uaccess.h | 2 +-
include/uapi/linux/prctl.h | 3 +
include/uapi/linux/uaccess-buffer.h | 27 ++++
kernel/Makefile | 1 +
kernel/bpf/helpers.c | 7 +-
kernel/entry/common.c | 14 +-
kernel/fork.c | 4 +
kernel/signal.c | 9 +-
kernel/sys.c | 6 +
kernel/uaccess-buffer.c | 145 +++++++++++++++++
lib/iov_iter.c | 2 +-
lib/usercopy.c | 2 +-
tools/testing/selftests/Makefile | 1 +
.../testing/selftests/uaccess_buffer/Makefile | 4 +
.../uaccess_buffer/uaccess_buffer_test.c | 126 +++++++++++++++
31 files changed, 802 insertions(+), 44 deletions(-)
create mode 100644 Documentation/admin-guide/uaccess-logging.rst
create mode 100644 include/linux/instrumented-uaccess.h
create mode 100644 include/linux/uaccess-buffer-info.h
create mode 100644 include/linux/uaccess-buffer.h
create mode 100644 include/uapi/linux/uaccess-buffer.h
create mode 100644 kernel/uaccess-buffer.c
create mode 100644 tools/testing/selftests/uaccess_buffer/Makefile
create mode 100644 tools/testing/selftests/uaccess_buffer/uaccess_buffer_test.c
--
2.34.1.173.g76aa8bc2d0-goog
Powered by blists - more mailing lists