NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc2b68b1

committed

initdb -S should now have an explicit check that $PGDATA is valid.

The fsync code from the backend essentially assumes that somebody's alreadyvalidated PGDATA, at least to the extent of it being a readable directory.That's safe enough for initdb's normal code path too, but "initdb -S"doesn't have any other processing at all that touches the target directory.To have reasonable error-case behavior, add a pg_check_dir call.Per gripe from Peter E.

1 parent35dd1b5 commitc2b68b1Copy full SHA for c2b68b1

File tree

1 file changed

+10

-1

lines changed

src/bin/initdb
- initdb.c

1 file changed

+10

-1

lines changed

`‎src/bin/initdb/initdb.c‎`

Lines changed: 10 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3650,10 +3650,19 @@ main(int argc, char *argv[])`
`3650`	`3650`	`exit(1);`
`3651`	`3651`	`}`
`3652`	`3652`
`3653`		`-/* If we only need to fsync, justto it and exit */`
	`3653`	`+/* If we only need to fsync, justdo it and exit */`
`3654`	`3654`	`if (sync_only)`
`3655`	`3655`	`{`
`3656`	`3656`	`setup_pgdata();`
	`3657`	`+`
	`3658`	`+/* must check that directory is readable */`
	`3659`	`+if (pg_check_dir(pg_data) <=0)`
	`3660`	`+{`
	`3661`	`+fprintf(stderr,_("%s: could not access directory \"%s\": %s\n"),`
	`3662`	`+progname,pg_data,strerror(errno));`
	`3663`	`+exit_nicely();`
	`3664`	`+}`
	`3665`	`+`
`3657`	`3666`	`fsync_pgdata();`
`3658`	`3667`	`return0;`
`3659`	`3668`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc2b68b1

File tree

1 file changed

1 file changed

`‎src/bin/initdb/initdb.c‎`

0 commit comments