NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd5515ca

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 parent8f2e74b commitd5515caCopy full SHA for d5515ca

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
`@@ -2182,6 +2182,17 @@ regression_main(int argc, char *argv[],`
`2182`	`2182`	`optind++;`
`2183`	`2183`	`}`
`2184`	`2184`
	`2185`	`+/*`
	`2186`	`+ * We must have a database to run the tests in; either a default name, or`
	`2187`	`+ * one supplied by the --dbname switch.`
	`2188`	`+ */`
	`2189`	`+if (!(dblist&&dblist->str&&dblist->str[0]))`
	`2190`	`+{`
	`2191`	`+fprintf(stderr,_("%s: no database name was specified\n"),`
	`2192`	`+progname);`
	`2193`	`+exit(2);`
	`2194`	`+}`
	`2195`	`+`
`2185`	`2196`	`if (config_auth_datadir)`
`2186`	`2197`	`{`
`2187`	`2198`	`#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!

Commitd5515ca

File tree

1 file changed

1 file changed

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

0 commit comments