Fix archive backup mode due to incorrect timeline assigned

Michael Paquier · Michael Paquier · commit 26b8870665d0 · 2013-12-25T04:36:07.000+09:00
This bug has been introduced by some older code, it looks that it will be
necessary to re-create a battery of regression tests to test all those
things automtically, as former tests contain nothing to test archive
mode directly.
diff --git a/backup.c b/backup.c
@@ -1002,14 +1002,12 @@ wait_for_archive(pgBackup *backup, const char *sql)
 	 * Enforce TLI obtention if backup is not present as this code
 	 * path can be taken as a callback at exit.
 	 */
-	if (backup != NULL)
-		tli = backup->tli;
-	else
-		tli = get_current_timeline();
+	tli = get_current_timeline();
 
 	/* Fill in fields if backup exists */
 	if (backup != NULL)
 	{
+		backup->tli = tli;
 		backup->stop_lsn = lsn;
 		elog(LOG, _("%s(): tli=%X lsn=%X/%08X"),
 			 __FUNCTION__, backup->tli,
