-
Notifications
You must be signed in to change notification settings - Fork 736
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
Qdb x86-64 support & Windows kernel structures #1217
Qdb x86-64 support & Windows kernel structures #1217
Conversation
Hi @HackingFrogWithSunglasses, welcome to Qiling and thanks for the contribution. My suggestion is that we take that step by step, and first incorporate the changes for QDB (given a review from its owner), and then move on to the changes in Windows. Would you be able to split the PR into two different PRs, so we can review and merge them separately? |
Hi, thanks. You are welcome. As of present the initializer for those additional structures only gets called when IsDriver() is true in the loader. From my understanding of the code flow, that means that the pointers should not be mapped into the GS register unless we're loading a driver. Nonetheless, I am more than happy to split this PR out certainly. I only submitted a PR because I needed this for my own project and saw it was not present. I'll split the PRs out and we can go from there! |
This PR adds x86-x64 support to QDB and adds limited implementation of additional Windows kernel structures such as _KTHREAD, _KPROCESS, _KPCR, _KPCRB and _KNODE. As well as adding their pointers to the correct offset in GS.
Checklist
Which kind of PR do you create?
Coding convention?
Extra tests?
Changelog?
Target branch?
One last thing