Looks good to me, I applied both patches and the issue I was having with lua remains fixed.
Thanks! Jesse On Wed, Feb 8, 2017 at 2:21 AM Willy Tarreau <[email protected]> wrote: > Hi Jesse, > > On Tue, Feb 07, 2017 at 06:37:09PM +0000, Jesse Schulman wrote: > > Thank you for the update, we are running the patch Thierry provided with > > success, but we only do a lua call within the %[] almost identically to > the > > simple reproducer I provided. I *think* we are safe considering we don't > > do any redirect in the way that your (Willy's) reproducer is doing it. > > > > We will definitely look to upgrade to the next available stable version > > that includes the proper fix. > > Here come the two patches I intend to merge. The first one implements a > simple dedicated trash allocator and the second one is Thierry's patch > rebased to use it. It fixes the issue for me in both cases (your Lua test > and my redirect involving the string conversion). > > Please test and let me know. I'm confident but I prefer to get your > confirmation before merging them. > > Thanks! > Willy >
