From: Shawn O. Pearce Date: Fri, 25 May 2007 02:09:39 GMT Subject: Re: [PATCH] fix memory leak in parse_object when check_sha1_signature fails Message-ID: <20070525020939.GT28023@spearce.org> In-Reply-To: <1b46aba20705241846u615785eeh7c3ebeaf840b2180@mail.gmail.com> Carlos Rica wrote: > When check_sha1_signature fails, program is not terminated: > it prints an error message and returns NULL, so the > buffer returned by read_sha1_file should be freed before. ... > if (check_sha1_signature(sha1, buffer, size, typename(type)) > < 0) { ... gmail strikes again. The above line is wrapped, which means your patch is whitespace damaged and won't apply properly. -- Shawn.