Greg Stein <[EMAIL PROTECTED]> writes:

> Looks like the IMMORTAL and TRANSIENT buckets' destory function should be
> switched over to apr_bucket_destroy_notimpl. They no longer have a separate
> sub-allocation (needing to be freed) for bkt->data.

yep, that is the patch I had, which Ryan just verified...  now
committed... 

But I didn't check the CVS logs to find the point at which the free()
stopped being valid for such buckets.  I gather that is a recent
change?

-- 
Jeff Trawick | [EMAIL PROTECTED] | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...

Reply via email to