pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd
Date: 2014-11-30 17:21:09
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: 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
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/1adbb347ec6ca059168a1866357ca7a23d117272

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

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2014-12-01 00:44:22 pgsql: Add a #define for the inet overlaps operator.
Previous Message Tom Lane 2014-11-30 03:12:41 Re: [COMMITTERS] pgsql: Revert "Add libpq function PQhostaddr()."