diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2011-08-27 14:16:14 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2011-08-27 14:16:14 -0400 |
commit | 724e30c9f886efd852f714d47c56336ffa6916ec (patch) | |
tree | f0e1ae925df18fefb48dcce7e34a33df1664760d /configure.in | |
parent | 4bd7333b14786a2d757195e907709d2aee116809 (diff) | |
download | postgresql-724e30c9f886efd852f714d47c56336ffa6916ec.tar.gz |
Ensure we discard unread/unsent data when abandoning a connection attempt.
There are assorted situations wherein PQconnectPoll() will abandon a
connection attempt and try again with different parameters (eg, SSL versus
not SSL). However, the code forgot to discard any pending data in libpq's
I/O buffers when doing this. In at least one case (server returns E
message during SSL negotiation), there is unread input data which bollixes
the next connection attempt. I have not checked to see whether this is
possible in the other cases where we close the socket and retry, but it
seems like a matter of good defensive programming to add explicit
buffer-flushing code to all of them.
This is one of several issues exposed by Daniel Farina's report of
misbehavior after a server-side fork failure.
This has been wrong since forever, so back-patch to all supported branches.
Diffstat (limited to 'configure.in')
0 files changed, 0 insertions, 0 deletions