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.

Sign up for GitHub

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

Jump to bottom

REPE instructions incorrectly disassembled #97

Closed
haxelion opened this issue Jun 7, 2013 · 1 comment
Closed

REPE instructions incorrectly disassembled #97

haxelion opened this issue Jun 7, 2013 · 1 comment

Comments

@haxelion
Copy link

haxelion commented Jun 7, 2013

According to the Intel® 64 and IA-32 Architectures Software Developer’s Manual,

"rep cmpsd" should be "repe cmpsd"
"rep scasd" should be "repe scasd"

example:

% rasm2 -d F3A7
rep cmpsd
% rasm2 -d F3AF
rep scasd
@radare radare mentioned this issue Jun 8, 2013
Closed
@radare
Copy link
Collaborator

radare commented Jun 8, 2013

According to @vmt this bug has been already fixed in his repo, and we should update our local copy. I will also review that no regressions appear after the merge. Will close the bug when done.

Thanks

@radare radare closed this as completed in 5e08714 Jun 9, 2013
yossizap pushed a commit to yossizap/radare2 that referenced this issue Dec 30, 2019
@hteso
Fixes radareorg#97. Responsiveness stopped working with some PR
7289337
yossizap pushed a commit to yossizap/radare2 that referenced this issue Dec 30, 2019
@mrexodia
compile with MSVC again (this partially screws up 7289337, and issue r…
95ca3d5 
…adareorg#97)
yossizap pushed a commit to yossizap/radare2 that referenced this issue Dec 30, 2019
@mrexodia
restore the 'responsive' functionality (closes issue radareorg#97)
400f6be
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@radare @haxelion