-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Wrong assembler output for x86 #11724
Comments
the probem is not the disassembler. but the assembler. fix the title pls
you can use keystone or try to fix it if you have some spare time. look at libr/asm/p/asm_x86_nz.c
… On 3 Oct 2018, at 22:07, izhuer ***@***.***> wrote:
Work environment
Questions Answers
OS/arch/bits (mandatory) Ubuntu x86 64
File format of the file you reverse (mandatory) None
Architecture/bits of the file (mandatory) x86/32
r2 -v full output, not truncated (mandatory) radare2 3.0.0-git 19606 @ linux-x86-64 git.2.9.0-271-g237e6c294 commit: 237e6c2 <237e6c2> build: 2018-10-03__15:56:23
Expected behavior
$ rasm2 -d -a x86 -b 32 8b444b10 # used for test the right result
$ mov eax, dword [ebx + ecx*2 + 0x10] # This is right
$ rasm2 -a x86 -b 32 "mov eax, dword [ebx + ecx*2 + 0x10]"
$ 8b444b10
Actual behavior
$ rasm2 -d -a x86 -b 32 8b444b10 # used for test the right result
$ mov eax, dword [ebx + ecx*2 + 0x10] # This is right
$ rasm2 -a x86 -b 32 "mov eax, dword [ebx + ecx*2 + 0x10]"
$ 8b040b # This is wrong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#11724>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3-luiVLpxPss9ucgaeh_pZJMjWNE7-ks5uhRkHgaJpZM4XGxk->.
|
Greetings, x86.nz (x86 handmade assembler) is one of the many x86 assemblers available in radare2 and is updated frequently. Please add a test case so we can fix x86.nz in https://github.com/radare/radare2-regressions/tree/master/new/db/asm (See https://github.com/radare/radare2-regressions/blob/master/new/README.md) You can do so with github web editor without even cloning the repo. You can contribute to x86.nz by completing the following file https://github.com/radare/radare2/blob/master/libr/asm/p/asm_x86_nz.c.
Others x86 assemblers are also available trough r2pm or master (see rasm2 -L list):
|
nobody is gonna fix this for today |
This issue has been automatically marked as stale because it has not had recent activity. Considering a lot has changed since its creation, we kindly ask you to check again if the issue you reported is still relevant in the current version of radare2. If it is, update this issue with a comment, otherwise it will be automatically closed if no further activity occurs. Thank you for your contributions. |
Work environment
Expected behavior
Actual behavior
The text was updated successfully, but these errors were encountered: