NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf2f9e11

committed

Reject missing database name in pg_regress and cohorts.

Writing "pg_regress --dbname= ..." led to a crash, becausewe weren't expecting there to be no database name supplied.It doesn't seem like a great idea to run regression testsin whatever is the user's default database; so rather thansupporting this case let's explicitly reject it.Per report from Xing Guo. Back-patch to all supportedbranches.Discussion:https://postgr.es/m/CACpMh+A8cRvtvtOWVAZsCM1DU81GK4DL26R83y6ugZ1osV=ifA@mail.gmail.com

1 parent8592b56 commitf2f9e11Copy full SHA for f2f9e11

File tree

1 file changed

+11

-0

lines changed

src/test/regress
- pg_regress.c

1 file changed

+11

-0

lines changed

`‎src/test/regress/pg_regress.c`

Lines changed: 11 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2198,6 +2198,17 @@ regression_main(int argc, char *argv[],`
`2198`	`2198`	`optind++;`
`2199`	`2199`	`}`
`2200`	`2200`
	`2201`	`+/*`
	`2202`	`+ * We must have a database to run the tests in; either a default name, or`
	`2203`	`+ * one supplied by the --dbname switch.`
	`2204`	`+ */`
	`2205`	`+if (!(dblist&&dblist->str&&dblist->str[0]))`
	`2206`	`+{`
	`2207`	`+fprintf(stderr,_("%s: no database name was specified\n"),`
	`2208`	`+progname);`
	`2209`	`+exit(2);`
	`2210`	`+}`
	`2211`	`+`
`2201`	`2212`	`if (config_auth_datadir)`
`2202`	`2213`	`{`
`2203`	`2214`	`#ifdefENABLE_SSPI`

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!

Commitf2f9e11

File tree

1 file changed

1 file changed

`‎src/test/regress/pg_regress.c`

0 commit comments