source: mod_gnutls/debian/patches/proxy-segfault-fix.patch @ 2db6923

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

refresh patches with gbp pq
  • Property mode set to 100644
File size: 1.1 KB

  • src/mod_gnutls.c 

    From: Thomas Klute <thomas2.klute@uni-dortmund.de>
Date: Tue, 13 Jan 2015 17:04:38 +0100
Subject: Check if filters exist before removing them in ssl_engine_disable

Trying to remove filters that are NULL leads to a segfault in the worker
thread. Check if c->input_filters and c->output_filters are defined
before removing and remove only if set.

Also, output filters should be removed with the dedicated function.
---
 src/mod_gnutls.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mod_gnutls.c b/src/mod_gnutls.c
index e576fb8..0a32ffd 100644
    a b int ssl_engine_disable(conn_rec *c) { 
    8080    if(sc->enabled == GNUTLS_ENABLED_FALSE) {
    8181        return 1;
    8282    }
    83     ap_remove_input_filter(c->input_filters);
    84     ap_remove_input_filter(c->output_filters);
     83    if (c->input_filters)
     84        ap_remove_input_filter(c->input_filters);
     85    if (c->output_filters)
     86        ap_remove_output_filter(c->output_filters);
    8587    mgs_cleanup_pre_config(c->pool);
    8688    sc->enabled = 0;
    8789    return 1;
Note: See TracBrowser for help on using the repository browser.