]> granicus.if.org Git - postgresql/commit
Be more paranoid about null return values from libpq status functions.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 25 Nov 2015 22:31:54 +0000 (17:31 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 25 Nov 2015 22:31:54 +0000 (17:31 -0500)
commitd44b4deaad9f347c2e83ab50c7252f14fad1ef63
tree31e406942b7ec37f5e462a81e2907a9d7c94aef7
parent0ca3c51e5520b423b17f498f6390129659ef68e0
Be more paranoid about null return values from libpq status functions.

PQhost() can return NULL in non-error situations, namely when a Unix-socket
connection has been selected by default.  That behavior is a tad debatable
perhaps, but for the moment we should make sure that psql copes with it.
Unfortunately, do_connect() failed to: it could pass a NULL pointer to
strcmp(), resulting in crashes on most platforms.  This was reported as a
security issue by ChenQin of Topsec Security Team, but the consensus of
the security list is that it's just a garden-variety bug with no security
implications.

For paranoia's sake, I made the keep_password test not trust PQuser or
PQport either, even though I believe those will never return NULL given
a valid PGconn.

Back-patch to all supported branches.
src/bin/psql/command.c