signal: Simplify tracehook_report_syscall_exit
authorEric W. Biederman <ebiederm@xmission.com>
Mon, 16 Apr 2018 19:18:26 +0000 (14:18 -0500)
committerEric W. Biederman <ebiederm@xmission.com>
Wed, 19 Sep 2018 13:45:42 +0000 (15:45 +0200)
commitefc463adbccf709c5dbaf6cfbf84b7e94b62810a
treeeee1ff080fa28324033a7400a0ad64c36ceea517
parentfb50f5a4011c499bc1b1fae77299cfcb3945e51b
signal: Simplify tracehook_report_syscall_exit

Replace user_single_step_siginfo with user_single_step_report
that allocates siginfo structure on the stack and sends it.

This allows tracehook_report_syscall_exit to become a simple
if statement that calls user_single_step_report or ptrace_report_syscall
depending on the value of step.

Update the default helper function now called user_single_step_report
to explicitly set si_code to SI_USER and to set si_uid and si_pid to 0.
The default helper has always been doing this (using memset) but it
was far from obvious.

The powerpc helper can now just call force_sig_fault.
The x86 helper can now just call send_sigtrap.

Unfortunately the default implementation of user_single_step_report
can not use force_sig_fault as it does not use a SIGTRAP si_code.
So it has to carefully setup the siginfo and use use force_sig_info.

The net result is code that is easier to understand and simpler
to maintain.

Ref: 85ec7fd9f8e5 ("ptrace: introduce user_single_step_siginfo() helper")
Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
arch/powerpc/include/asm/ptrace.h
arch/powerpc/kernel/traps.c
arch/x86/include/asm/ptrace.h
arch/x86/kernel/ptrace.c
include/linux/ptrace.h
include/linux/tracehook.h