Fix redundant error messages in client tools

petere · petere · commit 16eadc4695ef · 2020-11-07T22:42:17.000+01:00
A few client tools duplicate error messages already provided by libpq. Discussion: https://www.postgresql.org/message-id/flat/3e937641-88a1-e697-612e-99bba4b8e5e4%40enterprisedb.com
diff --git a/src/bin/pg_basebackup/streamutil.c b/src/bin/pg_basebackup/streamutil.c
@@ -201,8 +201,7 @@ GetConnection(void)
 
 	if (PQstatus(tmpconn) != CONNECTION_OK)
 	{
-		pg_log_error("could not connect to server: %s",
-					 PQerrorMessage(tmpconn));
+		pg_log_error("%s", PQerrorMessage(tmpconn));
 		PQfinish(tmpconn);
 		free(values);
 		free(keywords);
diff --git a/src/bin/pg_rewind/libpq_fetch.c b/src/bin/pg_rewind/libpq_fetch.c
@@ -49,8 +49,7 @@ libpqConnect(const char *connstr)
 
 	conn = PQconnectdb(connstr);
 	if (PQstatus(conn) == CONNECTION_BAD)
-		pg_fatal("could not connect to server: %s",
-				 PQerrorMessage(conn));
+		pg_fatal("%s", PQerrorMessage(conn));
 
 	if (showprogress)
 		pg_log_info("connected to server");
diff --git a/src/bin/psql/startup.c b/src/bin/psql/startup.c
@@ -300,7 +300,7 @@ main(int argc, char *argv[])
 
 	if (PQstatus(pset.db) == CONNECTION_BAD)
 	{
-		pg_log_error("could not connect to server: %s", PQerrorMessage(pset.db));
+		pg_log_error("%s", PQerrorMessage(pset.db));
 		PQfinish(pset.db);
 		exit(EXIT_BADCONN);
 	}

Original file line number	Diff line number	Diff line change
`@@ -201,8 +201,7 @@ GetConnection(void)`
`201`	`201`
`202`	`202`	`if (PQstatus(tmpconn) != CONNECTION_OK)`
`203`	`203`	`{`
`204`		`- pg_log_error("could not connect to server: %s",`
`205`		`- PQerrorMessage(tmpconn));`
	`204`	`+ pg_log_error("%s", PQerrorMessage(tmpconn));`
`206`	`205`	`PQfinish(tmpconn);`
`207`	`206`	`free(values);`
`208`	`207`	`free(keywords);`
Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,7 @@ main(int argc, char *argv[])`
`300`	`300`
`301`	`301`	`if (PQstatus(pset.db) == CONNECTION_BAD)`
`302`	`302`	`{`
`303`		`- pg_log_error("could not connect to server: %s", PQerrorMessage(pset.db));`
	`303`	`+ pg_log_error("%s", PQerrorMessage(pset.db));`
`304`	`304`	`PQfinish(pset.db);`
`305`	`305`	`exit(EXIT_BADCONN);`
`306`	`306`	`}`