Provide a clearer error message if the pg_control version number looks

wrong because of mismatched byte ordering.

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index ade0327fd4fbd411152971d3a8cb96327050cd03..ba8a3b661620a3254ae72312a9a7c125419b2b20 100644 (file)
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -3888,6 +3888,16 @@ ReadControlFile(void)
         * of bytes.  Complaining about wrong version will probably be more
         * enlightening than complaining about wrong CRC.
         */
+
+       if (ControlFile->pg_control_version != PG_CONTROL_VERSION && ControlFile->pg_control_version % 65536 == 0 && ControlFile->pg_control_version / 65536 != 0)
+               ereport(FATAL,
+                               (errmsg("database files are incompatible with server"),
+                                errdetail("The database cluster was initialized with PG_CONTROL_VERSION %d (0x%08x),"
+                                                  " but the server was compiled with PG_CONTROL_VERSION %d (0x%08x).",
+                                                  ControlFile->pg_control_version, ControlFile->pg_control_version,
+                                                  PG_CONTROL_VERSION, PG_CONTROL_VERSION),
+                                errhint("This could be a problem of mismatched byte ordering.  It looks like you need to initdb.")));
+
        if (ControlFile->pg_control_version != PG_CONTROL_VERSION)
                ereport(FATAL,
                                (errmsg("database files are incompatible with server"),
@@ -3895,6 +3905,7 @@ ReadControlFile(void)
                                  " but the server was compiled with PG_CONTROL_VERSION %d.",
                                                ControlFile->pg_control_version, PG_CONTROL_VERSION),
                                 errhint("It looks like you need to initdb.")));
+
        /* Now check the CRC. */
        INIT_CRC32(crc);
        COMP_CRC32(crc,

diff --git a/src/bin/pg_controldata/pg_controldata.c b/src/bin/pg_controldata/pg_controldata.c
index 92652ab426560654bd5399dd0f0fe850c20ce1b0..f466784c500100a32a62d9477ea2c61e40744fcc 100644 (file)
--- a/src/bin/pg_controldata/pg_controldata.c
+++ b/src/bin/pg_controldata/pg_controldata.c
@@ -151,6 +151,11 @@ main(int argc, char *argv[])
 
        printf(_("pg_control version number:            %u\n"),
                   ControlFile.pg_control_version);
+       if (ControlFile.pg_control_version % 65536 == 0 && ControlFile.pg_control_version / 65536 != 0)
+               printf(_("WARNING: possible byte ordering mismatch\n"
+                                "The byte ordering used to store the pg_control file might not match the one\n"
+                                "used by this program.  In that case the results below would be incorrect, and\n"
+                                "the PostgreSQL installation would be incompatible with this data directory.\n"));
        printf(_("Catalog version number:               %u\n"),
                   ControlFile.catalog_version_no);
        printf(_("Database system identifier:           %s\n"),