-
Notifications
You must be signed in to change notification settings - Fork 454
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
pushw imm #207
Comments
see related issue #10. it seems the fix for it is just partially correct. can you send a PR to add your case to x86_issue10.py ? pushw is an invalid mnemonic |
Added test case for issue keystone-engine#207. keystone-engine#207
Maybe I'm wrong but the only file modified to "fix" it is the regress test "x86_issue10.py" and it fails. I have this commit : And this is the result of regress.py :
This is for x86_issue10.py :
|
Up |
I've also encountered the issue on my side. I'm using the latest keystone engine bindings in a python script to compile 32 bit assembly. The issue can also be reproduced with this keystone engine online demo : For example compiling the following assembly : gives the following ouput : but it should be : Let me know if you guys need help for this issue. Best regards |
I discovered this issue again while running my code on version 0.9.2. Has the problem been addressed in #578, and is it planned to be resolved in which version? Looking forward to your reply. |
Wrong output for: "push word 0x1122". Expected: "66 68 22 11". Kstool output: "68 22 11 00 00".
I then tried with: "pushw 0x1122". Excepted: "66 68 22 11". Kstool output: KS_ERR_ASM_MNEMONICFAIL.
If the mnemonic "pushw" is not going to be supported, I think parsing correctly the size specification, like "word" in "push word 0x1122" is the fix. It all depends on the first byte 0x66.
The text was updated successfully, but these errors were encountered: