Mailing List Archive

remainder of patch for DIR_MERGE leak
I think this is also required (I've only just gotten around to testing
it):

*** lib/Apache/ExtUtils.pm.old Thu Nov 23 14:40:04 2000
--- lib/Apache/ExtUtils.pm Thu Nov 23 14:40:17 2000
*************** static mod_perl_perl_dir_config *newPerl
*** 195,201 ****
palloc(p, sizeof (mod_perl_perl_dir_config));
cld->obj = Nullsv;
cld->pclass = "$class";
- register_cleanup(p, cld, perl_perl_cmd_cleanup, null_cleanup);
return cld;
}

--- 195,200 ----

At least with my very bad knowledge of mod_perl internals, I think that is
required...

--
<Matt/>

/|| ** Director and CTO **
//|| ** AxKit.com Ltd ** ** XML Application Serving **
// || ** http://axkit.org ** ** XSLT, XPathScript, XSP **
// \\| // ** Personal Web Site: http://sergeant.org/ **
\\//
//\\
// \\
Re: remainder of patch for DIR_MERGE leak [ In reply to ]
On Thu, 23 Nov 2000, Matt Sergeant wrote:

> I think this is also required (I've only just gotten around to testing
> it):

if you take that out, from what i can see, cleanups will not be run for
{DIR,SERVER}CREATE.
the patch that went into 1.24_01 fixes the leak for {DIR,SERVER}MERGE.

> *** lib/Apache/ExtUtils.pm.old Thu Nov 23 14:40:04 2000
> --- lib/Apache/ExtUtils.pm Thu Nov 23 14:40:17 2000
> *************** static mod_perl_perl_dir_config *newPerl
> *** 195,201 ****
> palloc(p, sizeof (mod_perl_perl_dir_config));
> cld->obj = Nullsv;
> cld->pclass = "$class";
> - register_cleanup(p, cld, perl_perl_cmd_cleanup, null_cleanup);
> return cld;
> }
>
> --- 195,200 ----
>
> At least with my very bad knowledge of mod_perl internals, I think that is
> required...
>
> --
> <Matt/>
>
> /|| ** Director and CTO **
> //|| ** AxKit.com Ltd ** ** XML Application Serving **
> // || ** http://axkit.org ** ** XSLT, XPathScript, XSP **
> // \\| // ** Personal Web Site: http://sergeant.org/ **
> \\//
> //\\
> // \\
>