208293 - Loading a large script in the Rhino debugger results in an endless loop (100% CPU utilization)

Status: VERIFIED FIXED

(Rhino Graveyard :: Core, defect)

Description

[Steve Beal]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0; .NET CLR 1.0.3705)
Build Identifier: 

Loading a large script in the Rhino debugger results in an endless loop.
The CPU utilization will jump to 100% and stay there.  A <ctrl><break>
thread dump from the VM indicates an endless loop in 
org.mozilla.javascript.Context.readReader().

Looking at the code, an error was introduced with Context.java v1.110.
The buffer is never grown prior to the call to System.arraycopy().



Reproducible: Always

Steps to Reproduce:
1.Start the debugger
2.Load a script file containing more than 512 characters.
3.

Actual Results:  
CPU utilization rose to 100%

Expected Results:  
The text of the .js file should have been displayed in the
debugger.

Comment 1

[Steve Beal]

Attachment: patch correcting the described bug

Comment 2

[Phil Schwartau]

cc'ing Igor -

Comment 3

[Igor Bukanov]

I committed the fix. Thanks Steve for spotting this!

Comment 4

[Phil Schwartau]

Rubber-stamp vrfy -

Comment 5

[Igor Bukanov]

Targeting as resolved against 1.5R5