diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-23 12:04:24 -0400 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-29 10:31:43 -0400 |
commit | 2e1661d2673667d886cd40ad9f414cb6db48d8da (patch) | |
tree | 5a2c32eb8a5c575bfb5c7013f5d9d427f5c06c34 /arch/x86/mm/fault.c | |
parent | 91ca180dbdd687d45fe4aab055b02d29c91b90df (diff) |
signal: Remove the task parameter from force_sig_fault
As synchronous exceptions really only make sense against the current
task (otherwise how are you synchronous) remove the task parameter
from from force_sig_fault to make it explicit that is what is going
on.
The two known exceptions that deliver a synchronous exception to a
stopped ptraced task have already been changed to
force_sig_fault_to_task.
The callers have been changed with the following emacs regular expression
(with obvious variations on the architectures that take more arguments)
to avoid typos:
force_sig_fault[(]\([^,]+\)[,]\([^,]+\)[,]\([^,]+\)[,]\W+current[)]
->
force_sig_fault(\1,\2,\3)
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/x86/mm/fault.c')
-rw-r--r-- | arch/x86/mm/fault.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c index 16a5d1b615a7..46ac96aa7c81 100644 --- a/arch/x86/mm/fault.c +++ b/arch/x86/mm/fault.c | |||
@@ -756,8 +756,7 @@ no_context(struct pt_regs *regs, unsigned long error_code, | |||
756 | set_signal_archinfo(address, error_code); | 756 | set_signal_archinfo(address, error_code); |
757 | 757 | ||
758 | /* XXX: hwpoison faults will set the wrong code. */ | 758 | /* XXX: hwpoison faults will set the wrong code. */ |
759 | force_sig_fault(signal, si_code, (void __user *)address, | 759 | force_sig_fault(signal, si_code, (void __user *)address); |
760 | current); | ||
761 | } | 760 | } |
762 | 761 | ||
763 | /* | 762 | /* |
@@ -918,7 +917,7 @@ __bad_area_nosemaphore(struct pt_regs *regs, unsigned long error_code, | |||
918 | if (si_code == SEGV_PKUERR) | 917 | if (si_code == SEGV_PKUERR) |
919 | force_sig_pkuerr((void __user *)address, pkey); | 918 | force_sig_pkuerr((void __user *)address, pkey); |
920 | 919 | ||
921 | force_sig_fault(SIGSEGV, si_code, (void __user *)address, current); | 920 | force_sig_fault(SIGSEGV, si_code, (void __user *)address); |
922 | 921 | ||
923 | return; | 922 | return; |
924 | } | 923 | } |
@@ -1044,7 +1043,7 @@ do_sigbus(struct pt_regs *regs, unsigned long error_code, unsigned long address, | |||
1044 | return; | 1043 | return; |
1045 | } | 1044 | } |
1046 | #endif | 1045 | #endif |
1047 | force_sig_fault(SIGBUS, BUS_ADRERR, (void __user *)address, current); | 1046 | force_sig_fault(SIGBUS, BUS_ADRERR, (void __user *)address); |
1048 | } | 1047 | } |
1049 | 1048 | ||
1050 | static noinline void | 1049 | static noinline void |