Changeset 3d361b8 in mod_gnutls for src/mod_gnutls.c

Timestamp:

Jan 13, 2015, 11:04:38 AM (7 years ago)

Author:

Thomas Klute <thomas2.klute@…>

Branches:

asyncio, debian/master, debian/stretch-backports, jessie-backports, master, proxy-ticket, upstream

Children:

e4b58b6

Parents:

64dadf8

Message:

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.

File:

• src/mod_gnutls.c (modified) (1 diff)

src/mod_gnutls.c

@@ -81 +81 @@
         return 1;
     }
-    ap_remove_input_filter(c->input_filters);
-    ap_remove_input_filter(c->output_filters);
+    if (c->input_filters)
+        ap_remove_input_filter(c->input_filters);
+    if (c->output_filters)
+        ap_remove_output_filter(c->output_filters);
     mgs_cleanup_pre_config(c->pool);
     sc->enabled = GNUTLS_ENABLED_FALSE;