Add missing "goto err" statements in xlogreader.c.

Spotted by Andres Freund.

diff --git a/src/backend/access/transam/xlogreader.c b/src/backend/access/transam/xlogreader.c
index 60470b56af7264810d49e3ca5a8cdbbaef6f1d75..5bd07e381d8f7e0fb000bb510947e137bd8105f1 100644 (file)
--- a/src/backend/access/transam/xlogreader.c
+++ b/src/backend/access/transam/xlogreader.c
@@ -1019,14 +1019,20 @@ DecodeXLogRecord(XLogReaderState *state, XLogRecord *record, char **errormsg)
            COPY_HEADER_FIELD(&blk->data_len, sizeof(uint16));
            /* cross-check that the HAS_DATA flag is set iff data_length > 0 */
            if (blk->has_data && blk->data_len == 0)
+           {
                report_invalid_record(state,
                      "BKPBLOCK_HAS_DATA set, but no data included at %X/%X",
                                      (uint32) (state->ReadRecPtr >> 32), (uint32) state->ReadRecPtr);
+               goto err;
+           }
            if (!blk->has_data && blk->data_len != 0)
+           {
                report_invalid_record(state,
                 "BKPBLOCK_HAS_DATA not set, but data length is %u at %X/%X",
                                      (unsigned int) blk->data_len,
                                      (uint32) (state->ReadRecPtr >> 32), (uint32) state->ReadRecPtr);
+               goto err;
+           }
            datatotal += blk->data_len;
 
            if (blk->has_image)