Thread (18 messages) 18 messages, 6 authors, 1d ago

Re: [PATCH v2 2/2] packfile: fix corruption due to stale delta base cache entries

flat view

From: Junio C Hamano <hidden>
Date: 2026-10-06 19:57:44

Patrick Steinhardt [off-list ref] writes:
Note that the added test reliably reproduces the above bug on my machine
that uses NixOS at c59305bab206 (cosmic-applets: add missing runtime
dependency (#566040), 2026-10-01) with glibc 2.44-25. But as we rely on
specific allocation behaviour of glibc it is very likely that the test
will not work on other platforms.
In other words, the test will not detect the bug, when the fix is
reverted, unless the glibc allocator is used?

Adding an unreliable reproducer for a bug that is already fixed may
be of dubious value.  However, even if the test is unreliable (since
other allocators might hide the bug when the fix is reverted), it
may be OK as long as it catches the bug on widely used
configurations and does not trigger false positives.  On the other
hand, the earlier suggestion to write custom low-level code to
simulate a colliding allocation address somehow smells like a
maintenance burden to me.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help