PenaDevops/delve
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

proc: bugfix: wrong register used for CH (#763)

Browse Source 
We only use Registers.Get besides for evaluating the argument of a CALL
instruction so this doesn't matter in practice, but it's still wrong.
This commit is contained in:
Alessandro Arzilli 2017-03-13 18:54:41 +01:00 committed by Derek Parker
parent bd48358de3
commit b7f9e3c73e
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/proc/registers_darwin_amd64.go
View File

@ -134,7 +134,7 @@ func (r *Regs) Get(n int) (uint64, error) {
case x86asm.AH: case x86asm.AH:
return (r.rax >> 8) & mask8, nil return (r.rax >> 8) & mask8, nil
case x86asm.CH: case x86asm.CH:
return (r.rax >> 8) & mask8, nil return (r.rcx >> 8) & mask8, nil
case x86asm.DH: case x86asm.DH:
return (r.rdx >> 8) & mask8, nil return (r.rdx >> 8) & mask8, nil
case x86asm.BH: case x86asm.BH:

2
pkg/proc/registers_linux_amd64.go
View File

@ -112,7 +112,7 @@ func (r *Regs) Get(n int) (uint64, error) {
case x86asm.AH: case x86asm.AH:
return (r.regs.Rax >> 8) & mask8, nil return (r.regs.Rax >> 8) & mask8, nil
case x86asm.CH: case x86asm.CH:
return (r.regs.Rax >> 8) & mask8, nil return (r.regs.Rcx >> 8) & mask8, nil
case x86asm.DH: case x86asm.DH:
return (r.regs.Rdx >> 8) & mask8, nil return (r.regs.Rdx >> 8) & mask8, nil
case x86asm.BH: case x86asm.BH:

2
pkg/proc/registers_windows_amd64.go
View File

@ -159,7 +159,7 @@ func (r *Regs) Get(n int) (uint64, error) {
case x86asm.AH: case x86asm.AH:
return (r.rax >> 8) & mask8, nil return (r.rax >> 8) & mask8, nil
case x86asm.CH: case x86asm.CH:
return (r.rax >> 8) & mask8, nil return (r.rcx >> 8) & mask8, nil
case x86asm.DH: case x86asm.DH:
return (r.rdx >> 8) & mask8, nil return (r.rdx >> 8) & mask8, nil
case x86asm.BH: case x86asm.BH: