Skip to content

Commit 58b2cf0

Browse files
committed
Fix off-by-one error in Assertion.
The point of the assertion is to ensure that the arrays allocated in stack are large enough, but the check was one item short. This won't matter in practice because MaxIndexTuplesPerPage is an overestimate, so you can't have that many items on a page in reality. But let's be tidy. Spotted by Anastasia Lubennikova. Backpatch to all supported versions, like the patch that added the assertion.
1 parent 97ff2a5 commit 58b2cf0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/backend/storage/page/bufpage.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -722,7 +722,7 @@ PageIndexMultiDelete(Page page, OffsetNumber *itemnos, int nitems)
722722
int nextitm;
723723
OffsetNumber offnum;
724724

725-
Assert(nitems < MaxIndexTuplesPerPage);
725+
Assert(nitems <= MaxIndexTuplesPerPage);
726726

727727
/*
728728
* If there aren't very many items to delete, then retail

0 commit comments

Comments
 (0)