postgres.git / summary / log / commit / refs

commit    c2be18c333bc09bae27d6a518677232d6dbcb45d
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Sun Nov 30 17:20:57 2014 +0000

    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 | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c
index 6ac3774d842..43911a2dea3 100644
--- a/src/interfaces/libpq/fe-connect.c
+++ b/src/interfaces/libpq/fe-connect.c
@@ -643,7 +643,7 @@ fillPGconn(PGconn *conn, PQconninfoOption *connOptions)
 			free(conn->sslmode);
 		conn->sslmode = strdup("require");
 		if (!conn->sslmode)
-			return false;
+			goto oom_error;
 	}
 #endif
 	FILL_CONN_OPTION(conn->requirepeer, "requirepeer");
@@ -746,7 +746,6 @@ connectOptions2(PGconn *conn)
 			conn->pgpass = strdup(DefaultPassword);
 			if (!conn->pgpass)
 				goto oom_error;
-
 		}
 		else
 			conn->dot_pgpass_used = true;


[parent: 168636a9914e]