aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/tqueue.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2015-09-28 18:02:38 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2015-09-28 18:02:38 -0400
commit60f1e6bc130241c1f1da337d0a08a05ee8b982d8 (patch)
tree5dbab7284222b9fba24975e2dfa9ceb93d5fb5c5 /src/backend/executor/tqueue.c
parent992d702bfafdb4258e65cbfa7353a631460a8389 (diff)
downloadpostgresql-60f1e6bc130241c1f1da337d0a08a05ee8b982d8.tar.gz
postgresql-60f1e6bc130241c1f1da337d0a08a05ee8b982d8.zip
Fix poor errno handling in libpq's version of our custom OpenSSL BIO.
Thom Brown reported that SSL connections didn't seem to work on Windows in 9.5. Asif Naeem figured out that the cause was my_sock_read() looking at "errno" when it needs to look at "SOCK_ERRNO". This mistake was introduced in commit 680513ab79c7e12e402a2aad7921b95a25a4bcc8, which cloned the backend's custom SSL BIO code into libpq, and didn't translate the errno handling properly. Moreover, it introduced unnecessary errno save/restore logic, which was particularly confusing because it was incomplete; and it failed to check for all three of EINTR, EAGAIN, and EWOULDBLOCK in my_sock_write. (That might not be necessary; but since we're copying well-tested backend code that does do that, it seems prudent to copy it faithfully.)
Diffstat (limited to 'src/backend/executor/tqueue.c')
0 files changed, 0 insertions, 0 deletions