From 14570c28289f82030172c699ec877dd26d04940a Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Sun, 4 Jan 2015 15:35:46 +0100 Subject: [PATCH] Remove superflous variable from xlogreader's XLogFindNextRecord(). Pointed out by Coverity. Since this is mere, and debatable, cosmetics I'm not backpatching this. --- src/backend/access/transam/xlogreader.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/backend/access/transam/xlogreader.c b/src/backend/access/transam/xlogreader.c index 67d6223436..db5384c206 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; -- 2.40.0