diff options
| author |   Eric W. Biederman <ebiederm@xmission.com> | 2019-05-21 10:03:48 -0500 |
|---|---|---|
| committer | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-27 09:36:28 -0500 |
| commit | cb44c9a0ab21a9ae4dfcabac1ed8e38aa872d1af (patch) | |
| tree | e09131e28e3f55143862fc2377c30049a6a4b0bc /kernel/signal.c | |
| parent | signal/cifs: Fix cifs_put_tcp_session to call send_sig instead of force_sig (diff) | |
| download | wireguard-linux-cb44c9a0ab21a9ae4dfcabac1ed8e38aa872d1af.tar.xz wireguard-linux-cb44c9a0ab21a9ae4dfcabac1ed8e38aa872d1af.zip | |
signal: Remove task parameter from force_sigsegv
The function force_sigsegv is always called on the current task
so passing in current is redundant and not passing in current
makes this fact obvious.
This also makes it clear force_sigsegv always calls force_sig
on the current task.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'kernel/signal.c')
| -rw-r--r-- | kernel/signal.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/kernel/signal.c b/kernel/signal.c index 39a3eca5ce22..f7669d240ce4 100644 --- a/kernel/signal.c +++ b/kernel/signal.c @@ -1607,8 +1607,10 @@ EXPORT_SYMBOL(force_sig); * the problem was already a SIGSEGV, we'll want to * make sure we don't even try to deliver the signal.. */ -void force_sigsegv(int sig, struct task_struct *p) +void force_sigsegv(int sig) { + struct task_struct *p = current; + if (sig == SIGSEGV) { unsigned long flags; spin_lock_irqsave(&p->sighand->siglock, flags); @@ -2717,7 +2719,7 @@ static void signal_delivered(struct ksignal *ksig, int stepping) void signal_setup_done(int failed, struct ksignal *ksig, int stepping) { if (failed) - force_sigsegv(ksig->sig, current); + force_sigsegv(ksig->sig); else signal_delivered(ksig, stepping); } |