Changeset 28395 in webkit for trunk/LayoutTests/fast/js/resources/regexp-overflow.js

Timestamp:

Dec 4, 2007, 11:08:28 AM (18 years ago)

Author:

Darin Adler

Message:

JavaScriptCore:

Reviewed by Geoff.

• fix first part of ​http://bugs.webkit.org/show_bug.cgi?id=16220 <rdar://problem/5625221> Crash opening www.news.com (CNet)

Test: fast/js/regexp-overflow.html

• pcre/pcre_compile.cpp: (calculateCompiledPatternLengthAndFlags): Add room for the additional BRA/KET that was generated in the compile code but not taken into account here.

LayoutTests:

Reviewed by Geoff.

• test for first part of ​http://bugs.webkit.org/show_bug.cgi?id=16220 <rdar://problem/5625221> Crash opening www.news.com (CNet)

• fast/js/regexp-overflow-expected.txt: Updated.
• fast/js/resources/regexp-overflow.js: Added test cases.

File:

• trunk/LayoutTests/fast/js/resources/regexp-overflow.js (modified) (1 diff)

trunk/LayoutTests/fast/js/resources/regexp-overflow.js

@@ -1 +1 @@
 description(
-'Tests some regular expressions with braces in them that used to overflow the regular expression compilation preflight computation.'
+'Tests some regular expressions that used to overflow the regular expression compilation preflight computation.'
 );
 
 shouldBe('/(\\d)(\\1{1})/.exec("11").toString()', '"11,1,1"');
+
 shouldBe('/^(\\d{1,2})([ -:\\/\\.]{1})(\\d{1,2})(\\2{1})?(\\d{2,4})?$/.exec("1:1").toString()', '"1:1,1,:,1,,"');
-shouldBe('/^(\\d{4})([ -:\\/\\.]{1})(\\d{1,2})(\\2{1})(\\d{1,2})T(\\d{1,2})([ -:\\/\\.]{1})(\\d{1,2})(\\7{1})(\\d{1,2})Z$/.exec("1234:5:6T7/8/9Z").toString()', '"1234:5:6T7/8/9Z,1234,:,5,:,6,7,/,8,/,9"');
+
+shouldBe('/^(\\d{4})([ -:\\/\\.]{1})(\\d{1,2})(\\2{1})(\\d{1,2})T(\\d{1,2})([ -:\\/\\.]{1})(\\d{1,2})(\\7{1})(\\d{1,2})Z$/.exec("1234:5:6T7/8/9Z").toString()',
+    '"1234:5:6T7/8/9Z,1234,:,5,:,6,7,/,8,/,9"');
+
+shouldBe('/\\[["\'\\s]{0,1}([\\w-]*)["\'\\s]{0,1}([\\W]{0,1}=){0,2}["\'\\s]{0,1}([\\w-]*)["\'\\s]{0,1}\\]$/.exec("[]").toString()',
+    '"[],,,"');
+
+shouldBe('/(x){0,2}/.exec("").toString()', '","');
 
 debug('');