Remove sanity test in XRecOffIsValid.
authorRobert Haas <rhaas@postgresql.org>
Mon, 25 Jun 2012 16:14:43 +0000 (12:14 -0400)
committerRobert Haas <rhaas@postgresql.org>
Mon, 25 Jun 2012 16:14:43 +0000 (12:14 -0400)
Commit 061e7efb1b4c5b8a5d02122b7780531b8d5bf23d changed the rules
for splitting xlog records across pages, but neglected to update this
test.  It's possible that there's some better action here than just
removing the test completely, but this at least appears to get some
of the things that are currently broken (like initdb on MacOS X)
working again.

src/include/access/xlog_internal.h

index a74ff6526729894bfee42bebe81bfe9fa2bfec24..b5bfb7b407282373a7fbe162fcb91733700cf921 100644 (file)
@@ -154,8 +154,7 @@ typedef XLogLongPageHeaderData *XLogLongPageHeader;
 
 /* Check if an XLogRecPtr value is in a plausible range */
 #define XRecOffIsValid(xlrp) \
-       ((xlrp) % XLOG_BLCKSZ >= SizeOfXLogShortPHD && \
-        (XLOG_BLCKSZ - (xlrp) % XLOG_BLCKSZ) >= SizeOfXLogRecord)
+       ((xlrp) % XLOG_BLCKSZ >= SizeOfXLogShortPHD)
 
 /*
  * The XLog directory and control file (relative to $PGDATA)