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

Building MinGW32 + Clang 10 + wclang on Ubuntu 20.04 #105

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

Conversation

janedoe-lab
Copy link

These changes required to make master branch of donut compile using Clang 10 with Mingw32 under Ubuntu 20.04.

Things working for me:

  • Compiling release version of donut and loader 32/64bit in Ubuntu 20.04 using clang+lld from LLVM 10 running under mingw32 and wclang
  • Generating donut shellcode using wine donut.exe -o shellcode.bin -b 1 -a 1 exec.exe
  • Both 32 and 64 shellcodes work

What doesn't work:

  • AMSI bypass does NOT work - it crashes right now inside generated shellcode's loader
  • Debug version of loader32.exe/loader64.exe fail to load debug instance - fails on "Resolving address of CreateThread"
  • dev branch does not work with these changes for me

@S4ntiagoP
Copy link
Contributor

Nice work @janedoe-lab !
One question: Do you happen to know how to tell clang that it should use Intel syntax from the inline assembly?

@janedoe-lab
Copy link
Author

Sorry, not experienced much in all these tools. This branch is a result of bruteforcing a right set of options to get it going, and not based on any prior knowledge of these tools :)

@S4ntiagoP
Copy link
Contributor

understood, thanks 😄

@TheWover TheWover added help wanted Extra attention is needed good first issue Good for newcomers invalid This doesn't seem right labels Dec 5, 2022
@TheWover
Copy link
Owner

TheWover commented Dec 5, 2022

Since this is incomplete but may still be valuable to anyone attempting to do the same thing, I will not merge it for version 1.0 but will mark it as invalid and may revisit later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants