On Wed, Mar 27, 2013 at 17:13:04 +0100, Denys Vlasenko wrote:
On 03/27/2013 02:41 PM, Martin Milata wrote:
> The base_api_url is not free()d anywhere (I understand that using free
> in main() is not strictly necessary, but I believe we should be
> consistent -- either free everything or nothing).
> Other than that, the patch looks good:)
I'm ok with freeing at the end too.
Can you add that and push the change?
Alright, I pushed it. There are two paths where we still leak the memory
(together with other memory allocated elsewhere) - I suppose we should
fix those when we start using valgrind ...