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

Memory leak #32

Open
VeganPower opened this issue Feb 7, 2022 · 2 comments
Open

Memory leak #32

VeganPower opened this issue Feb 7, 2022 · 2 comments

Comments

@VeganPower
Copy link

If the allocation on line fast_obj.h#1450 fails the memory allocated at fast_obj.h#1397 is leaked.

return 0;

@thisistherk
Copy link
Owner

Dupe of #30. Happy for someone to have a go at fixing it, but all the array_push calls are a far bigger issue if you care about it failing nicely on out-of-memory.

@VeganPower
Copy link
Author

Ah.. sorry, I checked the open issue and I didn't find anything.
This was a warning from our static analysis tool and I didn't investigate further.
I can give it a try to fix it, but I need approval from my company... and they are slooooooow.

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

No branches or pull requests

2 participants