33239 - Bad trees cause NullPointerException

Status: RESOLVED FIXED

(Rhino Graveyard :: Core, defect, P3)

Description

[Norris Boyd]

Subject:
             a rhino bug with parsing
        Date:
             Fri, 24 Mar 2000 15:25:58 -0500
       From:
             Kumanan Yogaratnam <[e-mail]>
 Organization:
             Espial Group Inc.
         To:
             [e-mail]




hi norris,

the following was reported by our QA dept:

(just so you know, we do not use the optimizer.)

---------------------------------- 8<--------------
Test case for a JavaScript Bug:

The JS engine failed with the following piece of code:

    alert("inside js file");
    var anArray = new Array();
    var pos = 1;
    anArray[0] = 5;
    anArray[1] = 6;

    ++anArray[--pos];
    alert(anArray[pos]);

The above script works fine within netscape, but it throws the following
exception with our browser which uses rhino:

java.lang.NullPointerException
 at org.mozilla.javascript.Interpreter.generateICode(Compiled Code)
 at org.mozilla.javascript.Interpreter.generateICode(Compiled Code)
 at org.mozilla.javascript.Interpreter.generateICode(Compiled Code)
 at org.mozilla.javascript.Interpreter.generateICode(Compiled Code)
 at org.mozilla.javascript.Interpreter.generateICode(Compiled Code)
 at org.mozilla.javascript.Interpreter.generateICodeFromTree(Compiled
Code)
 at
org.mozilla.javascript.Interpreter.generateScriptICode(Interpreter.java)
 at org.mozilla.javascript.Interpreter.compile(Interpreter.java)
 at org.mozilla.javascript.Context.compile(Context.java)
 at org.mozilla.javascript.Context.compile(Context.java)
 at org.mozilla.javascript.Context.compileReader(Context.java)

------------------------ 8<---------------

regards,
kumanan

--
Kumanan Yogaratnam  <mailto:[e-mail]>
Espial Group Inc.
http://www.espial.com

Comment 1

[Norris Boyd]

Attachment: proposed fix

Comment 2

[Norris Boyd]

Checking in IRFactory.java;
/m/pub/mozilla/js/rhino/org/mozilla/javascript/IRFactory.java,v  <--  IRFactory.
java
new revision: 1.10; previous revision: 1.9
done