yafetch icon indicating copy to clipboard operation
yafetch copied to clipboard

Memory leaks

Open ar1ja opened this issue 3 years ago • 0 comments

https://github.com/yrwq/yafetch/blob/main/src/func.c#L35

You never free() new, I found this while inspecting my fork of yafetch, might want to take a look at this commit as a whole:

https://github.com/TruncatedDinosour/yafetch/commit/95226549f4603c4c36bc92949b6fae6aaa144bc4

And maybe even adopting this script to prevent future memory issues:

https://github.com/TruncatedDinosour/yafetch/blob/main/scripts/tests.sh

Just modify it to not include the ./configure script testing :)

ar1ja avatar Apr 03 '22 15:04 ar1ja