diff options
author | Michael Pratt <mpratt@google.com> | 2021-04-09 17:01:45 -0400 |
---|---|---|
committer | Michael Pratt <mpratt@google.com> | 2021-04-12 21:29:02 +0000 |
commit | aad13cbb749d1e6c085ff0556d306de1a2d5d063 (patch) | |
tree | abe14f4ed12d2a82166e894564a692711c39dff3 /src/runtime/symtab.go | |
parent | 5c9b6e8e63e012513b1cb1a4a08ff23dec4137a1 (diff) | |
download | go-aad13cbb749d1e6c085ff0556d306de1a2d5d063.tar.gz go-aad13cbb749d1e6c085ff0556d306de1a2d5d063.zip |
runtime: non-strict InlTreeIndex lookup in expandFinalInlineFrame
This is a follow-up to golang.org/cl/301369, which made the same change
in Frames.Next. The same logic applies here: a profile stack may have
been truncated at an invalid PC provided by cgoTraceback.
expandFinalInlineFrame will then try to lookup the inline tree and
crash.
The same fix applies as well: upon encountering a bad PC, simply leave
it as-is and move on.
Fixes #44971
Fixes #45480
Change-Id: I2823c67a1f3425466b05384cc6d30f5fc8ee6ddc
Reviewed-on: https://go-review.googlesource.com/c/go/+/309109
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Trust: Michael Pratt <mpratt@google.com>
Diffstat (limited to 'src/runtime/symtab.go')
-rw-r--r-- | src/runtime/symtab.go | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/runtime/symtab.go b/src/runtime/symtab.go index 3d1f6126a6..a382cf6002 100644 --- a/src/runtime/symtab.go +++ b/src/runtime/symtab.go @@ -185,7 +185,9 @@ func runtime_expandFinalInlineFrame(stk []uintptr) []uintptr { var cache pcvalueCache inltree := (*[1 << 20]inlinedCall)(inldata) for { - ix := pcdatavalue(f, _PCDATA_InlTreeIndex, tracepc, &cache) + // Non-strict as cgoTraceback may have added bogus PCs + // with a valid funcInfo but invalid PCDATA. + ix := pcdatavalue1(f, _PCDATA_InlTreeIndex, tracepc, &cache, false) if ix < 0 { break } |