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

Fix swapped arguments #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix swapped arguments #32

wants to merge 1 commit into from

Conversation

DaWouw
Copy link

@DaWouw DaWouw commented Dec 14, 2022

There seems to be an inconsistency between the x86 and x64 stubs. The shellcodebase (arg 5) and Flags (arg 6) seem to be swapped for some reason.

This usually results in a multi-hour long delay.

The delay is caused by Flags(shellcodebase) is larger than 16 bytes: Bytes 16->32 are reserved for the sleep variable which is used by Step 5: process our import table in combination with SRDI_OBFUSCATEIMPORTS in the pSleep function.

This fix is only tested and implemented for the Python language. The other languages are still TODO. Any feedback is welcome.

Note: This is, as far as I can see, a direct fix for #31

Signed-off-by: Dima van de Wouw <coffee@dima.ninja>
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

Successfully merging this pull request may close these issues.

None yet

1 participant