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@…>, 4 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.