On Thu, 30 Oct 2025 at 13:24, Tom Lane <[email protected]> wrote: > > The reason I left it an unsigned type was that the check is doing: if > > (chsize + ALLOC_CHUNKHDRSZ != blk_used), so we'd still catch this with > > an unsigned type, even if it wrapped due to going negative due to a > > bogus freeptr. Changing to a signed type would leave us with a few > > tests comparing signed to unsigned types. > > Yeah, that's fair. I'm content with what you have.
Thanks for the reviews. I've pushed and backpatched to 13. David
