NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit65c384c

committed

Avoid calling PageGetSpecialPointer() on an all-zeros page.

That was otherwise harmless, but tripped the new assertion inPageGetSpecialPointer().Reported by Amit Langote. Backpatch to 9.5, where the assertion was added.

1 parente3a9a19 commit65c384cCopy full SHA for 65c384c

File tree

1 file changed

-2

lines changed

src/backend/access/nbtree
- nbtree.c

1 file changed

-2

lines changed

`‎src/backend/access/nbtree/nbtree.c`

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -881,7 +881,7 @@ btvacuumpage(BTVacState *vstate, BlockNumber blkno, BlockNumber orig_blkno)`
`881`	`881`	`BlockNumberrecurse_to;`
`882`	`882`	`Bufferbuf;`
`883`	`883`	`Pagepage;`
`884`		`-BTPageOpaqueopaque;`
	`884`	`+BTPageOpaqueopaque=NULL;`
`885`	`885`
`886`	`886`	`restart:`
`887`	`887`	`delete_now= false;`
`@@ -900,9 +900,11 @@ btvacuumpage(BTVacState *vstate, BlockNumber blkno, BlockNumber orig_blkno)`
`900`	`900`	`info->strategy);`
`901`	`901`	`LockBuffer(buf,BT_READ);`
`902`	`902`	`page=BufferGetPage(buf);`
`903`		`-opaque= (BTPageOpaque)PageGetSpecialPointer(page);`
`904`	`903`	`if (!PageIsNew(page))`
	`904`	`+{`
`905`	`905`	`_bt_checkpage(rel,buf);`
	`906`	`+opaque= (BTPageOpaque)PageGetSpecialPointer(page);`
	`907`	`+}`
`906`	`908`
`907`	`909`	`/*`
`908`	`910`	`* If we are recursing, the only case we want to do anything with is a`

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!

Commit65c384c

File tree

1 file changed

1 file changed

`‎src/backend/access/nbtree/nbtree.c`

0 commit comments