Re: [PATCH] commit: Fix a memory leak in determine_author_info
On 07/15/2013 04:42 PM, Junio C Hamano wrote:
Show 24 quoted lines
> Stefan Beller <stefanbeller@googlemail.com> writes:
>
>> The date variable is assigned new memory via xmemdupz and 2 lines later
>> it is assigned new memory again via xmalloc, but the first assignment
>> is never freed nor used.
>> ---
>> builtin/commit.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/builtin/commit.c b/builtin/commit.c
>> index 790e5ab..00da83c 100644
>> --- a/builtin/commit.c
>> +++ b/builtin/commit.c
>> @@ -534,7 +534,6 @@ static void determine_author_info(struct strbuf *author_ident)
>> (lb - strlen(" ") -
>> (a + strlen("\nauthor "))));
>> email = xmemdupz(lb + strlen("<"), rb - (lb + strlen("<")));
>> - date = xmemdupz(rb + strlen("> "), eol - (rb + strlen("> ")));
>> len = eol - (rb + strlen("> "));
>> date = xmalloc(len + 2);
>> *date = '@';
>
> Makes sense. I'd assume this is signed-off?
> As I realised I did not sign it, I resend it with some other findings a few hours later, so you'll find it there as well.
See Message-ID <1373837749-14402-2-git-send-email-stefanbeller@googlemail.com> [PATCH 2/4] commit: Fix a memory leak in determine_author_info
Stefan Beller