Skip to content

fix(mem): possible memory leak on Windows#1722

Merged
shirou merged 1 commit intoshirou:masterfrom
uubulb:fix_windows_mem
Oct 11, 2024
Merged

fix(mem): possible memory leak on Windows#1722
shirou merged 1 commit intoshirou:masterfrom
uubulb:fix_windows_mem

Conversation

@uubulb
Copy link
Contributor

@uubulb uubulb commented Oct 5, 2024

Call PdhCloseQuery after a PdhOpenQuery call

Copy link
Owner

@shirou shirou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch! Thank you so much!

There might be similar issues in other functions on Windows as well, so I plan to investigate later and submit a PR if necessary (or PR is always welcome!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants