test: OpenFileGDB: verify that we can generate an output that is byte…

…-identical to the expected golden file.

autotest/ogr/data/openfilegdb/polygon_golden.gdb/timestamps

@@ -0,0 +1 @@
+����������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������������

autotest/ogr/ogr_openfilegdb_write.py

@@ -13,6 +13,7 @@
 # SPDX-License-Identifier: MIT
 ###############################################################################
 
+import os
 import struct
 import sys
 
@@ -4571,3 +4572,32 @@ def test_ogr_openfilegdb_write_OGRUnsetMarker(tmp_vsimem):
         lyr = ds.GetLayer(0)
         f = lyr.GetNextFeature()
         assert f["i32"] == -21121
+
+
+###############################################################################
+# Verify that we can generate an output that is byte-identical to the expected golden file.
+
+
+@pytest.mark.parametrize(
+    "src_directory",
+    [
+        # Generated with:
+        # ogr2ogr autotest/ogr/data/openfilegdb/polygon_golden.gdb '{"type":"Feature","properties":{"foo":"bar"},"geometry":{"type":"Polygon","coordinates":[[[0,0],[0,1],[1,0],[0,0]]]}}' --config OPENFILEGDB_CREATOR GDAL --config OPENFILEGDB_REPRODUCIBLE_UUID YES -f openfilegdb
+        "data/openfilegdb/polygon_golden.gdb",
+    ],
+)
+def test_ogr_openfilegdb_write_check_golden_file(tmp_path, src_directory):
+
+    out_directory = str(tmp_path / "test.gdb")
+    with gdaltest.config_options(
+        {"OPENFILEGDB_CREATOR": "GDAL", "OPENFILEGDB_REPRODUCIBLE_UUID": "YES"}
+    ):
+        gdal.VectorTranslate(out_directory, src_directory, format="OpenFileGDB")
+    for filename in os.listdir(src_directory):
+        src_filename = os.path.join(src_directory, filename)
+        out_filename = os.path.join(out_directory, filename)
+
+        assert os.stat(src_filename).st_size == os.stat(out_filename).st_size, filename
+        assert (
+            open(src_filename, "rb").read() == open(out_filename, "rb").read()
+        ), filename

ogr/ogrsf_frmts/openfilegdb/ogr_openfilegdb.h

@@ -28,7 +28,7 @@
 
 using namespace OpenFileGDB;
 
-std::string OFGDBGenerateUUID();
+std::string OFGDBGenerateUUID(bool bInit = false);
 
 int OGROpenFileGDBIsComparisonOp(int op);
 

ogr/ogrsf_frmts/openfilegdb/ogropenfilegdb_generate_uuid.cpp

@@ -54,65 +54,90 @@ static int CPLGettimeofday(struct CPLTimeVal *tp, void * /* timezonep*/)
 // Probably not the best UUID generator ever. One issue is that mt19937
 // uses only a 32-bit seed.
 CPL_NOSANITIZE_UNSIGNED_INT_OVERFLOW
-std::string OFGDBGenerateUUID()
+std::string OFGDBGenerateUUID(bool bInit)
 {
     struct CPLTimeVal tv;
     memset(&tv, 0, sizeof(tv));
     static uint32_t nCounter = 0;
     const bool bReproducibleUUID =
         CPLTestBool(CPLGetConfigOption("OPENFILEGDB_REPRODUCIBLE_UUID", "NO"));
 
+    if (bInit)
+    {
+        if (bReproducibleUUID)
+            nCounter = 0;
+        return std::string();
+    }
+
+    uint32_t nCounterLocal = nCounter;
+    // From POSIX.1-2001 as an example of an implementation of rand()
+    // for reproducible output.
+    // We have to use that rather than relying on std::mt19937 +
+    // std::uniform_int_distribution since they don't given the same output
+    // from the same seed on all platforms.
+    const auto reproducibleRand = [&nCounterLocal]()
+    {
+        nCounterLocal = nCounterLocal * 1103515245U + 12345U;
+        return (nCounterLocal / 65536U) % 32768U;
+    };
+
     std::stringstream ss;
 
     {
         if (!bReproducibleUUID)
+        {
             CPLGettimeofday(&tv, nullptr);
-        std::mt19937 gen(++nCounter +
-                         (bReproducibleUUID
-                              ? 0
-                              : static_cast<unsigned>(tv.tv_sec ^ tv.tv_usec)));
+            ++nCounter;
+        }
+        std::mt19937 gen(nCounter +
+                         static_cast<unsigned>(tv.tv_sec ^ tv.tv_usec));
         std::uniform_int_distribution<> dis(0, 15);
 
         ss << "{";
         ss << std::hex;
         for (int i = 0; i < 8; i++)
         {
-            ss << dis(gen);
+            ss << (bReproducibleUUID ? (reproducibleRand() % 16) : dis(gen));
         }
         ss << "-";
         for (int i = 0; i < 4; i++)
         {
-            ss << dis(gen);
+            ss << (bReproducibleUUID ? (reproducibleRand() % 16) : dis(gen));
         }
         ss << "-4";
         for (int i = 0; i < 3; i++)
         {
-            ss << dis(gen);
+            ss << (bReproducibleUUID ? (reproducibleRand() % 16) : dis(gen));
         }
     }
 
     {
         if (!bReproducibleUUID)
+        {
             CPLGettimeofday(&tv, nullptr);
-        std::mt19937 gen(++nCounter +
-                         (bReproducibleUUID
-                              ? 0
-                              : static_cast<unsigned>(tv.tv_sec ^ tv.tv_usec)));
+            ++nCounter;
+        }
+        std::mt19937 gen(nCounter +
+                         static_cast<unsigned>(tv.tv_sec ^ tv.tv_usec));
         std::uniform_int_distribution<> dis(0, 15);
         std::uniform_int_distribution<> dis2(8, 11);
 
         ss << "-";
-        ss << dis2(gen);
+        ss << (bReproducibleUUID ? 8 : dis2(gen));
         for (int i = 0; i < 3; i++)
         {
-            ss << dis(gen);
+            ss << (bReproducibleUUID ? (reproducibleRand() % 16) : dis(gen));
         }
         ss << "-";
         for (int i = 0; i < 12; i++)
         {
-            ss << dis(gen);
+            ss << (bReproducibleUUID ? (reproducibleRand() % 16) : dis(gen));
         };
         ss << "}";
-        return ss.str();
     }
+
+    if (bReproducibleUUID)
+        nCounter = nCounterLocal;
+
+    return ss.str();
 }

ogr/ogrsf_frmts/openfilegdb/ogropenfilegdbdatasource_write.cpp

@@ -1299,6 +1299,8 @@ bool OGROpenFileGDBDataSource::Create(const char *pszName)
         return false;
     }
 
+    CPL_IGNORE_RET_VAL(OFGDBGenerateUUID(/* bInit = */ true));
+
     m_osDirName = pszName;
     eAccess = GA_Update;