kernel/panic.c: do not append newline to the stack protector panic string
authorBorislav Petkov <[email protected]>
Tue, 30 Oct 2018 22:07:13 +0000 (15:07 -0700)
committerLinus Torvalds <[email protected]>
Wed, 31 Oct 2018 15:54:14 +0000 (08:54 -0700)
... 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/[email protected]
Signed-off-by: Borislav Petkov <[email protected]>
Acked-by: Kees Cook <[email protected]>
Cc: Masahiro Yamada <[email protected]>
Cc: "Steven Rostedt (VMware)" <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
kernel/panic.c

index 8b2e002d52eb0e6c927876077bddfcd0565d7929..837a94b7024d9a95ea3eeaaff78c1df6c80d9dd1 100644 (file)
@@ -631,7 +631,7 @@ device_initcall(register_warn_debugfs);
  */
 __visible void __stack_chk_fail(void)
 {
-       panic("stack-protector: Kernel stack is corrupted in: %pB\n",
+       panic("stack-protector: Kernel stack is corrupted in: %pB",
                __builtin_return_address(0));
 }
 EXPORT_SYMBOL(__stack_chk_fail);