pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd
Date
Msg-id E1Xv8BJ-0008VQ-Ci@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd8 et al.

Coverity complained that the "else" added to fillPGconn() was unreachable,
which it was.  Remove the dead code.  In passing, rearrange the tests so as
not to bother trying to fetch values for options that can't be assigned.

Pre-9.3 did not have that issue, but it did have a "return" that should be
"goto oom_error" to ensure that a suitable error message gets filled in.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/179a9afdd77e04c089013fd3d5d5abe2d96516ed

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


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd
Next
From: Tom Lane
Date:
Subject: pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd