diff options
author | Alan Donovan <adonovan@google.com> | 2024-04-23 12:44:54 -0400 |
---|---|---|
committer | Gopher Robot <gobot@golang.org> | 2024-05-08 19:10:41 +0000 |
commit | 69e75c8581e15328454bb6e2f1dc347f73616b37 (patch) | |
tree | f0fabe495e978fb211ef909e4c4b8d8a4085d559 /doc | |
parent | 4513f1a1c1037574bfd94b8637329246f475a534 (diff) | |
download | go-69e75c8581e15328454bb6e2f1dc347f73616b37.tar.gz go-69e75c8581e15328454bb6e2f1dc347f73616b37.zip |
runtime: properly frame panic values in tracebacks
This CL causes the printing of panic values to ensure that all
newlines in the output are immediately followed by a tab, so
that there is no way for a maliciously crafted panic value to
fool a program attempting to parse the traceback into thinking
that the panic value is in fact a goroutine stack.
See https://github.com/golang/go/issues/64590#issuecomment-1932675696
+ release note
Updates #64590
Updates #63455
Change-Id: I5142acb777383c0c122779d984e73879567dc627
Reviewed-on: https://go-review.googlesource.com/c/go/+/581215
Auto-Submit: Alan Donovan <adonovan@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Diffstat (limited to 'doc')
-rw-r--r-- | doc/next/4-runtime.md | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/doc/next/4-runtime.md b/doc/next/4-runtime.md index 1f8e445e0b..7553154a16 100644 --- a/doc/next/4-runtime.md +++ b/doc/next/4-runtime.md @@ -1 +1,7 @@ ## Runtime {#runtime} + +The traceback printed by the runtime after an unhandled panic or other +fatal error now indents the second and subsequent lines of the error +message (for example, the argument to panic) by a single tab, so that +it can be unambiguously distinguished from the stack trace of the +first goroutine. See [#64590](/issue/64590) for discussion. |