database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-06-26 12:21:12 +03:00
Code Activity

Fix minor bugs in commit 30bf4689a9 et al.

Browse Source 
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.
This commit is contained in:
Tom Lane
2014-11-30 12:20:57 -05:00
parent 168636a991
commit c2be18c333
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/interfaces/libpq/fe-connect.c
View File

@ -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;