From: Jeff King <peff@peff.net>
Date: Sun, 12 Oct 2014 01:53:21 GMT
Subject: Re: [PATCH] receive-pack: plug minor memory leak in unpack()
Message-ID: <20141012015321.GA15272@peff.net>
In-Reply-To: <54390DC0.8060302@web.de>

On Sat, Oct 11, 2014 at 01:00:16PM +0200, René Scharfe wrote:

> The argv_array used in unpack() is never freed.  Instead of adding
> explicit calls to argv_array_clear() use the args member of struct
> child_process and let run_command() and friends clean up for us.

Looks good. I notice that the recently added prepare_push_cert_sha1 uses
an argv_array to create the child_process.env, and we leak the result. I
wonder if run-command should provide a managed env array similar to the
"args" array.

-Peff