From: Heikki Linnakangas <heikki.linnakangas@iki.fi>
Date: Wed, 28 Nov 2012 09:45:30 +0000 (+0200)
Subject: If we don't have a backup-end-location, don't claim we've reached it.
X-Git-Tag: REL9_2_2~24
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ffc3172e4e3caee0327a7e4126b5e7a3c8a1c8cf;p=postgresql

If we don't have a backup-end-location, don't claim we've reached it.

This was apparently a typo, which caused recovery to think that it
immediately reached the end of backup, and allowed the database to start
up too early.

Reported by Jeff Janes. Backpatch to 9.2, where this code was introduced.
---

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index 3d6046eacf..9328ad38f4 100644
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -6763,7 +6763,7 @@ StartupXLOG(void)
 				/* Pop the error context stack */
 				error_context_stack = errcontext.previous;
 
-				if (!XLogRecPtrIsInvalid(ControlFile->backupStartPoint) &&
+				if (!XLogRecPtrIsInvalid(ControlFile->backupEndPoint) &&
 					XLByteLE(ControlFile->backupEndPoint, EndRecPtr))
 				{
 					/*