Bug 202882 – Regular expression hangs in Safari only

• Component: JavaScriptCore
• Status: RESOLVED FIXED

[Radu Samson]

Created attachment 380832 [details]
Compiled regular expression to match a legal act in multiple languages

The regular expression attached works in all browsers except Safari, probably due to character encoding representation on the underlying platform. Following input should match but crashes:

Directive 2001/83/EC

[Radar WebKit Bug Importer]

<rdar://problem/56236654>

[Christopher Reid]

Created attachment 380980 [details]
Patch

[Alex Christensen]

You should add a test case that fails without this change and passes with this change.

[Saam Barati]

Comment on attachment 380980 [details]
Patch

nice. Agree with Alex. Please add a test. It's super easy to do inside JSTests/stress. The test. can be as simple as a JavaScript program which doesn't throw.

[Christopher Reid]

Created attachment 382563 [details]
Patch

Added BumpPointerAllocator API test and a stress test.
I wasn't able to get a smaller regex that hits this infinite loop so I used the full expression for a stress test.

[Yusuke Suzuki]

Comment on attachment 382563 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 382563 [details]
Patch

Rejecting attachment 382563 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 382563, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as [e-mail]...
Fetching: https://bugs.webkit.org/attachment.cgi?id=382563&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=202882&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 382563 from bug 202882.
Fetching: https://bugs.webkit.org/attachment.cgi?id=382563
Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	A	JSTests/stress/regress-202882.js
	A	Tools/TestWebKitAPI/Tests/WTF/BumpPointerAllocator.cpp
	M	JSTests/ChangeLog
	M	Source/WTF/ChangeLog
	M	Source/WTF/wtf/BumpPointerAllocator.h
	M	Tools/ChangeLog
	M	Tools/TestWebKitAPI/CMakeLists.txt
	M	Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj

ERROR from SVN:
A repository hook failed: Commit blocked by pre-commit hook (exit code 1) with output:

    The following files contain tab characters:

        trunk/JSTests/stress/regress-202882.js

    Please use spaces instead to indent.
    If you must commit a file with tabs, use svn propset to set the "allow-tabs" property.
W: 4c033311588a0338e1d55e21edf8aad018f43b3a and refs/remotes/origin/master differ, using rebase:
:040000 040000 8725fbb9d31ae8497d4e803658928d288ed78320 598d5e740e3a09b1c1748b58121babbb1bed9b13 M	JSTests
:040000 040000 265775ad8d6861b254c338f19a222bcd2df408eb 52c195d9a341dc8f90723c81660ebf756fddf032 M	Source
:040000 040000 e67be50be89f0ceaf97394052f5f1eb0de3c5acc e37ce7bf8a5071dae2c4c720fe8e08b07cf89d7c M	Tools
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	A	JSTests/stress/regress-202882.js
	A	Tools/TestWebKitAPI/Tests/WTF/BumpPointerAllocator.cpp
	M	JSTests/ChangeLog
	M	Source/WTF/ChangeLog
	M	Source/WTF/wtf/BumpPointerAllocator.h
	M	Tools/ChangeLog
	M	Tools/TestWebKitAPI/CMakeLists.txt
	M	Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj

ERROR from SVN:
A repository hook failed: Commit blocked by pre-commit hook (exit code 1) with output:

    The following files contain tab characters:

        trunk/JSTests/stress/regress-202882.js

    Please use spaces instead to indent.
    If you must commit a file with tabs, use svn propset to set the "allow-tabs" property.
W: 4c033311588a0338e1d55e21edf8aad018f43b3a and refs/remotes/origin/master differ, using rebase:
:040000 040000 8725fbb9d31ae8497d4e803658928d288ed78320 598d5e740e3a09b1c1748b58121babbb1bed9b13 M	JSTests
:040000 040000 265775ad8d6861b254c338f19a222bcd2df408eb 52c195d9a341dc8f90723c81660ebf756fddf032 M	Source
:040000 040000 e67be50be89f0ceaf97394052f5f1eb0de3c5acc e37ce7bf8a5071dae2c4c720fe8e08b07cf89d7c M	Tools
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
Current branch master is up to date.

Full output: https://webkit-queues.webkit.org/results/13200148

[Yusuke Suzuki]

Can we land this? Or is it landed?

[Christopher Reid]

(In reply to Yusuke Suzuki from comment #8)
> Can we land this? Or is it landed?

The tab characters can be removed in the test for landing and still reproduce the issue. I was just worried how the commit queue will handle the utf-8 encoding.

Will there be any problems using the commit-queue with those utf-8 characters in the js test or should it be landed manually?

[Ross Kirsling]

(In reply to Christopher Reid from comment #9)
> Will there be any problems using the commit-queue with those utf-8
> characters in the js test or should it be landed manually?

You shouldn't need to worry about that; the restriction only applies to tab characters.

For instance, here's a test file with raw UTF-8 but an empty `svn proplist`:
https://github.com/WebKit/webkit/blob/master/JSTests/test262/test/intl402/Collator/prototype/this-value-collator-prototype.js

[Christopher Reid]

(In reply to Ross Kirsling from comment #10)
> (In reply to Christopher Reid from comment #9)
> > Will there be any problems using the commit-queue with those utf-8
> > characters in the js test or should it be landed manually?
> 
> You shouldn't need to worry about that; the restriction only applies to tab
> characters.
> 
> For instance, here's a test file with raw UTF-8 but an empty `svn proplist`:
> https://github.com/WebKit/webkit/blob/master/JSTests/test262/test/intl402/
> Collator/prototype/this-value-collator-prototype.js

Sounds good, I'll try re-landing it using the commit-queue without the tab characters.

[Christopher Reid]

Created attachment 384747 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 384747 [details]
Patch for landing

Clearing flags on attachment: 384747

Committed r253061: <https://trac.webkit.org/changeset/253061>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.