fast_obj icon indicating copy to clipboard operation
fast_obj copied to clipboard

Published 20 hours ago • verified publisher icon thisistherk

Memory leak

Open VeganPower opened this issue 3 years ago • 2 comments

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

https://github.com/thisistherk/fast_obj/blob/85778da5fc320b7e52885f8e869edc079695cc79/fast_obj.h#L1450

VeganPower avatar Feb 07 '22 09:02 VeganPower

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.

thisistherk avatar Feb 07 '22 10:02 thisistherk

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.

VeganPower avatar Feb 07 '22 10:02 VeganPower