diff options
author | Borislav Petkov <bp@suse.de> | 2018-10-30 18:07:13 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2018-10-31 11:54:14 -0400 |
commit | 95c4fb78fb23081472465ca20d5d31c4b780ed82 (patch) | |
tree | 1e61e00bfb8837c2ae96c0cb94197f19a672beab | |
parent | 3819ddec1f8c8a5daf215bec4067f8fd1dc40d6d (diff) |
kernel/panic.c: do not append newline to the stack protector panic string
... because panic() itself already does this. Otherwise you have
line-broken trailer:
[ 1.836965] ---[ end Kernel panic - not syncing: stack-protector: Kernel stack is corrupted in: pgd_alloc+0x29e/0x2a0
[ 1.836965] ]---
Link: http://lkml.kernel.org/r/20181008202901.7894-1-bp@alien8.de
Signed-off-by: Borislav Petkov <bp@suse.de>
Acked-by: Kees Cook <keescook@chromium.org>
Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
Cc: "Steven Rostedt (VMware)" <rostedt@goodmis.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | kernel/panic.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/panic.c b/kernel/panic.c index 8b2e002d52eb..837a94b7024d 100644 --- a/kernel/panic.c +++ b/kernel/panic.c | |||
@@ -631,7 +631,7 @@ device_initcall(register_warn_debugfs); | |||
631 | */ | 631 | */ |
632 | __visible void __stack_chk_fail(void) | 632 | __visible void __stack_chk_fail(void) |
633 | { | 633 | { |
634 | panic("stack-protector: Kernel stack is corrupted in: %pB\n", | 634 | panic("stack-protector: Kernel stack is corrupted in: %pB", |
635 | __builtin_return_address(0)); | 635 | __builtin_return_address(0)); |
636 | } | 636 | } |
637 | EXPORT_SYMBOL(__stack_chk_fail); | 637 | EXPORT_SYMBOL(__stack_chk_fail); |