On Sat, 24 May 2008 15:04:01 +0200 Ruediger Pluem <[EMAIL PROTECTED]> wrote:
> Sorry for pointing out this that late. I just thought about it again > and I guess we have a leak here. I think the following two lines are > missing here (before and after destroy_resource): > > reslist->ntotal--; > > > + rv = destroy_resource(reslist, res); > > free_container(reslist, res); Hmmm, good catch. Maybe it would be better to call apr_reslist_invalidate at this point, and keep the relevant cleanup in one place. -- Nick Kew Application Development with Apache - the Apache Modules Book http://www.apachetutor.org/
