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

16bit disassembly using wrong syntax #213

Closed
radare opened this issue Nov 10, 2014 · 8 comments
Closed

16bit disassembly using wrong syntax #213

radare opened this issue Nov 10, 2014 · 8 comments

Comments

@radare
Copy link
Contributor

radare commented Nov 10, 2014

[pancake@koega ~]$ rasm2 -b16 -d eaaaff00f0
ljmp 0xffaa, 0xf000
[pancake@koega ~]$ rasm2 -a x86.udis -b16 -d eaaaff00f0
jmp word 0xf000:0xffaa

the first output is capstone, the second udis86. the capstone output for 16bit disassembly is somewhat confusing to read.

@aquynh
Copy link
Collaborator

aquynh commented Nov 10, 2014

hmm this is the same output generated by Intel's XED.

what is the output of Gnu & IDA in this case?

thanks.

@XVilka
Copy link
Contributor

XVilka commented Nov 10, 2014

@aquynh IDA using the 2nd notation, and it is more common in the 16-bit (aka DOS) world due to the early segment addressing adoption.

@mrexodia
Copy link
Contributor

Intel XED uses a very strange syntax indeed, also noticed that while developing a tool with XED.

@aquynh
Copy link
Collaborator

aquynh commented Nov 10, 2014

while you guys are at it, can you also try jmp m16:16? do we have similar issue here?

@radare
Copy link
Contributor Author

radare commented Nov 10, 2014

can you give us some hexpairs to construct that instruction?

On 11/10/2014 01:51 PM, Nguyen Anh Quynh wrote:

while you guys are at it, can you also try jmp m16:16? do we have
similar
issue here?


Reply to this email directly or view it on GitHub
#213 (comment).

@aquynh
Copy link
Collaborator

aquynh commented Nov 10, 2014

this instruction:

jmp ptr ds:[di]

but this has no issue with Capstone.

@aquynh
Copy link
Collaborator

aquynh commented Nov 10, 2014

fixed this issue with commit 4f99ed2

thanks.

@radare
Copy link
Contributor Author

radare commented Nov 10, 2014

That was quick! thanks!

@radare radare closed this as completed Nov 10, 2014
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

4 participants