From: Jeff King Date: Tue, 03 Oct 2017 23:41:54 GMT Subject: Re: [PATCH 0/3] fixes for running the test suite with --valgrind Message-ID: <20171003234153.pmslizidgbifl6en@sigill.intra.peff.net> In-Reply-To: <20171003195713.13395-1-t.gummerer@gmail.com> On Tue, Oct 03, 2017 at 08:57:10PM +0100, Thomas Gummerer wrote: > I recently tried to run the git test suite with --valgrind. > Unfortunately it didn't come back completely clean. This patch series > fixes a bunch of these errors, although unfortunately not quite all of > them yet. Thanks for trying that. I've largely switched to ASan over valgrind because it runs _so_ much faster. Unfortunately it seems to miss these uninitialized-memory cases. I think using SANITIZE=memory would catch these, but it needs some suppressions tuning. The weird "zlib reads uninitialized memory" error is a problem (valgrind sees this, too, but we have suppressions). -Peff