OSDN Git Service

sh: fix syscall tracing
authorRich Felker <dalias@libc.org>
Sat, 29 Aug 2020 01:01:41 +0000 (21:01 -0400)
committerRich Felker <dalias@libc.org>
Mon, 14 Sep 2020 01:22:55 +0000 (21:22 -0400)
commitb0cfc315ff38c423a5ce9ce159bd5baa4135e688
treed644d9e035223476718925fee68c57f1d13719a9
parentca6345de57a46ba1bd35bd15b0ceb42e05b3d71f
sh: fix syscall tracing

Addition of SECCOMP_FILTER exposed a longstanding bug in
do_syscall_trace_enter, whereby r0 (the 5th argument register) was
mistakenly used where r3 (syscall_nr) was intended. By overwriting r0
rather than r3 with -1 when attempting to block a syscall, the
existing code would instead have caused the syscall to execute with an
argument clobbered.

Commit 0bb605c2c7f2b4b3 then introduced skipping of the syscall when
do_syscall_trace_enter returns -1, so that the return value set by
seccomp filters would not be clobbered by -ENOSYS. This eliminated the
clobbering of the 5th argument register, but instead caused syscalls
made with a 5th argument of -1 to be misinterpreted as a request by
do_syscall_trace_enter to suppress the syscall.

Fixes: 0bb605c2c7f2b4b3 ("sh: Add SECCOMP_FILTER")
Fixes: ab99c733ae73cce3 ("sh: Make syscall tracer use tracehook notifiers, add TIF_NOTIFY_RESUME.")
Tested-by: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
Signed-off-by: Rich Felker <dalias@libc.org>
arch/sh/kernel/entry-common.S
arch/sh/kernel/ptrace_32.c