On 09/14/2009 07:32 PM, Stefan Fritsch wrote: > On Sun, 13 Sep 2009, Ruediger Pluem wrote: >>>> But your patch is causing core dumps during the proxy tests when >>>> running the test suite :-(. >>>> I currently don't understand why. >>>> >>> >>> Hmmm... either ctx->tmp_flush_bb is NULL or, since it was added in the >>> middle of the struct, you didn't do a make distclean 1st.... >> >> That is not the problem. I did a slightly modified patch that added it >> to the >> end. I suppose it has something to do with not matching pools or bucket >> allocators between bb and ctx->tmp_flush_bb. >> It fails on in the proxy case and in the proxy case we have some >> mixtures going >> on there regarding pools and bucket allocators caused by the pooled >> backend >> connections. > > Yes, the lifetime of the brigade was wrong. The attached patch works > without segfaults.
Thanks for the update. I committed a slightly modified version as r814807. It avoids the constant if check in the flush bucket case at the expense of always creating the brigade when setting up the context. Regards Rüdiger
