Re: [PATCH] pkt-line: re-'static'-ify buffer in packet_write_fmt_1()
- Date: Sun, 27 Aug 2017 22:04:55 +0200
- From: Lars Schneider <larsxschneider@xxxxxxxxx>
- Subject: Re: [PATCH] pkt-line: re-'static'-ify buffer in packet_write_fmt_1()
> On 27 Aug 2017, at 21:09, Martin Ågren <martin.agren@xxxxxxxxx> wrote:
> On 27 August 2017 at 20:21, Lars Schneider <larsxschneider@xxxxxxxxx> wrote:
>>> On 27 Aug 2017, at 09:37, Martin Ågren <martin.agren@xxxxxxxxx> wrote:
>>> The static-ness was silently dropped in commit 70428d1a5 ("pkt-line: add
>>> packet_write_fmt_gently()", 2016-10-16). As a result, for each call to
>>> packet_write_fmt_1, we allocate and leak a buffer.
>> Oh :-( Thanks for detecting and fixing the leak.
>> How did you actually find it? Reading the code or did you use some
> Valgrind found it for me. Most leaks it reports are "ok" since we're
> about to exit, it just takes more or less effort to realize that...
I am sorry if I am bugging you here, but I would really like to
understand how you found it.
I did run all tests under valgrind using "make valgrind" and I found
the following files with potential issues:
cat valgrind.out | perl -nE 'say /^==.+\((.+)\:.+\)$/' | sort | uniq -c
No mention of "pkt-line.c". Did you run Git with valgrind on one of
your repositories to find it?