Revert "pg_resetwal: Improve error with wrong/missing data directory"
authorPeter Eisentraut <peter@eisentraut.org>
Fri, 29 Sep 2023 08:59:46 +0000 (10:59 +0200)
committerPeter Eisentraut <peter@eisentraut.org>
Fri, 29 Sep 2023 08:59:46 +0000 (10:59 +0200)
This reverts commit 1d863c250461410e60c9ed5d3180f32336f4c3e2.

This broke specifying the data directory as a relative path.

Reported-by: Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
Discussion: https://www.postgresql.org/message-id/flat/TYAPR01MB58664AD301F511B1EA5B72B4F5C0A%40TYAPR01MB5866.jpnprd01.prod.outlook.com

src/bin/pg_resetwal/pg_resetwal.c

index 47e05bd2c95ff6e9df46d1ca126e008f1a81f0f1..04567f349d67446a4bb1892b058429a7580ad10b 100644 (file)
@@ -345,10 +345,6 @@ main(int argc, char *argv[])
 
    get_restricted_token();
 
-   if (chdir(DataDir) < 0)
-       pg_fatal("could not change directory to \"%s\": %m",
-                DataDir);
-
    /* Set mask based on PGDATA permissions */
    if (!GetDataDirectoryCreatePerm(DataDir))
        pg_fatal("could not read permissions of directory \"%s\": %m",
@@ -356,6 +352,10 @@ main(int argc, char *argv[])
 
    umask(pg_mode_mask);
 
+   if (chdir(DataDir) < 0)
+       pg_fatal("could not change directory to \"%s\": %m",
+                DataDir);
+
    /* Check that data directory matches our server version */
    CheckDataVersion();