Re: [PATCH] t: remove unexpected SANITIZE_LEAK variables

From: Jeff King <peff@peff.net>
Date: May 22, 2025, 06:09 UTC
Message-ID: <20250522060933.GA1135565@coredump.intra.peff.net>
In-Reply-To: <20250522060626.GB1137854@coredump.intra.peff.net>

On Thu, May 22, 2025 at 02:06:26AM -0400, Jeff King wrote:

Show 6 quoted lines

> As a side note, we do still use the SANITIZE_LEAK prereq in a few
> places, and I believe that it does actually work. It might be nice to
> clean up any leaks in those few spots, though we probably want to keep
> the prereq around forever (e.g., if you introduce a test which shows off
> a leak and then fixes it later). All orthogonal to your patch, though,
> which looks good to me.

Ah, nevermind. These are all due to 8415595203 (t5601: work around leak sanitizer issue, 2024-11-20). They are leak-free (and the tests pass on my system), but apparently some upstream bug can cause issues.

-Peff