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.

Sign up for GitHub

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

Jump to bottom

Invoke-ExploitableLeakedHandleCheck - Incorrect thread handle check #66

Closed
v-p-b opened this issue Feb 13, 2025 · 1 comment
Closed

Invoke-ExploitableLeakedHandleCheck - Incorrect thread handle check #66

v-p-b opened this issue Feb 13, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@v-p-b
Copy link

v-p-b commented Feb 13, 2025 •
edited
Loading

$HandleThreadId variable seems to be referenced before assignment (no other mention of it in the entire file):

https://github.com/itm4n/PrivescCheck/blob/cbe802f54cbfe42483c5122a8e8cba1d44379954/src/check/Misc.ps1#L882C26-L882C40

I believe $TargetThreadId should be checked instead.
itm4n added a commit that referenced this issue Feb 13, 2025
@itm4n
Fix issue #66 (incorrect variable name)
5668748
@itm4n
Copy link
Owner

itm4n commented Feb 13, 2025

Indeed, nice catch! Thanks. It turns out there was a similar issue with the process ID.

@itm4n itm4n closed this as completed Feb 15, 2025
@itm4n itm4n added the bug Something isn't working label Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@v-p-b @itm4n