hash-object: prefix_filename() returns allocated memory these days
commit92a5d1c9b446839a748a465518e5926451b63460
authorJunio C Hamano <gitster@pobox.com>
Tue, 7 Sep 2021 22:10:22 +0000 (7 15:10 -0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 7 Sep 2021 22:18:59 +0000 (7 15:18 -0700)
tree11bf18dd2f07b19ae2fe861c1287924871e58303
parent225bc32a989d7a22fa6addafd4ce7dcd04675dbf
hash-object: prefix_filename() returns allocated memory these days

Back when a1be47e4 (hash-object: fix buffer reuse with --path in a
subdirectory, 2017-03-20) was written, the prefix_filename() helper
used a static piece of memory to the caller, making the caller
responsible for copying it, if it wants to keep it across another
call to the same function.  Two callers of the prefix_filename() in
hash-object were made to xstrdup() the value obtained from it.

But in the same series, when e4da43b1 (prefix_filename: return newly
allocated string, 2017-03-20) changed the rule to gave the caller
possession of the memory, we forgot to revert one of the xstrdup()
changes, allowing the returned value to leak.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/hash-object.c