On Sep 27, 2010, at 9:19 AM, Jim Jagielski wrote:

> 
> On Sep 27, 2010, at 8:17 AM, Jim Jagielski wrote:
> 
>> Let me look...
>> On Sep 26, 2010, at 12:13 PM, Stefan Fritsch wrote:
>> 
>>> Hi,
>>> 
>>> mod_slotmem_shm creates a new global pool on every graceful restart 
>>> but never destroys it.
>>> 
>>> From quickly looking at the code it is not clear to me what the 
>>> correct behaviour would be. Should the pool be created from pconf 
>>> instead? Or should it be destroyed in cleanup_slotmem()?
>>> 
> 
> creating from pconf is cleaner...
> 

Hold on a tic... I'm not seeing the leak. During
cleanup_slotmem we do destroy next->gpool which
is set to gpool.

Reply via email to