[Github-comments] [geany/geany] TM cleanups (#957)

[Colomban Wendling]

>  
> +	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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160503/66a1d869/attachment.html>