Windows support #33
Merged
Windows support #33
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Apparently it matters or it gets ignored
This provides the same functionality, while allowing windows to use file descriptors for the pipes
Change build location for cbs, add exclude option
No longer using __asm__ to explicitly name the symbol
A few challenges had a `-include cgc__defines.h` flag when compiling. Now they explicitly include the header instead
This changes the signature of the main() function in the challenges and POVs to be "int main(int cgc_argc, char *cgc_argv[])". The argument names were chosen to avoid conflicts with challenges that use the names argc/argv for their own purposes. This should help out some analysis tools (specifically KLEE) that expect main() to have at least two arguments. This commit doesn't touch challenges that already had arguments to main. Such challenges need a bit more careful handling, since they expect the address of the flag page to be passed in the first argument.
As opposed to trying to run tests for everything in `challenges`, which may not all be built
Instead of trying to connect to a separately running server, cb-replay will now directly launch the challenge it's trying to test, gather the results and kill it. NOTE: as of this commit multibin support is still a WIP POLLs can now be run on Windows
To accomplish this, minidumps are enabled and are kept in a known directory. Then `cdb` is used to to get the registers from the crash dumps
Use the `-msvc` flag to try building with normal cl
Similar to cb-replay, this now launches the challenges it needs instead of connecting to a server POVs now working on all OSs
withzombies
approved these changes
Jul 18, 2017
|
Rebase and merge please |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds support for building the challenges on Windows using
clang-cl, and testing them with the included utilitesWhile the current build process works, some future work could include:
Clang/C2- this would allow us to use Microsoft CodeGen instead ofclangclCloses #11
Closes #9 (see #27)