Bug 186477

Summary:

Test js/regexp-zero-length-alternatives.html fails when RegExpJIT is disabled

Product:

WebKit

Reporter:

Michael Saboff <msaboff>

Component:

JavaScriptCore

Assignee:

Michael Saboff <msaboff>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, ews-watchlist, fpizlo, keith_miller, mark.lam, saam, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

185989

Attachments:

Description	Flags
Patch	none

Description Michael Saboff 2018-06-10 22:30:16 PDT

A couple of subtests of subtests of js/regexp-zero-length-alternatives.html fail when using the YARR interpreter.

These subtests all involve the RegExp: /(?:a*?|b*?)*/.

var re32 = new RegExp(/(?:a*?|b*?)*/);
"aaaa".match(re32); // Hangs, should return "aaaa"
"abab".match(re32); // Hangs, should return "abab"
"ab".match(re32); // Returns "", should return "ab"
"xabx".match(re32); // Hangs, should return "ab"

Comment 1 Radar WebKit Bug Importer 2018-06-10 22:30:49 PDT

<rdar://problem/40992533>

Comment 2 Michael Saboff 2018-06-11 08:01:00 PDT

Created attachment 342429 [details]
Patch

Comment 3 WebKit Commit Bot 2018-06-11 09:32:27 PDT

Comment on attachment 342429 [details]
Patch

Clearing flags on attachment: 342429

Committed r232718: <https://trac.webkit.org/changeset/232718>

Comment 4 WebKit Commit Bot 2018-06-11 09:32:29 PDT

All reviewed patches have been landed.  Closing bug.