diff options
author | Jann Horn <jannh@google.com> | 2022-03-19 02:08:37 +0100 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2022-03-22 19:06:05 +0100 |
commit | ee1fee900537b5d9560e9f937402de5ddc8412f3 (patch) | |
tree | 7a1faba78812c3d8e884671d9e28cdd7540c1142 /kernel/exit.c | |
parent | ptrace: Return the signal to continue with from ptrace_stop (diff) | |
download | linux-ee1fee900537b5d9560e9f937402de5ddc8412f3.tar.xz linux-ee1fee900537b5d9560e9f937402de5ddc8412f3.zip |
ptrace: Check PTRACE_O_SUSPEND_SECCOMP permission on PTRACE_SEIZE
Setting PTRACE_O_SUSPEND_SECCOMP is supposed to be a highly privileged
operation because it allows the tracee to completely bypass all seccomp
filters on kernels with CONFIG_CHECKPOINT_RESTORE=y. It is only supposed to
be settable by a process with global CAP_SYS_ADMIN, and only if that
process is not subject to any seccomp filters at all.
However, while these permission checks were done on the PTRACE_SETOPTIONS
path, they were missing on the PTRACE_SEIZE path, which also sets
user-specified ptrace flags.
Move the permissions checks out into a helper function and let both
ptrace_attach() and ptrace_setoptions() call it.
Cc: stable@kernel.org
Fixes: 13c4a90119d2 ("seccomp: add ptrace options for suspend/resume")
Signed-off-by: Jann Horn <jannh@google.com>
Link: https://lkml.kernel.org/r/20220319010838.1386861-1-jannh@google.com
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Diffstat (limited to 'kernel/exit.c')
0 files changed, 0 insertions, 0 deletions