NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit720e529

committed

Fix potential integer overflow in bringetbitmap()

This function expects an "int64" as result and stores the number ofpages to add to the index scan bitmap as an "int", multiplying its finalresult by 10. For a relation large enough, this can theoreticallyoverflow if counting more than (INT32_MAX / 10) pages, knowing that thenumber of pages is upper-bounded by MaxBlockNumber.To avoid the overflow, this commit redefines "totalpages", used tocalculate the result, to be an "int64" rather than an "int".Reported-by: Evgeniy GorbanyovAuthor: James HunterDiscussion:https://www.postgresql.org/message-id/07704817-6fa0-460c-b1cf-cd18f7647041@basealt.ruBackpatch-through: 13

1 parentd35ea27 commit720e529Copy full SHA for 720e529

File tree

1 file changed

-1

lines changed

src/backend/access/brin
- brin.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -567,7 +567,7 @@ bringetbitmap(IndexScanDesc scan, TIDBitmap *tbm)`
`567`	`567`	`BrinOpaque*opaque;`
`568`	`568`	`BlockNumbernblocks;`
`569`	`569`	`BlockNumberheapBlk;`
`570`		`-inttotalpages=0;`
	`570`	`+int64totalpages=0;`
`571`	`571`	`FmgrInfo*consistentFn;`
`572`	`572`	`MemoryContextoldcxt;`
`573`	`573`	`MemoryContextperRangeCxt;`

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!

Commit720e529

File tree

1 file changed

1 file changed

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

0 commit comments