diff options
| author | Itagaki Takahiro <itagaki.takahiro@gmail.com> | 2010-02-19 01:09:02 +0000 |
|---|---|---|
| committer | Itagaki Takahiro <itagaki.takahiro@gmail.com> | 2010-02-19 01:09:02 +0000 |
| commit | 739898dc3b300025ebc8d352648e191db5c9b82c (patch) | |
| tree | 3cada1e531bd8082f6b47302f4a4e008ad0e9275 | |
| parent | 7925085e61897f2d966f38b0db4a2c9a0024f266 (diff) | |
| download | postgresql-739898dc3b300025ebc8d352648e191db5c9b82c.tar.gz postgresql-739898dc3b300025ebc8d352648e191db5c9b82c.zip | |
Fix STOP WAL LOCATION in backup history files no to return the next
segment of XLOG_BACKUP_END record even if the the record is placed
at a segment boundary. Furthermore the previous implementation could
return nonexistent segment file name when the boundary is in segments
that has "FE" suffix; We never use segments with "FF" suffix.
Backpatch to 8.0, where hot backup was introduced.
Reported by Fujii Masao.
| -rw-r--r-- | src/backend/access/transam/xlog.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c index 342d37311ee..666f74ffb7c 100644 --- a/src/backend/access/transam/xlog.c +++ b/src/backend/access/transam/xlog.c @@ -7,7 +7,7 @@ * Portions Copyright (c) 1996-2005, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.180.4.7 2008/05/13 20:53:56 mha Exp $ + * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.180.4.8 2010/02/19 01:09:02 itagaki Exp $ * *------------------------------------------------------------------------- */ @@ -5479,7 +5479,7 @@ pg_stop_backup(PG_FUNCTION_ARGS) INSERT_RECPTR(stoppoint, Insert, Insert->curridx); LWLockRelease(WALInsertLock); - XLByteToSeg(stoppoint, _logId, _logSeg); + XLByteToPrevSeg(stoppoint, _logId, _logSeg); XLogFileName(stopxlogfilename, ThisTimeLineID, _logId, _logSeg); /* |