database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-02 11:44:50 +03:00
Code

Fix unobvious interaction between -X switch and subdirectory creation.

Browse Source 
Turns out the only reason initdb -X worked is that pg_mkdir_p won't
whine if you point it at something that's a symlink to a directory.
Otherwise, the attempt to create pg_xlog/ just like all the other
subdirectories would have failed.  Let's be a little more explicit
about what's happening.  Oversight in my patch for bug #13853
(mea culpa for not testing -X ...)
This commit is contained in:
Tom Lane 2016-01-07 18:20:57 -05:00
parent 32e7e7fa77
commit 506ef1d07f
1 changed files with 24 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
src/bin/initdb/initdb.c
View File

@ -189,7 +189,6 @@ static const char *backend_options = "--single -F -O -c search_path=pg_catalog -
static const char *subdirs[] = {
"global",
"pg_xlog",
"pg_xlog/archive_status",
"pg_clog",
"pg_commit_ts",
@ -276,7 +275,7 @@ void setup_locale_encoding(void);
void setup_signals(void);
void setup_text_search(void);
void create_data_directory(void);
void create_xlog_symlink(void);
void create_xlog_or_symlink(void);
void warn_on_mount_point(int error);
void initialize_data_directory(void);
@ -3167,13 +3166,17 @@ create_data_directory(void)
}
/* Create transaction log directory, and symlink if required */
void
create_xlog_symlink(void)
create_xlog_or_symlink(void)
{
/* Create transaction log symlink, if required */
char *subdirloc;
/* form name of the place for the subdirectory or symlink */
subdirloc = psprintf("%s/pg_xlog", pg_data);
if (strcmp(xlog_dir, "") != 0)
{
char *linkloc;
int ret;
/* clean up xlog directory name, check it's absolute */
@ -3246,22 +3249,30 @@ create_xlog_symlink(void)
exit_nicely();
}
/* form name of the place where the symlink must go */
linkloc = psprintf("%s/pg_xlog", pg_data);
#ifdef HAVE_SYMLINK
if (symlink(xlog_dir, linkloc) != 0)
if (symlink(xlog_dir, subdirloc) != 0)
{
fprintf(stderr, _("%s: could not create symbolic link \"%s\": %s\n"),
progname, linkloc, strerror(errno));
progname, subdirloc, strerror(errno));
exit_nicely();
}
#else
fprintf(stderr, _("%s: symlinks are not supported on this platform"));
exit_nicely();
#endif
free(linkloc);
}
else
{
/* Without -X option, just make the subdirectory normally */
if (mkdir(subdirloc, S_IRWXU) < 0)
{
fprintf(stderr, _("%s: could not create directory \"%s\": %s\n"),
progname, subdirloc, strerror(errno));
exit_nicely();
}
}
free(subdirloc);
}
@ -3292,9 +3303,9 @@ initialize_data_directory(void)
create_data_directory();
create_xlog_symlink();
create_xlog_or_symlink();
/* Create required subdirectories */
/* Create required subdirectories (other than pg_xlog) */
printf(_("creating subdirectories ... "));
fflush(stdout);