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

Re: [perl #128420] Changes in regex recursion in 5.24

Thread Previous | Thread Next
From:
Steve Hay via perl5-porters
Date:
May 2, 2017 20:49
Subject:
Re: [perl #128420] Changes in regex recursion in 5.24
Message ID:
CADED=K7uZS0S_CXmD8iWr78V_aqvqZVL1CuPDYsSrwcawvbt3w@mail.gmail.com
On 2 May 2017 at 18:38, demerphq <demerphq@gmail.com> wrote:
> On 2 May 2017 at 17:39, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:
>> Dave Mitchell <davem@iabyn.com> writes:
>>
>>> On Mon, May 01, 2017 at 02:51:16AM +0100, Zefram wrote:
>>>> Jan Goyvaerts wrote:
>>>> >if ("aqzaqzaqz" =~ /(a(?R)z|q)*/) {
>>>>
>>>> This has been fixed in blead.
>>>
>>> in particular, v5.25.0-36-gda7cf1c:
>>>
>>>     fix #128109 - do not move RExC_open_parens[0] in reginsert
>>
>> That commit, and v5.25.1-196-gec5bd2262b, which adds a test for it,
>> cherry-pick cleanly to maint-5.24.  Seeing as it's a crasher and
>> regression from 5.22, it's a candidate for backporting according to
>> perlpolicy.
>
> I just pushed
>
> 78b60e1 Add tests for regex recursion
> 4539ae3 fix #128109 - do not move RExC_open_parens[0] in reginsert
> 5edefd5 fix #128085 - SIGSEGV in S_regmatch with S_study_chunk:
> Assertion "!frame" failed.
>
> to maint-5.24 -- I took the liberty of including 5edefd5 which was
> part of how I found out about #128109
>

I don't think this should have been done right now. maint-5.22 and
maint-5.24 are both still in code freeze pending the release of 5.22.4
and 5.24.2, which are intended only to contain the last @INC fixes
(and possibly some other security fixes, as per the previous releases
on those branches).

There are many, many other bug fixes (crashes and otherwise) that
could be backported, but haven't been yet. They were deliberately
being left for 5.24.3, once the last of the @INC fixes was out of the
way.

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