golang/delve
1
0
Fork 0
mirror of https://github.com/go-delve/delve.git synced 2025-10-31 02:36:18 +08:00
Code Issues Packages Projects Releases Wiki Activity

proc/variables: chanRecvReturnAddress uses outdated frame info

Browse Source 
g.SP refers to the frame the goroutine was in the last time it was
scheduled out. Instead of calling proc.(*Process).stacktrace directly
we should call proc.(*Process).GoroutineStacktrace that substitutes
fresh values retrieved from thread registers when necessary.
This bug leads to occasional problems with `next`.
This commit is contained in:
aarzilli
2015-10-02 13:09:41 +02:00
parent f2e5214e1b
commit d30221dac7
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
proc/variables.go
View File

@ -152,7 +152,7 @@ func (g *G) ChanRecvBlocked() bool {
// chanRecvReturnAddr returns the address of the return from a channel read. // chanRecvReturnAddr returns the address of the return from a channel read.
func (g *G) chanRecvReturnAddr(dbp *Process) (uint64, error) { func (g *G) chanRecvReturnAddr(dbp *Process) (uint64, error) {
locs, err := dbp.stacktrace(g.PC, g.SP, 4) locs, err := dbp.GoroutineStacktrace(g, 4)
if err != nil { if err != nil {
return 0, err return 0, err
} }