aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorFujii Masao <fujii@postgresql.org>2014-11-19 19:10:04 +0900
committerFujii Masao <fujii@postgresql.org>2014-11-19 19:10:04 +0900
commitd5f4df7264ec73abda41eb78b81cf430b33f7606 (patch)
treeac3b1ae7d758c0f7ebfbc7d73c73bca0857de96b /src
parentf66c20b317578838a39a1de8014c4363bdc98b9a (diff)
downloadpostgresql-d5f4df7264ec73abda41eb78b81cf430b33f7606.tar.gz
postgresql-d5f4df7264ec73abda41eb78b81cf430b33f7606.zip
Fix bug in the test of file descriptor of current WAL file in pg_receivexlog.
In pg_receivexlog, in order to check whether the current WAL file is being opened or not, its file descriptor has to be checked against -1 as an invalid value. But, oops, 7900e94 added the incorrect test checking the descriptor against 1. This commit fixes that bug. Back-patch to 9.4 where the bug was added. Spotted by Magnus Hagander
Diffstat (limited to 'src')
-rw-r--r--src/bin/pg_basebackup/receivelog.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/bin/pg_basebackup/receivelog.c b/src/bin/pg_basebackup/receivelog.c
index 1b8a5ad15ce..f0f8760e2db 100644
--- a/src/bin/pg_basebackup/receivelog.c
+++ b/src/bin/pg_basebackup/receivelog.c
@@ -1021,7 +1021,7 @@ ProcessKeepaliveMsg(PGconn *conn, char *copybuf, int len,
if (replyRequested && still_sending)
{
if (reportFlushPosition && lastFlushPosition < blockpos &&
- walfile != 1)
+ walfile != -1)
{
/*
* If a valid flush location needs to be reported,