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

tglsfdc · tglsfdc · commit 25c7faa1feee · 2021-12-14T11:46:44.000-05:00
This could only matter if (a) long is wider than int, and (b) the heap of free blocks exceeds UINT_MAX entries, which seems pretty unlikely. Still, it's a theoretical bug, so backpatch to v13 where the typo came in (in commit c02fdc9). In passing, also make swap_nodes() use consistent datatypes. Ma Liangzhu Discussion: https://postgr.es/m/17336-fc4e522d26a750fd@postgresql.org
diff --git a/src/backend/utils/sort/logtape.c b/src/backend/utils/sort/logtape.c
@@ -345,7 +345,7 @@ ltsReadFillBuffer(LogicalTapeSet *lts, LogicalTape *lt)
 static inline void
 swap_nodes(long *heap, unsigned long a, unsigned long b)
 {
-	unsigned long swap;
+	long		swap;
 
 	swap = heap[a];
 	heap[a] = heap[b];
@@ -359,7 +359,7 @@ left_offset(unsigned long i)
 }
 
 static inline unsigned long
-right_offset(unsigned i)
+right_offset(unsigned long i)
 {
 	return 2 * i + 2;
 }

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ ltsReadFillBuffer(LogicalTapeSet lts, LogicalTape lt)`
`345`	`345`	`static inline void`
`346`	`346`	`swap_nodes(long *heap, unsigned long a, unsigned long b)`
`347`	`347`	`{`
`348`		`- unsigned long swap;`
	`348`	`+ long swap;`
`349`	`349`
`350`	`350`	`swap = heap[a];`
`351`	`351`	`heap[a] = heap[b];`
`@@ -359,7 +359,7 @@ left_offset(unsigned long i)`
`359`	`359`	`}`
`360`	`360`
`361`	`361`	`static inline unsigned long`
`362`		`-right_offset(unsigned i)`
	`362`	`+right_offset(unsigned long i)`
`363`	`363`	`{`
`364`	`364`	`return 2 * i + 2;`
`365`	`365`	`}`