Report the true database name on connection errors

alvherre · alvherre · commit f2dc962360b3 · 2021-01-26T16:42:13.000-03:00
When reporting connection errors, we might show a database name in the message that's not the one we actually tried to connect to, if the database was taken from libpq defaults instead of from user parameters. Fix such error messages to use PQdb(), which reports the correct name. (But, per commit 2930c05, make sure not to try to print NULL.) Apply to branches 9.5 through 13. Branch master has already been changed differently by commit 58cd8dc. Reported-by: Robert Haas <robertmhaas@gmail.com> Discussion: https://postgr.es/m/CA+TgmobssJ6rS22dspWnu-oDxXevGmhMD8VcRBjmj-b9UDqRjw@mail.gmail.com
diff --git a/contrib/oid2name/oid2name.c b/contrib/oid2name/oid2name.c
@@ -319,7 +319,7 @@ sql_conn(struct options * my_opts)
 	if (PQstatus(conn) == CONNECTION_BAD)
 	{
 		fprintf(stderr, "%s: could not connect to database %s: %s",
-				"oid2name", my_opts->dbname, PQerrorMessage(conn));
+				"oid2name", PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));
 		PQfinish(conn);
 		exit(1);
 	}
diff --git a/contrib/vacuumlo/vacuumlo.c b/contrib/vacuumlo/vacuumlo.c
@@ -124,7 +124,7 @@ vacuumlo(const char *database, const struct _param * param)
 	if (PQstatus(conn) == CONNECTION_BAD)
 	{
 		fprintf(stderr, "Connection to database \"%s\" failed:\n%s",
-				database, PQerrorMessage(conn));
+				PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));
 		PQfinish(conn);
 		return -1;
 	}
diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c
@@ -1954,7 +1954,7 @@ connectDatabase(const char *dbname, const char *connection_string,
 		{
 			fprintf(stderr,
 					_("%s: could not connect to database \"%s\": %s"),
-					progname, dbname, PQerrorMessage(conn));
+					progname, PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));
 			exit_nicely(1);
 		}
 		else
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
@@ -686,7 +686,7 @@ doConnect(void)
 	if (PQstatus(conn) == CONNECTION_BAD)
 	{
 		fprintf(stderr, "connection to database \"%s\" failed:\n%s",
-				dbName, PQerrorMessage(conn));
+				PQdb(conn), PQerrorMessage(conn));
 		PQfinish(conn);
 		return NULL;
 	}
@@ -3312,7 +3312,8 @@ main(int argc, char **argv)
 
 	if (PQstatus(con) == CONNECTION_BAD)
 	{
-		fprintf(stderr, "connection to database \"%s\" failed\n", dbName);
+		fprintf(stderr, "connection to database \"%s\" failed\n",
+				PQdb(con) ? PQdb(con) : "");
 		fprintf(stderr, "%s", PQerrorMessage(con));
 		exit(1);
 	}

Original file line number	Diff line number	Diff line change
`@@ -319,7 +319,7 @@ sql_conn(struct options * my_opts)`
`319`	`319`	`if (PQstatus(conn) == CONNECTION_BAD)`
`320`	`320`	`{`
`321`	`321`	`fprintf(stderr, "%s: could not connect to database %s: %s",`
`322`		`- "oid2name", my_opts->dbname, PQerrorMessage(conn));`
	`322`	`+ "oid2name", PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));`
`323`	`323`	`PQfinish(conn);`
`324`	`324`	`exit(1);`
`325`	`325`	`}`
Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ vacuumlo(const char database, const struct _param param)`
`124`	`124`	`if (PQstatus(conn) == CONNECTION_BAD)`
`125`	`125`	`{`
`126`	`126`	`fprintf(stderr, "Connection to database \"%s\" failed:\n%s",`
`127`		`- database, PQerrorMessage(conn));`
	`127`	`+ PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));`
`128`	`128`	`PQfinish(conn);`
`129`	`129`	`return -1;`
`130`	`130`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1954,7 +1954,7 @@ connectDatabase(const char dbname, const char connection_string,`
`1954`	`1954`	`{`
`1955`	`1955`	`fprintf(stderr,`
`1956`	`1956`	`_("%s: could not connect to database \"%s\": %s"),`
`1957`		`- progname, dbname, PQerrorMessage(conn));`
	`1957`	`+ progname, PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn));`
`1958`	`1958`	`exit_nicely(1);`
`1959`	`1959`	`}`
`1960`	`1960`	`else`
Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,7 @@ doConnect(void)`
`686`	`686`	`if (PQstatus(conn) == CONNECTION_BAD)`
`687`	`687`	`{`
`688`	`688`	`fprintf(stderr, "connection to database \"%s\" failed:\n%s",`
`689`		`- dbName, PQerrorMessage(conn));`
	`689`	`+ PQdb(conn), PQerrorMessage(conn));`
`690`	`690`	`PQfinish(conn);`
`691`	`691`	`return NULL;`
`692`	`692`	`}`
`@@ -3312,7 +3312,8 @@ main(int argc, char **argv)`
`3312`	`3312`
`3313`	`3313`	`if (PQstatus(con) == CONNECTION_BAD)`
`3314`	`3314`	`{`
`3315`		`- fprintf(stderr, "connection to database \"%s\" failed\n", dbName);`
	`3315`	`+ fprintf(stderr, "connection to database \"%s\" failed\n",`
	`3316`	`+ PQdb(con) ? PQdb(con) : "");`
`3316`	`3317`	`fprintf(stderr, "%s", PQerrorMessage(con));`
`3317`	`3318`	`exit(1);`
`3318`	`3319`	`}`