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

Refactored assembly, added NASM support, Added 32-Bit Support (Wow64 & x86 Native) #18

Closed
wants to merge 6 commits into from

Conversation

xenoscr
Copy link
Contributor

@xenoscr xenoscr commented Mar 3, 2022

  • Added x86 support (wow64+native) with inspiration from the existing pull request (Enhancements - x86 support (WoW64 + native), ability to change function prefix #9) and another project (https://github.com/mai1zhi2/SysWhispers2_x86).
  • Added NASM assembler support
  • Updated README.md with instructions to build using MinGW and NASM
  • Updated README.md to include instructions to support adding the resulting code to Visual Studio to support both x86 and x64 targets and the x64 limitation was removed.
  • The main script has been updated to include a stub with common code for both x86 and x64 assembly. I've introduced a function called "WhisperMain" that is shared by all syscalls in both x86 and x64. The result is a reduced amount of duplicated code.
  • base.c was updated to support the fs register used in x86.
  • New version of the example output code were generated.

…milar to jthuraisamy#9 and some inspiration from https://github.com/mai1zhi2/SysWhispers2_x86/blob/main/SysWhispers2_x86_WOW64Gate/SyscallsStubs.asm.

Updated README.md with instructions to support both x86 and x64 targets.
Updated base.c to support x86 registers
Updated syswhispers.py to add x86 and x64 stubs to support a refactored version that uses fewer redundant instructions.
@odzhan
Copy link
Collaborator

odzhan commented Mar 13, 2022

Hello,
Since there already was a PR to support x86 syscalls for native 32-Bit and WOW64, it was merged first. If you still have improvements to add, please make changes to the existing branch and submit a PR again. Apologies for the inconvenience.

@xenoscr
Copy link
Contributor Author

xenoscr commented Mar 13, 2022

Understood. Don't worry, it's not much trouble. The forks and pulls from your project are indicators of the interest in this capability. I will close this and submit a new request. I have some ideas to pull a few of these forks together and keep some of the refactoring.

@xenoscr xenoscr closed this Mar 13, 2022
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

2 participants