NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2de3c10

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 parentece8c76 commit2de3c10Copy full SHA for 2de3c10

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(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!

Commit2de3c10

File tree

1 file changed

1 file changed

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

0 commit comments