Thread (5 messages) 5 messages, 3 authors, 2025-05-22

Re: [PATCH] t: remove unexpected SANITIZE_LEAK variables

From: Jeff King <hidden>
Date: 2025-05-22 06:09:34

On Thu, May 22, 2025 at 02:06:26AM -0400, Jeff King wrote:
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help