686803 - RegExp throws multiple Java exceptions for invalid input

Status: RESOLVED FIXED

(Rhino Graveyard :: Core, defect)

Description

[André Bargull [:anba]]

Attachment: Conformance_RegExp_Objects.java

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0.2) Gecko/20100101 Firefox/6.0.2
Build ID: 20110902133214

Steps to reproduce:

Executed several regular expressions from test262

See test262 for the original test cases:
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.9_AtomEscape/S15.10.2.9_A1_T4.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.11_DecimalEscape/S15.10.2.11_A1_T2.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.11_DecimalEscape/S15.10.2.11_A1_T3.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.15_NonemptyClassRanges/S15.10.2.15_A1_T4.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.15_NonemptyClassRanges/S15.10.2.15_A1_T5.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.15_NonemptyClassRanges/S15.10.2.15_A1_T6.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.15_NonemptyClassRanges/S15.10.2.15_A1_T7.js
sputnik/Conformance/15_Native/15.10_RegExp_Objects/15.10.2_Pattern_Semantics/15.10.2.15_NonemptyClassRanges/S15.10.2.15_A1_T8.js


Actual results:

The following Java exceptions are thrown by the RegExp implementation:
- java.lang.ArrayIndexOutOfBoundsException
- java.lang.NullPointerException (already covered by bug389278)
- java.lang.RuntimeException


Expected results:

No exception resp. org.mozilla.javascript.EcmaError of type "SyntaxError" should be thrown.

Comment 1

[André Bargull [:anba]]

No longer reproducible in master/rhino_1_8, cf.
https://github.com/mozilla/rhino/commit/52e0456f76ed4cef784e55dd12ec844a06051f9d
https://github.com/mozilla/rhino/commit/c9005a11f2410aa77e7345dd4369e14c49e911dc

Comment 2

[Hannes Wallnoefer]

Thanks for testing! More test262 improvements are on the way.

Comment 3

[Hannes Wallnoefer]

Sorry, only saw your test case after closing the bug. Adding it right now.

Comment 4

[Hannes Wallnoefer]

Done. Thanks again!

https://github.com/mozilla/rhino/commit/2dd5abb896b4f283e60f9860214bdcaf98bd1b06