diff options
| author |   Eric W. Biederman <ebiederm@xmission.com> | 2018-01-22 14:37:25 -0600 |
|---|---|---|
| committer | Eric W. Biederman <ebiederm@xmission.com> | 2018-01-22 19:07:11 -0600 |
| commit | f71dd7dc2dc989dc712b246a74d243e4b2c5f8a7 (patch) | |
| tree | c126cf571c0a25c540210fa7bfe8f03b19d0c2a4 /include/linux/sched | |
| parent | signal/powerpc: Remove unnecessary signal_code parameter of do_send_trap (diff) | |
| download | linux-dev-f71dd7dc2dc989dc712b246a74d243e4b2c5f8a7.tar.xz linux-dev-f71dd7dc2dc989dc712b246a74d243e4b2c5f8a7.zip | |
signal/ptrace: Add force_sig_ptrace_errno_trap and use it where needed
There are so many places that build struct siginfo by hand that at
least one of them is bound to get it wrong. A handful of cases in the
kernel arguably did just that when using the errno field of siginfo to
pass no errno values to userspace. The usage is limited to a single
si_code so at least does not mess up anything else.
Encapsulate this questionable pattern in a helper function so
that the userspace ABI is preserved.
Update all of the places that use this pattern to use the new helper
function.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'include/linux/sched')
| -rw-r--r-- | include/linux/sched/signal.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/linux/sched/signal.h b/include/linux/sched/signal.h index 944fe6356f4a..23b4f9cb82db 100644 --- a/include/linux/sched/signal.h +++ b/include/linux/sched/signal.h @@ -311,6 +311,8 @@ int send_sig_mceerr(int code, void __user *, short, struct task_struct *); int force_sig_bnderr(void __user *addr, void __user *lower, void __user *upper); int force_sig_pkuerr(void __user *addr, u32 pkey); +int force_sig_ptrace_errno_trap(int errno, void __user *addr); + extern int send_sig_info(int, struct siginfo *, struct task_struct *); extern int force_sigsegv(int, struct task_struct *); extern int force_sig_info(int, struct siginfo *, struct task_struct *); |