Author: gozer
Date: Mon Jan  3 14:03:23 2005
New Revision: 124029

URL: http://svn.apache.org/viewcvs?view=rev&rev=124029
Log:
Reverting the leak fix for Perl, as it should have been
left alone, parms->pool was the right pool to allocate
from, since the created ap_directive_t has to live after
configuration.
 

Modified:
   perl/modperl/trunk/Changes
   perl/modperl/trunk/src/modules/perl/modperl_cmd.c

Modified: perl/modperl/trunk/Changes
Url: 
http://svn.apache.org/viewcvs/perl/modperl/trunk/Changes?view=diff&rev=124029&p1=perl/modperl/trunk/Changes&r1=124028&p2=perl/modperl/trunk/Changes&r2=124029
==============================================================================
--- perl/modperl/trunk/Changes  (original)
+++ perl/modperl/trunk/Changes  Mon Jan  3 14:03:23 2005
@@ -12,7 +12,7 @@
 
 =item 1.99_20-dev
 
-Perl, <Perl> and PerlPostConfigRequires were leaking some memory at
+<Perl> and PerlPostConfigRequires were leaking some memory at
 startup. Use parms->temp_pool instead of parms->pool for temporary
 memory allocations. [Gozer]
 

Modified: perl/modperl/trunk/src/modules/perl/modperl_cmd.c
Url: 
http://svn.apache.org/viewcvs/perl/modperl/trunk/src/modules/perl/modperl_cmd.c?view=diff&rev=124029&p1=perl/modperl/trunk/src/modules/perl/modperl_cmd.c&r1=124028&p2=perl/modperl/trunk/src/modules/perl/modperl_cmd.c&r2=124029
==============================================================================
--- perl/modperl/trunk/src/modules/perl/modperl_cmd.c   (original)
+++ perl/modperl/trunk/src/modules/perl/modperl_cmd.c   Mon Jan  3 14:03:23 2005
@@ -420,7 +420,7 @@
 
 MP_CMD_SRV_DECLARE(perl)
 {
-    apr_pool_t *p = parms->temp_pool;
+    apr_pool_t *p = parms->pool;
     const char *endp = ap_strrchr_c(arg, '>');
     const char *errmsg;
     char *code = "";

Reply via email to