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

Fix lsass finder #69

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

Fix lsass finder #69

wants to merge 1 commit into from

Conversation

ebfe90
Copy link

@ebfe90 ebfe90 commented Jan 19, 2021

Hello again,

Sorry I've forgot a byref in the NtQuerySystemInformation. Thats why it wasn't working on 32 bit.
I've tested the code on 32 bit and 64 bit and added a method in case the main one failed (QueryFullProcessImageNameW).
Do pypykatz supposed to support older systems XP/2000 ?
Let me know if you have any comment.
Thank you for your work.

@skelsec
Copy link
Owner

skelsec commented Jan 23, 2021

Win 7 32 bit CloseHandle raises exception "Handle invalid". I checked and CloseHandle receives the handle value correctly so I'm not sure why it gives that error. Will do further checks on that.
Commenting out CloseHandle "solves" the issue but there must be a better way.

pypykatz live parsing is not supported on winxp and 2000 because python3 is not supported on those platforms.

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