qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-17 13:04:54 -07:00
Code Releases Activity

runtime: handle end PC in textAddr

Browse Source 
As the func table contains the end marker of the text section, we
sometimes need to get that address from an offset. Currently
textAddr doesn't handle that address, as it is not within any
text section. Instead of letting the callers not call textAddr
with the end offset, just handle it more elegantly in textAddr.

For #48837.

Change-Id: I6e97e455f6cb66e9680a7aac6152ba6f4cda2e12
Reviewed-on: https://go-review.googlesource.com/c/go/+/354635
Trust: Cherry Mui <cherryyz@google.com>
Trust: Josh Bleecher Snyder <josharian@gmail.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
Reviewed-by: Josh Bleecher Snyder <josharian@gmail.com>
This commit is contained in:
Cherry Mui 2021-10-07 16:26:29 -04:00
parent b69f823ece
commit 6436f5c13d
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/runtime/symtab.go
View File

@ -623,9 +623,7 @@ func moduledataverify1(datap *moduledata) {
} }
min := datap.textAddr(datap.ftab[0].entryoff) min := datap.textAddr(datap.ftab[0].entryoff)
// The max PC is outside of the text section. max := datap.textAddr(datap.ftab[nftab].entryoff)
// Subtract 1 to get a PC inside the text section, look it up, then add 1 back in.
max := datap.textAddr(datap.ftab[nftab].entryoff-1) + 1
if datap.minpc != min || datap.maxpc != max { if datap.minpc != min || datap.maxpc != max {
println("minpc=", hex(datap.minpc), "min=", hex(min), "maxpc=", hex(datap.maxpc), "max=", hex(max)) println("minpc=", hex(datap.minpc), "min=", hex(min), "maxpc=", hex(datap.maxpc), "max=", hex(max))
throw("minpc or maxpc invalid") throw("minpc or maxpc invalid")
@ -660,9 +658,10 @@ func (md *moduledata) textAddr(off32 uint32) uintptr {
off := uintptr(off32) off := uintptr(off32)
res := md.text + off res := md.text + off
if len(md.textsectmap) > 1 { if len(md.textsectmap) > 1 {
for i := range md.textsectmap { for i, sect := range md.textsectmap {
if off >= md.textsectmap[i].vaddr && off < md.textsectmap[i].end { // For the last section, include the end address (etext), as it is included in the functab.
res = md.textsectmap[i].baseaddr + off - md.textsectmap[i].vaddr if off >= sect.vaddr && off < sect.end || (i == len(md.textsectmap)-1 && off == sect.end) {
res = sect.baseaddr + off - sect.vaddr
break break
} }
} }