Skip to content

Potential json_file memory leak #561

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
jacobwilliams opened this issue May 27, 2024 · 0 comments · Fixed by #562
Closed

Potential json_file memory leak #561

jacobwilliams opened this issue May 27, 2024 · 0 comments · Fixed by #562

Comments

@jacobwilliams
Copy link
Owner

json_file_load and json_file_load_from_string don't explicitly destroy the pointer p before loading, so if this type is used in a loop, then there would be a memory leak. I think we see this in test 36.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant