mirror of
https://github.com/postgres/postgres.git
synced 2025-08-15 14:02:29 +03:00
Add missing SYSTEMQUOTEs
Some popen() calls were missing SYSTEMQUOTEs, which caused initdb and pg_upgrade to fail on Windows, if the installation path contained both spaces and @ signs. Patch by Nikhil Deshpande. Backpatch to all supported versions.
This commit is contained in:
@@ -1503,7 +1503,7 @@ bootstrap_template1(void)
|
||||
unsetenv("PGCLIENTENCODING");
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" --boot -x1 %s %s %s",
|
||||
SYSTEMQUOTE "\"%s\" --boot -x1 %s %s %s" SYSTEMQUOTE,
|
||||
backend_exec,
|
||||
data_checksums ? "-k" : "",
|
||||
boot_options, talkargs);
|
||||
@@ -1544,7 +1544,7 @@ setup_auth(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -1622,7 +1622,7 @@ get_set_pwd(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -1722,7 +1722,7 @@ setup_depend(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -1755,7 +1755,7 @@ setup_sysviews(void)
|
||||
* We use -j here to avoid backslashing stuff in system_views.sql
|
||||
*/
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s -j template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s -j template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -1786,7 +1786,7 @@ setup_description(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -1893,7 +1893,7 @@ setup_collation(void)
|
||||
|
||||
#if defined(HAVE_LOCALE_T) && !defined(WIN32)
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2038,7 +2038,7 @@ setup_conversion(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2076,7 +2076,7 @@ setup_dictionary(void)
|
||||
* We use -j here to avoid backslashing stuff
|
||||
*/
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s -j template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s -j template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2127,7 +2127,7 @@ setup_privileges(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2190,7 +2190,7 @@ setup_schema(void)
|
||||
* We use -j here to avoid backslashing stuff in information_schema.sql
|
||||
*/
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s -j template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s -j template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2207,7 +2207,7 @@ setup_schema(void)
|
||||
PG_CMD_CLOSE;
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2241,7 +2241,7 @@ load_plpgsql(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2266,7 +2266,7 @@ vacuum_db(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2322,7 +2322,7 @@ make_template0(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
@@ -2354,7 +2354,7 @@ make_postgres(void)
|
||||
fflush(stdout);
|
||||
|
||||
snprintf(cmd, sizeof(cmd),
|
||||
"\"%s\" %s template1 >%s",
|
||||
SYSTEMQUOTE "\"%s\" %s template1 >%s" SYSTEMQUOTE,
|
||||
backend_exec, backend_options,
|
||||
DEVNULL);
|
||||
|
||||
|
Reference in New Issue
Block a user