Changeset b22def6 in mod_gnutls

Timestamp:

Jan 14, 2020, 4:21:56 AM (3 years ago)

Author:

Fiona Klute <fiona.klute@…>

Branches:

asyncio, master, proxy-ticket

Children:

407ca6e

Parents:

05e2d9e

Message:

Remove SIGPIPE signal block

There's no discernible reason to keep this, the main HTTPD code
doesn't use it at all, and the proxy code with which it was originally
added in 33826c53d7991024eeed255f860e9818188e2bcb works fine without
it.

Files:

• include/mod_gnutls.h.in (modified) (1 diff)
• src/gnutls_hooks.c (modified) (1 diff)

include/mod_gnutls.h.in

@@ -261 +261 @@
 
 
-/* apr_signal_block() for blocking SIGPIPE */
-apr_status_t apr_signal_block(int signum);
-
 /* Proxy Support */
 /* An optional function which returns non-zero if the given connection

src/gnutls_hooks.c

@@ -834 +834 @@
         ap_log_error(APLOG_MARK, APLOG_EMERG, rv, s,
                      "Failed to reinit mutex '" MGS_OCSP_MUTEX_NAME "'.");
-
-    /* Block SIGPIPE Signals */
-    rv = apr_signal_block(SIGPIPE);
-    if(rv != APR_SUCCESS) {
-        /* error sending output */
-        ap_log_error(APLOG_MARK, APLOG_EMERG, rv, s,
-                "GnuTLS: Error Blocking SIGPIPE Signal!");
-    }
 }