On Tue, Oct 6, 2015 at 5:53 PM, Yann Ylavic <[email protected]> wrote: > On Tue, Oct 6, 2015 at 5:34 PM, Graham Leggett <[email protected]> wrote: >> >> apr_bucket_simple_copy() looks wrong - in theory we should have a proper >> copy function that does the right thing with the second copy, for example by >> not copying the pool. If we blindly copy the pool (or the request containing >> the pool) I see nothing that would prevent an attempt to free the pool twice. > > Agreed, we probably need something like this:
Committed in r1707084, all tests pass now.
