develooper Front page | perl.perl5.porters | Postings from May 2011

Re: [perl #91032] Invalid free in t/op/write.t

Thread Previous | Thread Next
From:
Dave Mitchell
Date:
May 31, 2011 05:56
Subject:
Re: [perl #91032] Invalid free in t/op/write.t
Message ID:
20110531125618.GF3593@iabyn.com
On Sun, May 29, 2011 at 09:29:49PM +0100, Dave Mitchell wrote:
> On Sun, May 29, 2011 at 12:54:45PM -0700, Father Chrysostomos via RT wrote:
> > On Wed May 25 12:40:13 2011, rmbarker.cpan@btinternet.com wrote:
> > > This misbehaviour first appeared at 
> > > commit 815f25c6e302f84ecce02c74fa717a19d787f662
> > > Author: Dave Mitchell <davem@fdisolutions.com>
> > > Date:   Sat May 10 02:45:23 2003 +0100
> > > 
> > > The code prior to that change suggests that C<maxops += 10> should be
> > > C<maxopss += 3>.
> > > 
> > > This patch follows that suggestion and includes the previous patch from
> > > Nicholas Clark and passes all tests. 
> > 
> > Did you compile with -DDEBUGGING? I tried your patch and got an
> > assertion failure:
> > 
> > Assertion failed: (fpc <= fops + maxops), function S_doparseform, file
> > pp_ctl.c, line 5118.
> > 
> > I’m probably too late in responding, because I see Dave Mitchell has
> > made about two dozens commits with various fixes to formline, notably
> > 3808a68.
> 
> Hang about, I'm confused now. All the failures I saw were in pp_parseform,
> related to buffer overruns in PL_formtarget. I never saw the assertion
> failure in S_doparseform, and none of my commits would have addressed such
> a failure.

But regardless, Robin's patch looks successful.

Can anyone see the assertion failure since my series of commits?

-- 
"Strange women lying in ponds distributing swords is no basis for a system
of government. Supreme executive power derives from a mandate from the
masses, not from some farcical aquatic ceremony."
    -- Dennis, "Monty Python and the Holy Grail"

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About