Fix pg_rewind debug output to print the source timeline history

hlinnaka · hlinnaka · commit bb5592cacede · 2024-08-08T10:22:39.000+03:00
getTimelineHistory() is called twice, to read the source and the target timeline history files. However, the loop to print the file with the --debug option used the wrong variable when dealing with the source. As a result, the source's history was always printed as empty. Spotted while debugging bug #18575, but this does not fix that bug, just the debugging output. Backpatch to all supported versions. Discussion: https://www.postgresql.org/message-id/092dd515-b7b4-4fd0-8407-ceca2f02f6ec@iki.fi
diff --git a/src/bin/pg_rewind/pg_rewind.c b/src/bin/pg_rewind/pg_rewind.c
@@ -838,6 +838,7 @@ getTimelineHistory(ControlFileData *controlFile, int *nentries)
 		pg_free(histfile);
 	}
 
+	/* In debugging mode, print what we read */
 	if (debug)
 	{
 		int			i;
@@ -849,10 +850,7 @@ getTimelineHistory(ControlFileData *controlFile, int *nentries)
 		else
 			Assert(false);
 
-		/*
-		 * Print the target timeline history.
-		 */
-		for (i = 0; i < targetNentries; i++)
+		for (i = 0; i < *nentries; i++)
 		{
 			TimeLineHistoryEntry *entry;
 

Original file line number	Diff line number	Diff line change
`@@ -838,6 +838,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`838`	`838`	`pg_free(histfile);`
`839`	`839`	`}`
`840`	`840`
	`841`	`+ /* In debugging mode, print what we read */`
`841`	`842`	`if (debug)`
`842`	`843`	`{`
`843`	`844`	`int i;`
`@@ -849,10 +850,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`849`	`850`	`else`
`850`	`851`	`Assert(false);`
`851`	`852`
`852`		`- /*`
`853`		`- * Print the target timeline history.`
`854`		`- */`
`855`		`- for (i = 0; i < targetNentries; i++)`
	`853`	`+ for (i = 0; i < *nentries; i++)`
`856`	`854`	`{`
`857`	`855`	`TimeLineHistoryEntry *entry;`
`858`	`856`