>  
> +     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

Reply via email to