diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-01-26 16:42:13 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-01-26 16:42:13 -0300 |
| commit | f17e8f33f781ce156ee73e8e7b6d104fcfa9c811 (patch) | |
| tree | 5becb139bb3003c9fbc305da039064653812b3de /src | |
| parent | 64bdb6e5f8451a14e7349d3bbdb45dbac447eacc (diff) | |
| download | postgresql-f17e8f33f781ce156ee73e8e7b6d104fcfa9c811.tar.gz postgresql-f17e8f33f781ce156ee73e8e7b6d104fcfa9c811.zip | |
Report the true database name on connection errors
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 2930c05634bc, make sure not to try to print NULL.)
Apply to branches 9.5 through 13. Branch master has already been
changed differently by commit 58cd8dca3de0.
Reported-by: Robert Haas <robertmhaas@gmail.com>
Discussion: https://postgr.es/m/CA+TgmobssJ6rS22dspWnu-oDxXevGmhMD8VcRBjmj-b9UDqRjw@mail.gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pg_dump/pg_dumpall.c | 2 | ||||
| -rw-r--r-- | src/bin/pgbench/pgbench.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c index 2c82b39af0d..0bc74da9cfb 100644 --- a/src/bin/pg_dump/pg_dumpall.c +++ b/src/bin/pg_dump/pg_dumpall.c @@ -1773,7 +1773,7 @@ connectDatabase(const char *dbname, const char *connection_string, if (fail_on_error) { pg_log_error("could not connect to database \"%s\": %s", - dbname, PQerrorMessage(conn)); + PQdb(conn) ? PQdb(conn) : "", PQerrorMessage(conn)); exit_nicely(1); } else diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c index 768218565db..84e3aa9ece5 100644 --- a/src/bin/pgbench/pgbench.c +++ b/src/bin/pgbench/pgbench.c @@ -1228,7 +1228,7 @@ doConnect(void) if (PQstatus(conn) == CONNECTION_BAD) { pg_log_error("connection to database \"%s\" failed: %s", - dbName, PQerrorMessage(conn)); + PQdb(conn), PQerrorMessage(conn)); PQfinish(conn); return NULL; } @@ -6047,7 +6047,7 @@ main(int argc, char **argv) if (PQstatus(con) == CONNECTION_BAD) { pg_log_fatal("connection to database \"%s\" failed: %s", - dbName, PQerrorMessage(con)); + PQdb(con) ? PQdb(con) : "", PQerrorMessage(con)); exit(1); } |