Fix pg_rewind debug output to print the source timeline history

hlinnaka · hlinnaka · commit b5a5027c9796 · 2024-08-08T10:22:45.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
@@ -837,6 +837,7 @@ getTimelineHistory(ControlFileData *controlFile, int *nentries)
 		pg_free(histfile);
 	}
 
+	/* In debugging mode, print what we read */
 	if (debug)
 	{
 		int			i;
@@ -848,10 +849,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
`@@ -837,6 +837,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`837`	`837`	`pg_free(histfile);`
`838`	`838`	`}`
`839`	`839`
	`840`	`+ /* In debugging mode, print what we read */`
`840`	`841`	`if (debug)`
`841`	`842`	`{`
`842`	`843`	`int i;`
`@@ -848,10 +849,7 @@ getTimelineHistory(ControlFileData controlFile, int nentries)`
`848`	`849`	`else`
`849`	`850`	`Assert(false);`
`850`	`851`
`851`		`- /*`
`852`		`- * Print the target timeline history.`
`853`		`- */`
`854`		`- for (i = 0; i < targetNentries; i++)`
	`852`	`+ for (i = 0; i < *nentries; i++)`
`855`	`853`	`{`
`856`	`854`	`TimeLineHistoryEntry *entry;`
`857`	`855`