>
> + if (!outf)
> + return NULL;
> + if (!tmp_errfile)
> + {
> + g_unlink(outf);
> + g_free(outf);
> + return NULL;
> + }
that's not correct, if `outf` failed to be created but not `tmp_errfile`, that
last one is leaked. Unlikely I agreed, but still possible and probably worth
fixing.
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/957/files/1c4aaa0eb72aa9e3c60cd604d4c20ca5f7dc5c8f..b72b8596346c5dcecb4ecb55d12311b2405a470f#r61931561