aboutsummaryrefslogtreecommitdiff
path: root/src/backend/access/transam/xlogreader.c
diff options
context:
space:
mode:
authorAndres Freund <andres@anarazel.de>2015-01-04 15:35:46 +0100
committerAndres Freund <andres@anarazel.de>2015-01-04 15:35:46 +0100
commit14570c28289f82030172c699ec877dd26d04940a (patch)
tree547ef0dc78fe0cde1730ddb6eee4167f6032968e /src/backend/access/transam/xlogreader.c
parent58bc4747be26f6849ca38e67e24abdd6aecba3b1 (diff)
downloadpostgresql-14570c28289f82030172c699ec877dd26d04940a.tar.gz
postgresql-14570c28289f82030172c699ec877dd26d04940a.zip
Remove superflous variable from xlogreader's XLogFindNextRecord().
Pointed out by Coverity. Since this is mere, and debatable, cosmetics I'm not backpatching this.
Diffstat (limited to 'src/backend/access/transam/xlogreader.c')
-rw-r--r--src/backend/access/transam/xlogreader.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/access/transam/xlogreader.c b/src/backend/access/transam/xlogreader.c
index 67d62234369..db5384c2062 100644
--- a/src/backend/access/transam/xlogreader.c
+++ b/src/backend/access/transam/xlogreader.c
@@ -833,7 +833,6 @@ XLogFindNextRecord(XLogReaderState *state, XLogRecPtr RecPtr)
XLogRecPtr found = InvalidXLogRecPtr;
uint32 pageHeaderSize;
XLogPageHeader header;
- XLogRecord *record;
int readLen;
char *errormsg;
@@ -875,7 +874,7 @@ XLogFindNextRecord(XLogReaderState *state, XLogRecPtr RecPtr)
* because either we're at the first record after the beginning of a page
* or we just jumped over the remaining data of a continuation.
*/
- while ((record = XLogReadRecord(state, tmpRecPtr, &errormsg)))
+ while (XLogReadRecord(state, tmpRecPtr, &errormsg) != NULL)
{
/* continue after the record */
tmpRecPtr = InvalidXLogRecPtr;