Closed Bug 161642 Opened 23 years ago Closed 7 years ago

Loading JS with garbage chars in it will cause XML parse errors

Categories

(Other Applications Graveyard :: Venkman JS Debugger, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: bzbarsky, Assigned: rginda)

Details

Attachments

(2 files)

Loading the to-be-attached JS file in the "Source Code" vuew shows a pretty XML parse error message due to the bogus chars around 1300 bytes into the file.
Attached file JS file
Evil JS file in question (warning: 50k or so).
Attached patch proposed patchSplinter Review
Patch fixes problems with the previous control-character stripping regex. I've also removed the String object creation. We used to need a real String object to hold flags for the source line, but not anymore.
Product: Core → Other Applications
QA Contact: caillon → venkman
Component is obsolete so resolving bugs as INCOMPLETE
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: