On Mon, Mar 16, 2015 at 1:38 AM, Andres Freund <[email protected]> wrote:
> Merge the various forms of transaction commit & abort records.

Coverity is complaining about the following block of code:
+       xact_info = XLogRecGetInfo(record) & XLOG_XACT_COMMIT;
+
+       if (xact_info != XLOG_XACT_COMMIT &&
+               xact_info != XLOG_XACT_COMMIT_PREPARED)
                return false;

Instead of XLOG_XACT_COMMIT, shouldn't this function use XLOG_XACT_OPMASK?
-- 
Michael


-- 
Sent via pgsql-committers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

Reply via email to