magna25/Attack-Lab

phase 4 correction

Closed this issue · 2 comments

cswpy commented

The popq %rax in the farm does not necessarily require 58 (the corresponding machine code) to be at the rightmost position of the instruciton. It works even if it is in the middle.

Good catch! submit a pull request and I'll merge it.

Hey there I know it has been a while but I have created the necessary pull request (#12) for this issue.