From: Taylor Blau <me@ttaylorr.com>
To: git@vger.kernel.org
Cc: "Victoria Dye" <vdye@github.com>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
"Jeff King" <peff@peff.net>, "Junio C Hamano" <gitster@pobox.com>
Subject: linux-leaks CI failure on master
Date: Tue, 18 Oct 2022 14:37:44 -0400 [thread overview]
Message-ID: <Y07yeEQu+C7AH7oN@nand.local> (raw)
After Junio pushed out the tags for v2.38.1 and friends, I noticed that
our linux-leaks CI job is failing t1300.104 and t1300.109, claiming that
there is a leak here:
Direct leak of 7 byte(s) in 1 object(s) allocated from:
#0 0x7fc4a5b319c1 in __interceptor_malloc ../../../../src/libsanitizer/lsan/lsan_interceptors.cpp:54
#1 0x7fc4a598638e in __GI___strdup /build/glibc-SzIz7B/glibc-2.31/string/strdup.c:42
#2 0x55ba23538f7c in xstrdup wrapper.c:39
#3 0x55ba233e258c in git_config_string config.c:1445
#4 0x55ba233e6b06 in git_config_include config.c:439
#5 0x55ba233e063f in get_value config.c:910
#6 0x55ba233e063f in git_parse_source config.c:1092
#7 0x55ba233e063f in do_config_from config.c:1937
#8 0x55ba233e3d2d in do_config_from_file config.c:1965
#9 0x55ba233e3d2d in git_config_from_file_with_options config.c:1987
#10 0x55ba233e4793 in git_config_from_file config.c:1996
#11 0x55ba233e4793 in do_git_config_sequence config.c:2143
#12 0x55ba233e4793 in config_with_options config.c:2198
#13 0x55ba233e4a50 in read_early_config config.c:2255
#14 0x55ba233acb36 in alias_lookup alias.c:35
#15 0x55ba232d0748 in handle_alias git.c:346
#16 0x55ba232d0748 in run_argv git.c:851
#17 0x55ba232d0748 in cmd_main git.c:921
#18 0x55ba232cee03 in main common-main.c:57
#19 0x7fc4a590b082 in __libc_start_main ../csu/libc-start.c:308
#20 0x55ba232cee8d in _start (git+0x1fe8d)
I can't reproduce the failure locally with gcc (Debian 10.3.0-15)
10.3.0, but Victoria (CC'd) can reproduce the failure with 9.4.0.
Interestingly, the failure only appears when compiling with `-O2`, but
not `-O0` or `-O1`.
This is reminiscent to me of the discussion in:
https://lore.kernel.org/git/Yy4eo6500C0ijhk+@coredump.intra.peff.net/
I'm not sure if I'm content to treat the 9.4.0 behavior as a compiler
bug, but definitely running the linux-leaks build with `-O2` is
suspicious.
I suppose we could temporarily mark t1300 as not passing with
SANITIZE=leak turned on, but I tend to agree with Peff that that feels
like a hack working around compiler behavior, that will ultimately
result in us playing whack-a-mole.
So my preference would be to run the linux-leaks build with `-O0` in its
CFLAGS, optionally with a newer compiler if one is available for Focal.
Thoughts?
Thanks,
Taylor
next reply other threads:[~2022-10-18 18:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 18:37 Taylor Blau [this message]
2022-10-18 19:40 ` linux-leaks CI failure on master Jeff King
2022-10-18 20:15 ` [PATCH] Makefile: force -O0 when compiling with SANITIZE=leak Jeff King
2022-10-18 21:00 ` Ævar Arnfjörð Bjarmason
2022-10-19 7:10 ` Jeff King
2022-10-20 18:57 ` Rubén Justo
2022-10-21 6:04 ` Jeff King
2022-10-19 15:36 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y07yeEQu+C7AH7oN@nand.local \
--to=me@ttaylorr.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=peff@peff.net \
--cc=vdye@github.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).