Memory leaks fixes - Githubissues

alexfru / SmallerC

Simple C compiler

BSD 2-Clause "Simplified" License

1.35k stars 155 forks source link

Closed aleexmaftei closed 3 months ago

aleexmaftei commented 2 years ago

Fixes for Issue #32.

sskras commented 11 months ago

Looks quite nice. Ping @alexfru :)

alexfru commented 3 months ago

Haven't looked at everything yet, but the added va_end(vl);'s need to be under the same #if... as the matching va_start(vl, format);.

alexfru commented 3 months ago

On top of that:

declaration of oldpp will fail to compile with pre-C99 compilers
oldpp needs to be updated after successful call to realloc()
there's another instance of fatargs() in ucpp
main(), which calls fatargs(), doesn't free memory it receives from fatargs(), but I'm too lazy to fix this one now

I'm gonna fix these (except main()'s) myself.