# [PATCH 1/2] transport: Free leaking head in transport_print_push_status 4 messages from 2014-10-21 to 2014-10-22. Participants: Stefan Beller, Junio C Hamano. Thread: https://gitlist.dev/t/37769 ## Stefan Beller, 2014-10-21 01:50 Subject: [PATCH 1/2] transport: Free leaking head in transport_print_push_status Message-ID: <1413856245-5443-1-git-send-email-stefanbeller@gmail.com> URL: https://gitlist.dev/e/1413856245-5443-1-git-send-email-stefanbeller%40gmail.com ``` From: Stefan Beller Found by scan.coverity.com (ID: 1248110) Signed-off-by: Stefan Beller --- transport.c | 1 + 1 file changed, 1 insertion(+) diff --git a/transport.c b/transport.c index 055d2a2..fbab6ee 100644 --- a/transport.c +++ b/transport.c @@ -774,6 +774,7 @@ void transport_print_push_status(const char *dest, struct ref *refs, *reject_reasons |= REJECT_NEEDS_FORCE; } } + free(head); } void transport_verify_remote_names(int nr_heads, const char **heads) -- 2.1.2.332.g8071e93 ``` ## Stefan Beller, 2014-10-21 01:50 Subject: [PATCH 2/2] .mailmap: Add Stefan Bellers corporate mail address Message-ID: <1413856245-5443-2-git-send-email-stefanbeller@gmail.com> URL: https://gitlist.dev/e/1413856245-5443-2-git-send-email-stefanbeller%40gmail.com In-Reply-To: <1413856245-5443-1-git-send-email-stefanbeller@gmail.com> ``` From: Stefan Beller Note that despite the private address being first, Google owns the copyright on this patch as any other patch I'll be sending signed off by the sbeller@google.com address. Signed-off-by: Stefan Beller --- .mailmap | 1 + 1 file changed, 1 insertion(+) diff --git a/.mailmap b/.mailmap index 8aefb5a..bb6f52e 100644 --- a/.mailmap +++ b/.mailmap @@ -205,6 +205,7 @@ Shawn O. Pearce Simon Hausmann Simon Hausmann Stefan Beller +Stefan Beller Stefan Naewe Stefan Naewe Stefan Sperling -- 2.1.2.332.g8071e93 ``` ## Junio C Hamano, 2014-10-21 05:53 Subject: Re: [PATCH 2/2] .mailmap: Add Stefan Bellers corporate mail address Message-ID: URL: https://gitlist.dev/e/xmqqppdm0y02.fsf%40gitster.dls.corp.google.com In-Reply-To: <1413856245-5443-2-git-send-email-stefanbeller@gmail.com> ``` Stefan Beller writes: > From: Stefan Beller > > Note that despite the private address being first, Google owns the > copyright on this patch as any other patch I'll be sending signed off > by the sbeller@google.com address. While Googlers are encouraged to interact with external open source projects using their corp address when working as new contributors, Googlers are also encouraged to keep using the same pre-corp address when working with projects they have been part of for continuity. After all, even if you send patches from your @gmail.com address while being employed by Google, the patches are copyright Google, not yours, and DCO (a) "I have the right to submit" covers your submissions just fine (i.e. it does not say "I own the copyright"), as long as you are cleared to contribute your patches to us by Google's open source office (which I happen to know that you have already done). The mailmap is primarily to correct past mistakes and document the new addresses of those who lost the old address to maintain their reachability. It is not about planning to send patches from two different addresses chosen by the phase of the moon or something. I do not terribly mind, but is this patch really necessary? > > Signed-off-by: Stefan Beller > --- > .mailmap | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.mailmap b/.mailmap > index 8aefb5a..bb6f52e 100644 > --- a/.mailmap > +++ b/.mailmap > @@ -205,6 +205,7 @@ Shawn O. Pearce > Simon Hausmann > Simon Hausmann > Stefan Beller > +Stefan Beller > Stefan Naewe > Stefan Naewe > Stefan Sperling ``` ## Junio C Hamano, 2014-10-22 22:03 Subject: Re: [PATCH 2/2] .mailmap: Add Stefan Bellers corporate mail address Message-ID: URL: https://gitlist.dev/e/xmqqbnp3u5hz.fsf%40gitster.dls.corp.google.com In-Reply-To: ``` Stefan Beller writes: > If you want me to continue using my gmail address, please check the > authorship > from the previous patch ([PATCH 1/2] transport: Free leaking head in > transport_print_push_status) > and remove the "From: Stefan Beller " line, > so we don't end up needing this patch. As I said, I do not care too deeply either way. If you feel that switching to a different e-mail address to sign off your patches from now on until your employment is terminated has merit to the project, that's fine by me. Thanks. ```