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

Add basic support for writing minidumps #30

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

paulkermann
Copy link

With this pull request, windbg will be able to open the dump and parse it.
Obviously, some windbg features will not work (stack trace wont work because thread context is not retrieved and !teb won't work because the teb is not retrieved), but its main functionality will (viewing memory, loaded modules, etc...).

This PR also merges your writer branch into the master.

@paulkermann
Copy link
Author

I have written a tool that does what I have needed. It is present here

@skelsec
Copy link
Owner

skelsec commented Mar 30, 2022

Thank you for the PR. I did some tests on it and it seems to me that you left some debug prints and input calls there.
I have removed those so you don't need to bother, but it will take some more tests until I can merge this because I'd need to find a solution that allows writer to work without including PythonForWindows.
In case I can't find a better solution I'll merge this.

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.

2 participants