On Jun 17, 2011, at 9:42 AM, Joe Orton wrote:

> mod_slotmem_shm is creating a subpool of pconf ("gpool") in the 
> pre_config hook.  It then hangs a cleanup off pconf in the post_config 
> hook, which uses something with the structures in "gpool".
> 
> This doesn't work (and segfaults with APR pool debugging) because the 
> gpool contents are invalidated by the time the cleanup runs.
> 
> Patch below moves the cleanup to the "gpool", which avoids the problem 
> and doesn't seem to break anything else, but the subpool is never 
> cleared or destroyed... so could we just remove the subpool altogether?  
> (Is there is an unfullfilled intention to bound memory use across 
> generations here perhaps?)
> 

The latter... but for now, removing the subpool and just using
pconf makes sense.

Reply via email to