diff options
author | Andres Freund <andres@anarazel.de> | 2015-01-04 15:35:46 +0100 |
---|---|---|
committer | Andres Freund <andres@anarazel.de> | 2015-01-04 15:35:46 +0100 |
commit | 14570c28289f82030172c699ec877dd26d04940a (patch) | |
tree | 547ef0dc78fe0cde1730ddb6eee4167f6032968e /src/backend/access/transam/xlogreader.c | |
parent | 58bc4747be26f6849ca38e67e24abdd6aecba3b1 (diff) | |
download | postgresql-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.c | 3 |
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; |