]> granicus.if.org Git - postgresql/commit
Fix minor bugs in commit 30bf4689a96cd283af33edcdd6b7210df3f20cd8 et al.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 30 Nov 2014 17:20:57 +0000 (12:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 30 Nov 2014 17:20:57 +0000 (12:20 -0500)
commitc2be18c333bc09bae27d6a518677232d6dbcb45d
tree1ca41567a4330e5be7f678cbecc672d1d291c60f
parent168636a9914ed01abc890d9e7e416c07af415d6a
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.
src/interfaces/libpq/fe-connect.c