Thread (1 message) 1 message, 1 author, 2021-08-31

Re: [PATCH] protocol-caps.c: fix memory leak in send_info()

From: Junio C Hamano <hidden>
Date: 2021-08-31 18:15:37

Bruno Albuquerque [off-list ref] writes:
On Tue, Aug 31, 2021 at 6:46 AM Ævar Arnfjörð Bjarmason
[off-list ref] wrote:

[Replying again as I used HTML mail by mistake. Sorry.]
quoted
Fix a memory leak in a2ba162cda (object-info: support for retrieving
object info, 2021-04-20) which appears to have been based on a
misunderstanding of how the pkt-line.c API works. There is no need to
strdup() input to packet_writer_write(), it's just a printf()-like
format function.

This fixes a potentially large memory leak, since the number of OID
lines the "object-info" call can be arbitrarily large (or a small one
if the request is small).

This makes t5701-git-serve.sh pass again under SANITIZE=leak, as it
did before a2ba162cda2.

Thanks for cleaning up after me. Yes, this was my lack of knowledge on
how the internals of Git works. I was also not aware of SANITIZE=leak
so thanks for the heads up. This looks good to me.
Thanks, both.

Will apply.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help