]> granicus.if.org Git - postgresql/commit
Use OpenSSL's SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER flag.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 24 Jul 2011 19:18:16 +0000 (15:18 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 24 Jul 2011 19:18:16 +0000 (15:18 -0400)
commita5b0d95744cf27f8b68dce3eb0415c0d3a0e949b
tree3514dcd728183b9a6514b2a6b8ccc77213c58dbb
parent18e52ae1722bb0e6b9c263a9f7120aaad6a7386c
Use OpenSSL's SSL_MODE_ACCEPT_MOVING_WRITE_BUFFER flag.

This disables an entirely unnecessary "sanity check" that causes failures
in nonblocking mode, because OpenSSL complains if we move or compact the
write buffer.  The only actual requirement is that we not modify pending
data once we've attempted to send it, which we don't.  Per testing and
research by Martin Pihlak, though this fix is a lot simpler than his patch.

I put the same change into the backend, although it's less clear whether
it's necessary there.  We do use nonblock mode in some situations in
streaming replication, so seems best to keep the same behavior in the
backend as in libpq.

Back-patch to all supported releases.
src/backend/libpq/be-secure.c
src/interfaces/libpq/fe-secure.c