diff options
| author | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-15 11:11:09 -0400 |
|---|---|---|
| committer | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-29 10:31:44 -0400 |
| commit | a89e9b8abf82725e4ac96100e07c8104dbe8a240 (patch) | |
| tree | a25fe1216ab1148b3991afc738c237716e1440c3 /include | |
| parent | 59c0e696a6c0fe6a8d718a43aecd72347db6a7f0 (diff) | |
signal: Remove the signal number and task parameters from force_sig_info
force_sig_info always delivers to the current task and the signal
parameter always matches info.si_signo. So remove those parameters to
make it a simpler less error prone interface, and to make it clear
that none of the callers are doing anything clever.
This guarantees that force_sig_info will not grow any new buggy
callers that attempt to call force_sig on a non-current task, or that
pass an signal number that does not match info.si_signo.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/ptrace.h | 2 | ||||
| -rw-r--r-- | include/linux/sched/signal.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/ptrace.h b/include/linux/ptrace.h index d5084ebd9f03..2a9df80ea887 100644 --- a/include/linux/ptrace.h +++ b/include/linux/ptrace.h | |||
| @@ -355,7 +355,7 @@ static inline void user_single_step_report(struct pt_regs *regs) | |||
| 355 | info.si_code = SI_USER; | 355 | info.si_code = SI_USER; |
| 356 | info.si_pid = 0; | 356 | info.si_pid = 0; |
| 357 | info.si_uid = 0; | 357 | info.si_uid = 0; |
| 358 | force_sig_info(info.si_signo, &info, current); | 358 | force_sig_info(&info); |
| 359 | } | 359 | } |
| 360 | #endif | 360 | #endif |
| 361 | 361 | ||
diff --git a/include/linux/sched/signal.h b/include/linux/sched/signal.h index 7f872506e1de..532458698bde 100644 --- a/include/linux/sched/signal.h +++ b/include/linux/sched/signal.h | |||
| @@ -329,7 +329,7 @@ int force_sig_ptrace_errno_trap(int errno, void __user *addr); | |||
| 329 | 329 | ||
| 330 | extern int send_sig_info(int, struct kernel_siginfo *, struct task_struct *); | 330 | extern int send_sig_info(int, struct kernel_siginfo *, struct task_struct *); |
| 331 | extern void force_sigsegv(int sig); | 331 | extern void force_sigsegv(int sig); |
| 332 | extern int force_sig_info(int, struct kernel_siginfo *, struct task_struct *); | 332 | extern int force_sig_info(struct kernel_siginfo *); |
| 333 | extern int __kill_pgrp_info(int sig, struct kernel_siginfo *info, struct pid *pgrp); | 333 | extern int __kill_pgrp_info(int sig, struct kernel_siginfo *info, struct pid *pgrp); |
| 334 | extern int kill_pid_info(int sig, struct kernel_siginfo *info, struct pid *pid); | 334 | extern int kill_pid_info(int sig, struct kernel_siginfo *info, struct pid *pid); |
| 335 | extern int kill_pid_usb_asyncio(int sig, int errno, sigval_t addr, struct pid *, | 335 | extern int kill_pid_usb_asyncio(int sig, int errno, sigval_t addr, struct pid *, |
