pgsql: Clear conn->errorMessage at successful completion of PQconnectdb - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Clear conn->errorMessage at successful completion of PQconnectdb
Date
Msg-id E1mPsx7-00067D-HZ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Clear conn->errorMessage at successful completion of PQconnectdb().

Commits ffa2e4670 and 52a10224e caused libpq's connection-establishment
functions to usually leave a nonempty string in the connection's
errorMessage buffer, even after a successful connection.  While that
was intentional on my part, more sober reflection says that it wasn't
a great idea: the string would be a bit confusing.  Also this broke at
least one application that checked for connection success by examining
the errorMessage, instead of using PQstatus() as documented.  Let's
clear the buffer at success exit, restoring the pre-v14 behavior.

Discussion: https://postgr.es/m/4170264.1620321747@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/138531f1bbc333745bd8422371c07e7e108d5528

Modified Files
--------------
src/interfaces/libpq/fe-connect.c | 7 +++++++
1 file changed, 7 insertions(+)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Doc: improve documentation of CREATE/ALTER SUBSCRIPTION.
Next
From: Andres Freund
Date:
Subject: pgsql: Fix potential for compiler warning in GlobalVisTestFor().