NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit0a56820

committed

Fix datatype confusion in logtape.c's right_offset().

This could only matter if (a) long is wider than int, and (b) the heapof free blocks exceeds UINT_MAX entries, which seems pretty unlikely.Still, it's a theoretical bug, so backpatch to v13 where the typo camein (in commitc02fdc9).In passing, also make swap_nodes() use consistent datatypes.Ma LiangzhuDiscussion:https://postgr.es/m/17336-fc4e522d26a750fd@postgresql.org

1 parent3f710fc commit0a56820Copy full SHA for 0a56820

File tree

1 file changed

-2

lines changed

src/backend/utils/sort
- logtape.c

1 file changed

-2

lines changed

`‎src/backend/utils/sort/logtape.c‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -343,7 +343,7 @@ ltsReadFillBuffer(LogicalTapeSet lts, LogicalTape lt)`
`343`	`343`	`staticinlinevoid`
`344`	`344`	`swap_nodes(long*heap,unsigned longa,unsigned longb)`
`345`	`345`	`{`
`346`		`-unsignedlongswap;`
	`346`	`+longswap;`
`347`	`347`
`348`	`348`	`swap=heap[a];`
`349`	`349`	`heap[a]=heap[b];`
`@@ -357,7 +357,7 @@ left_offset(unsigned long i)`
`357`	`357`	`}`
`358`	`358`
`359`	`359`	`staticinlineunsigned long`
`360`		`-right_offset(unsignedi)`
	`360`	`+right_offset(unsignedlongi)`
`361`	`361`	`{`
`362`	`362`	`return2*i+2;`
`363`	`363`	`}`

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!

Commit0a56820

File tree

1 file changed

1 file changed

`‎src/backend/utils/sort/logtape.c‎`

0 commit comments