radareorg / radare2

UNIX-like reverse engineering framework and command-line toolset
https://www.radare.org/
GNU Lesser General Public License v3.0
20.38k stars 2.97k forks source link

`vbg` last line drown with off-by-one position #13367

Closed XVilka closed 5 years ago

XVilka commented 5 years ago

Work environment

Questions Answers
OS/arch/bits (mandatory) Fedora 29 x86_64
File format of the file you reverse (mandatory) ELF
Architecture/bits of the file (mandatory) x86/64
r2 -v full output, not truncated (mandatory) radare2 3.4.0-git 22180 @ linux-x86-64 git.3.3.0-138-g0c9ccb45d

Expected behavior

Proper

Actual behavior

Steps to reproduce the behavior

r2 /bin/ls
> aaa
> s main
> vbg

Additional Logs, screenshots, source-code, configuration dump, ...

image

Drag and drop zip archives containing the Additional info here, don't use external services or link. fedora29_bin_ls.zip

radare commented 5 years ago

shuold be fixed after https://github.com/radare/radare2/pull/13472

can you confirm?

XVilka commented 5 years ago

It wasn't fixed and now is even worse: image

radare commented 5 years ago

? Wat

On 25 Mar 2019, at 06:20, Anton Kochkov notifications@github.com wrote:

It wasn't fixed and now even worse:

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.

radare commented 5 years ago

Aaah i though you were refering to the bug that was before the pr that the disasm was using the offset one row off the one you had selected. That was fixed :p

On 25 Mar 2019, at 06:20, Anton Kochkov notifications@github.com wrote:

It wasn't fixed and now even worse:

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.