search

quiet / libcorrect

C library for Convolutional codes and Reed-Solomon
BSD 3-Clause "New" or "Revised" License
362 stars 95 forks source link

memory leak in RS decoding #14

Closed piotrszulcaptiv closed 6 years ago

piotrszulcaptiv commented 6 years ago

Hi, My colleague has found a memory leak in the RS decoder. You do malloc(), but never call free(). I suggest that you take a look if the the same problem exists in your other decoders.

Piotr Szulc

brian-armstrong commented 6 years ago

Thanks for the report. I went ahead and fixed those leaks. I had previously run valgrind on my tests but this involved a more recent section of code that hadn't been run under valgrind.