tbeu / matio

MATLAB MAT File I/O Library
https://matio.sourceforge.io
BSD 2-Clause "Simplified" License
330 stars 97 forks source link

The fix for issue 131 "memory leaks issue in Mat_VarCalloc" does not seem effective #175

Closed lewyssmith closed 2 years ago

lewyssmith commented 3 years ago

Original github ref: https://github.com/tbeu/matio/issues/131 Doubt: https://bugs.mageia.org/show_bug.cgi?id=27969 (comment 4) Note that the tester used a test file '006-memleak', and commented "gives the impression that the patch does not work. Upstream had difficulty verifying the fix or even the issue". So there may be a misunderstanding somewhere - yet a segfault is a segfault, which should never happen.

tbeu commented 3 years ago

If you have a fix at hand feel free to create a new PR, too.

tbeu commented 2 years ago

I can not reproduce with 12b211cda2ef5abec89304bd1ba3a79a82229feb.