source: mod_gnutls/debian/patches/proxy-segfault-fix.patch @ 8ed8e96

debian/masterdebian/stretch-backportsjessie-backports
Last change on this file since 8ed8e96 was 8ed8e96, checked in by Daniel Kahn Gillmor <dkg@…>, 5 years ago

committing NMU 0.6-1.4 (thanks, Thomas!)

  • Property mode set to 100644
File size: 1.0 KB
RevLine 
[8ed8e96]1From 3d361b8e5d7c4c971d344658728979fe978dc759 Mon Sep 17 00:00:00 2001
2From: Thomas Klute <thomas2.klute@uni-dortmund.de>
3Date: Tue, 13 Jan 2015 17:04:38 +0100
4Subject: [PATCH] Check if filters exist before removing them in
5 ssl_engine_disable
6
7Trying to remove filters that are NULL leads to a segfault in the worker
8thread. Check if c->input_filters and c->output_filters are defined
9before removing and remove only if set.
10
11Also, output filters should be removed with the dedicated function.
12---
13 src/mod_gnutls.c | 6 ++++--
14 1 file changed, 4 insertions(+), 2 deletions(-)
15
16--- a/src/mod_gnutls.c
17+++ b/src/mod_gnutls.c
18@@ -80,8 +80,10 @@
19     if(sc->enabled == GNUTLS_ENABLED_FALSE) {
20         return 1;
21     }
22-    ap_remove_input_filter(c->input_filters);
23-    ap_remove_input_filter(c->output_filters);
24+    if (c->input_filters)
25+        ap_remove_input_filter(c->input_filters);
26+    if (c->output_filters)
27+        ap_remove_output_filter(c->output_filters);
28     mgs_cleanup_pre_config(c->pool);
29     sc->enabled = 0;
30     return 1;
Note: See TracBrowser for help on using the repository browser.