Skip to content

Conversation

@xezon
Copy link

@xezon xezon commented Nov 5, 2025

This change implements a reference counter in DbgHelpLoader to allow for independent users.

TODO

  • Replicate in Generals

@xezon xezon added Minor Severity: Minor < Major < Critical < Blocker Debug Is mostly debug functionality ThisProject The issue was introduced by this project, or this task is specific to this project Fix Is fixing something, but is not user facing labels Nov 5, 2025
Copy link

@slurmlord slurmlord left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link

@Mauller Mauller left a comment

Choose a reason for hiding this comment

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

The Critical Section will protect us!

@Mauller Mauller added the Approved Pull Request was approved label Nov 7, 2025
@xezon
Copy link
Author

xezon commented Nov 8, 2025

Replicated in Generals without conflicts

@xezon xezon changed the title fix(debug): Implement reference counter in DbgHelpLoader to allow for independent users fix(debug): Implement reference counter and thread safety in DbgHelpLoader to allow for independent users Nov 8, 2025
@xezon xezon merged commit 5f6a948 into TheSuperHackers:main Nov 8, 2025
19 checks passed
@xezon xezon deleted the xezon/fix-dbghelploader branch November 8, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Pull Request was approved Debug Is mostly debug functionality Fix Is fixing something, but is not user facing Minor Severity: Minor < Major < Critical < Blocker ThisProject The issue was introduced by this project, or this task is specific to this project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants