[REVIEW][PATCH 03/26] signal/arm64: Use force_sig not force_sig_fault for SIGKILL

From: Eric W. Biederman
Date: Wed May 22 2019 - 20:43:26 EST


It really only matters to debuggers but the SIGKILL does not have any
si_codes that use the fault member of the siginfo union. Correct this
the simple way and call force_sig instead of force_sig_fault when the
signal is SIGKILL.

Cc: stable@xxxxxxxxxxxxxxx
Cc: Dave Martin <Dave.Martin@xxxxxxx>
Cc: James Morse <james.morse@xxxxxxx>
Cc: Will Deacon <will.deacon@xxxxxxx>
Fixes: af40ff687bc9 ("arm64: signal: Ensure si_code is valid for all fault signals")
Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
---
arch/arm64/kernel/traps.c | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/arch/arm64/kernel/traps.c b/arch/arm64/kernel/traps.c
index ade32046f3fe..0feb17bdcaa0 100644
--- a/arch/arm64/kernel/traps.c
+++ b/arch/arm64/kernel/traps.c
@@ -282,6 +282,11 @@ void arm64_notify_die(const char *str, struct pt_regs *regs,
current->thread.fault_address = 0;
current->thread.fault_code = err;

+ if (signo == SIGKILL) {
+ arm64_show_signal(signo, str);
+ force_sig(signo, current);
+ return;
+ }
arm64_force_sig_fault(signo, sicode, addr, str);
} else {
die(str, regs, err);
--
2.21.0