Fix MSVC pg_upgrade test.
authorNathan Bossart <nathan@postgresql.org>
Mon, 11 Aug 2025 17:37:00 +0000 (12:37 -0500)
committerNathan Bossart <nathan@postgresql.org>
Mon, 11 Aug 2025 17:37:00 +0000 (12:37 -0500)
In the back-branch versions of commit 71ea0d6795, I missed updating
this test to use --restrict-key so that the generated dump output
is comparable.

Per buildfarm member hamerkop.

Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Security: CVE-2025-8714
Backpatch-through: 13-14

src/tools/msvc/vcregress.pl

index 44f5b27900e8d6a0daf0622af65d9c64b1715b43..57b1a0c22134b8df98d294bca9ffa3c2144cc70c 100644 (file)
@@ -675,7 +675,7 @@ sub upgradecheck
    # now we can chdir into the source dir
    chdir "$topdir/src/bin/pg_upgrade";
    print "\nDumping old cluster\n\n";
-   @args = ('pg_dumpall', '-f', "$tmp_root/dump1.sql");
+   @args = ('pg_dumpall', '-f', "$tmp_root/dump1.sql", '--restrict-key', 'test');
    system(@args) == 0 or exit 1;
    print "\nStopping old cluster\n\n";
    system("pg_ctl stop") == 0 or exit 1;
@@ -689,7 +689,7 @@ sub upgradecheck
    @args = ('pg_ctl', '-l', "$logdir/postmaster2.log", 'start');
    system(@args) == 0 or exit 1;
    print "\nDumping new cluster\n\n";
-   @args = ('pg_dumpall', '-f', "$tmp_root/dump2.sql");
+   @args = ('pg_dumpall', '-f', "$tmp_root/dump2.sql", '--restrict-key', 'test');
    system(@args) == 0 or exit 1;
    print "\nStopping new cluster\n\n";
    system("pg_ctl stop") == 0 or exit 1;