Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid instruction decoding #23

Closed
XVilka opened this issue Nov 10, 2012 · 6 comments
Closed

Invalid instruction decoding #23

XVilka opened this issue Nov 10, 2012 · 6 comments

Comments

@XVilka
Copy link
Contributor

XVilka commented Nov 10, 2012

See output of radare2 (especially wrong last string at text.0x576 which is obviously wrong) - http://sprunge.us/QVcP

This is result of disassemling this file http://xvilka.me/t1 ( C source is http://xvilka.me/t1.c )

Here is IDA Pro result for comparison - http://xvilka.me/t1.asm and http://xvilka.me/t1.lst

@XVilka
Copy link
Contributor Author

XVilka commented Nov 10, 2012

See text.0x50c too - r2 output is 'o16 nop' while ida pro output is 'align 10h'

@radare
Copy link
Collaborator

radare commented Nov 10, 2012

Align 10h is 16 bits of nop padding. Which is a o16 nop. We can change that output to just show "nop16".. But if i guessed right both representations are the same.

On Nov 10, 2012, at 17:33, Anton Kochkov [email protected] wrote:

See text.0x50c too - r2 output is 'o16 nop' while ida pro output is 'align 10h'


Reply to this email directly or view it on GitHub.

@radare
Copy link
Collaborator

radare commented Nov 10, 2012

Can you upload the disassembly of radare with scr.color=false? With just the affected code? I'm not at home now, but I can try to solve it from my phone.. :)

Thanks for reporting!

On Nov 10, 2012, at 17:31, Anton Kochkov [email protected] wrote:

See output of radare2 (especially wrong last string at text.0x576 which is obviously wrong) - http://sprunge.us/AEZW

This is result of disassemling this file http://xvilka.me/t1 ( C source is http://xvilka.me/t1.c )

Here is IDA Pro result for comparison - http://xvilka.me/t1.asm and http://xvilka.me/t1.lst


Reply to this email directly or view it on GitHub.

@XVilka
Copy link
Contributor Author

XVilka commented Nov 10, 2012

Already fixed that, see updated message or direct link http://sprunge.us/QVcP

@radare
Copy link
Collaborator

radare commented Nov 13, 2012

should be fixed now. can you confirm?

@XVilka
Copy link
Contributor Author

XVilka commented Nov 13, 2012

RESOLVED FIXED ;)

@XVilka XVilka closed this as completed Nov 13, 2012
@radare radare mentioned this issue Oct 25, 2015
17 tasks
@radare radare mentioned this issue Nov 7, 2015
radare added a commit that referenced this issue Nov 23, 2015
@radare radare mentioned this issue Sep 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants