]> git.ipfire.org Git - thirdparty/git.git/commit
Makefile: turn on NO_MMAP when building with LSan
authorJeff King <peff@peff.net>
Thu, 5 Mar 2026 23:13:05 +0000 (18:13 -0500)
committerJunio C Hamano <gitster@pobox.com>
Sat, 7 Mar 2026 05:12:10 +0000 (21:12 -0800)
commit00611d86c66f4230eb229b2b7dc5ac413aef2221
treed31f030719dc2802e492957a4e9f4358a54fc919
parentb68e875bec29e538a67ad38009ea1c02fa877258
Makefile: turn on NO_MMAP when building with LSan

The past few commits fixed some cases where we leak memory allocated by
mmap(). Building with SANITIZE=leak doesn't detect these because it
covers only heap buffers allocated by malloc().

But if we build with NO_MMAP, our compat mmap() implementation will
allocate a heap buffer and pread() into it. And thus Lsan will detect
these leaks for free.

Using NO_MMAP is less performant, of course, since we have to use extra
memory and read in the whole file, rather than faulting in pages from
disk. But LSan builds are already slow, and this doesn't make them
measurably worse. Getting extra coverage for our leak-checking is worth
it.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Makefile