]> git.ipfire.org Git - thirdparty/postgresql.git/commitdiff
pg_resetwal: Fix incorrect error message related to pg_wal/summaries/
authorMichael Paquier <michael@paquier.xyz>
Wed, 4 Feb 2026 07:38:10 +0000 (16:38 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 4 Feb 2026 07:38:10 +0000 (16:38 +0900)
A failure while closing pg_wal/summaries/ incorrectly generated a report
about pg_wal/archive_status/.

While at it, this commit adds #undefs for the macros used in
KillExistingWALSummaries() and KillExistingArchiveStatus() to prevent
those values from being misused in an incorrect function context.

Oversight in dc212340058b.

Author: Tianchen Zhang <zhang_tian_chen@163.com>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Reviewed-by: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Discussion: https://postgr.es/m/SE2P216MB2390C84C23F428A7864EE07FA19BA@SE2P216MB2390.KORP216.PROD.OUTLOOK.COM
Backpatch-through: 17

src/bin/pg_resetwal/pg_resetwal.c

index 3cbf898d031387850f043f8452bd94409bc89bbc..efb515e8a1ac2ecb2b05438410f50e07cf447f5b 100644 (file)
@@ -1067,6 +1067,8 @@ KillExistingArchiveStatus(void)
 
        if (closedir(xldir))
                pg_fatal("could not close directory \"%s\": %m", ARCHSTATDIR);
+
+#undef ARCHSTATDIR
 }
 
 /*
@@ -1101,7 +1103,10 @@ KillExistingWALSummaries(void)
                pg_fatal("could not read directory \"%s\": %m", WALSUMMARYDIR);
 
        if (closedir(xldir))
-               pg_fatal("could not close directory \"%s\": %m", ARCHSTATDIR);
+               pg_fatal("could not close directory \"%s\": %m", WALSUMMARYDIR);
+
+#undef WALSUMMARY_NHEXCHARS
+#undef WALSUMMARYDIR
 }
 
 /*