NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2c2c7bc

committed

Fix off-by-one error in Assertion.

The point of the assertion is to ensure that the arrays allocated in stackare large enough, but the check was one item short.This won't matter in practice because MaxIndexTuplesPerPage is anoverestimate, 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, likethe patch that added the assertion.

1 parent2360eea commit2c2c7bcCopy full SHA for 2c2c7bc

File tree

1 file changed

-1

lines changed

src/backend/storage/page
- bufpage.c

1 file changed

-1

lines changed

`‎src/backend/storage/page/bufpage.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -723,7 +723,7 @@ PageIndexMultiDelete(Page page, OffsetNumber *itemnos, int nitems)`
`723`	`723`	`intnextitm;`
`724`	`724`	`OffsetNumberoffnum;`
`725`	`725`
`726`		`-Assert(nitems<MaxIndexTuplesPerPage);`
	`726`	`+Assert(nitems <=MaxIndexTuplesPerPage);`
`727`	`727`
`728`	`728`	`/*`
`729`	`729`	`* If there aren't very many items to delete, then retail`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit2c2c7bc

File tree

1 file changed

1 file changed

`‎src/backend/storage/page/bufpage.c`

0 commit comments