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 crash #413

Merged
merged 3 commits into from
Jan 18, 2025
Merged

Fix crash #413

merged 3 commits into from
Jan 18, 2025

Conversation

WiserTixx
Copy link
Collaborator

@WiserTixx WiserTixx commented Jan 16, 2025

This PR fixes the segfault caused by rapidjson on the first heartbeat, and the memory leak that happens during mod hashing.


By creating this pull request, I understand that code that is AI generated or otherwise automatically generated may be rejected without further discussion.
I declare that I fully understand all code I pushed into this PR, and wrote all this code myself and own the rights to this code.

Copy link
Collaborator

@SaltySnail SaltySnail left a comment

Choose a reason for hiding this comment

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

lgtm

@WiserTixx WiserTixx merged commit e90f1af into minor Jan 18, 2025
9 checks passed
@WiserTixx WiserTixx deleted the fix-crash branch January 18, 2025 20:36
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