Hi David, The patch looks right to me. All three CHUNKHDRSZ macros are sizeof(MemoryChunk), so the checked byte is the same.
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? Thanks, Shihao
