[SCM] Debian packaging of libapache2-mod-perl2 branch, master, updated. debian/2.0.8+httpd24-r1449661-3-3-gfaf04cd

gregor herrmann gregoa at debian.org
Sat Jun 15 15:35:32 UTC 2013


The following commit has been merged in the master branch:
commit 93ad52f87c827df57aecb223dc8784818004343d
Author: gregor herrmann <gregoa at debian.org>
Date:   Sat Jun 15 17:14:42 2013 +0200

    Add 320-cleanup-authn-authz-providers.patch from Niko Tyni.
    
    The patch makes sure authn and authz providers are cleaned up in time by
    registering them in the configuration pool rather than the server process
    pool.
    
    Thanks: Stefan Fritsch for suggesting the fix.
    Closes: #711213

diff --git a/debian/patches/320-cleanup-authn-authz-providers.patch b/debian/patches/320-cleanup-authn-authz-providers.patch
new file mode 100644
index 0000000..0a2f0a7
--- /dev/null
+++ b/debian/patches/320-cleanup-authn-authz-providers.patch
@@ -0,0 +1,43 @@
+From 75662f4285bad8d28fe787d1293bdd95a535cad1 Mon Sep 17 00:00:00 2001
+From: Niko Tyni <ntyni at debian.org>
+Date: Fri, 14 Jun 2013 14:07:22 +0300
+Subject: [PATCH] Make sure authn and authz providers are cleaned up in time
+
+The authn and authz providers need to be registered in the configuration
+pool rather than the server process pool.  The latter outlives the load
+time of the mod_perl shared object, so cleanup hooks could get called
+after the registered functions were unloaded, causing SIGSEGVs and other
+undefined behaviour.
+
+Fix suggested by Stefan Fritsch.
+
+Bug-Debian: http://bugs.debian.org/711213
+---
+ src/modules/perl/modperl_cmd.c |    4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/src/modules/perl/modperl_cmd.c b/src/modules/perl/modperl_cmd.c
+index 4ae0a62..d14b330 100644
+--- a/src/modules/perl/modperl_cmd.c
++++ b/src/modules/perl/modperl_cmd.c
+@@ -419,7 +419,7 @@ MP_CMD_SRV_DECLARE(init_handlers)
+ 
+ MP_CMD_SRV_DECLARE2(authz_provider)
+ {
+-    apr_pool_t *p = parms->server->process->pool;
++    apr_pool_t *p = parms->pool;
+     char *name = apr_pstrdup(p, arg1);
+     char *cb = apr_pstrdup(p, arg2);
+ 
+@@ -431,7 +431,7 @@ MP_CMD_SRV_DECLARE2(authz_provider)
+ 
+ MP_CMD_SRV_DECLARE2(authn_provider)
+ {
+-    apr_pool_t *p = parms->server->process->pool;
++    apr_pool_t *p = parms->pool;
+     char *name = apr_pstrdup(p, arg1);
+     char *cb = apr_pstrdup(p, arg2);
+ 
+-- 
+1.7.10.4
+
diff --git a/debian/patches/series b/debian/patches/series
index 46f384d..d0b6deb 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -13,3 +13,4 @@ avoid-db-linkage.patch
 290-httpd-transition-test-todo.patch
 300-http-headers-version.patch
 310-defaultruntimedir.patch
+320-cleanup-authn-authz-providers.patch

-- 
Debian packaging of libapache2-mod-perl2



More information about the Pkg-perl-cvs-commits mailing list