On Fri, 16 Oct 2026 at 16:00, shihao zhong <[email protected]> wrote: > The patch looks right to me. All three CHUNKHDRSZ macros are > sizeof(MemoryChunk), so the checked byte is the same.
Thanks for checking. > While looking at the other callers I noticed bump.c sets the sentinel > and requested_size in BumpAllocLarge() and BumpAllocChunkFromBlock(), > but nothing ever reads them. Bump has no pfree, so BumpCheck() is the > only place that could check it, and it doesn't. Is that on purpose, or > should BumpCheck() call sentinel_ok() like the others do? It should be checked. It's quite a bad oversight. I've pushed a fix back to v17 to resolve that. David
