Fix comment in libpq OpenSSL code about why a substitue BIO is used.
authorHeikki Linnakangas <[email protected]>
Mon, 16 Feb 2015 21:05:20 +0000 (23:05 +0200)
committerHeikki Linnakangas <[email protected]>
Mon, 16 Feb 2015 21:05:20 +0000 (23:05 +0200)
The comment was copy-pasted from the backend code along with the
implementation, but libpq has different reasons for using the BIO.

src/interfaces/libpq/fe-secure-openssl.c

index a32af343a50a32bcb713537d30a8734d5284b3f8..1b9f3a4a7b0a7ad065c62bfcd1a85ba3c3b91df1 100644 (file)
@@ -1569,12 +1569,9 @@ PQsslAttribute(PGconn *conn, const char *attribute_name)
 }
 
 /*
- * Private substitute BIO: this does the sending and receiving using send() and
- * recv() instead. This is so that we can enable and disable interrupts
- * just while calling recv(). We cannot have interrupts occurring while
- * the bulk of openssl runs, because it uses malloc() and possibly other
- * non-reentrant libc facilities. We also need to call send() and recv()
- * directly so it gets passed through the socket/signals layer on Win32.
+ * Private substitute BIO: this does the sending and receiving using
+ * pqsecure_raw_write() and pqsecure_raw_read() instead, to allow those
+ * functions to disable SIGPIPE and give better error messages on I/O errors.
  *
  * These functions are closely modelled on the standard socket BIO in OpenSSL;
  * see sock_read() and sock_write() in OpenSSL's crypto/bio/bss_sock.c.