signal/x86: Remove task parameter from send_sigtrap
authorEric W. Biederman <ebiederm@xmission.com>
Wed, 6 Feb 2019 00:25:11 +0000 (18:25 -0600)
committerEric W. Biederman <ebiederm@xmission.com>
Wed, 29 May 2019 14:30:48 +0000 (09:30 -0500)
The send_sigtrap function is always called with task == current.  Make
that explicit by removing the task parameter.

This also makes it clear that the x86 send_sigtrap passes current
into force_sig_fault.

Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
arch/x86/include/asm/ptrace.h
arch/x86/kernel/ptrace.c
arch/x86/kernel/traps.c

index 8a7fc0c..28779bf 100644 (file)
@@ -102,8 +102,7 @@ extern unsigned long profile_pc(struct pt_regs *regs);
 
 extern unsigned long
 convert_ip_to_linear(struct task_struct *child, struct pt_regs *regs);
-extern void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs,
-                        int error_code, int si_code);
+extern void send_sigtrap(struct pt_regs *regs, int error_code, int si_code);
 
 
 static inline unsigned long regs_return_value(struct pt_regs *regs)
index 4b8ee05..0014814 100644 (file)
@@ -1360,9 +1360,10 @@ const struct user_regset_view *task_user_regset_view(struct task_struct *task)
 #endif
 }
 
-void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs,
-                                        int error_code, int si_code)
+void send_sigtrap(struct pt_regs *regs, int error_code, int si_code)
 {
+       struct task_struct *tsk = current;
+
        tsk->thread.trap_nr = X86_TRAP_DB;
        tsk->thread.error_code = error_code;
 
@@ -1373,5 +1374,5 @@ void send_sigtrap(struct task_struct *tsk, struct pt_regs *regs,
 
 void user_single_step_report(struct pt_regs *regs)
 {
-       send_sigtrap(current, regs, 0, TRAP_BRKPT);
+       send_sigtrap(regs, 0, TRAP_BRKPT);
 }
index e54f0ca..30a9b84 100644 (file)
@@ -805,7 +805,7 @@ dotraplinkage void do_debug(struct pt_regs *regs, long error_code)
        }
        si_code = get_si_code(tsk->thread.debugreg6);
        if (tsk->thread.debugreg6 & (DR_STEP | DR_TRAP_BITS) || user_icebp)
-               send_sigtrap(tsk, regs, error_code, si_code);
+               send_sigtrap(regs, error_code, si_code);
        cond_local_irq_disable(regs);
        debug_stack_usage_dec();