Script("") no longer works

VERIFIED FIXED

Status

()

Core
JavaScript Engine
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: shutdown, Assigned: brendan)

Tracking

({regression, verified1.8.0.8, verified1.8.1})

Trunk
regression, verified1.8.0.8, verified1.8.1
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
$ dbg.obj/js -e "Script('')"
-e:1: TypeError: self-modifying script detected
(Assignee)

Comment 1

11 years ago
Created attachment 241673 [details] [diff] [review]
fix to handle null case

Sorry about that -- this will be fixed when the bug it blocks is fixed on the 1.8 branch.

/be
Assignee: general → brendan
Status: NEW → ASSIGNED
Attachment #241673 - Flags: review+
(Assignee)

Comment 2

11 years ago
Fixed on trunk:

Checking in jsscript.c;
/cvsroot/mozilla/js/src/jsscript.c,v  <--  jsscript.c
new revision: 3.120; previous revision: 3.119
done

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Updated

11 years ago
Keywords: fixed1.8.1

Comment 3

11 years ago
Checking in regress-355982.js;
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-355982.js,v  <--  regress-355982.js
initial revision: 1.1
done
Flags: in-testsuite+

Comment 4

11 years ago
verified fixed 20061009 1.8 windows/linux/mac*, 1.9 windows/linux
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1 → verified1.8.1
marking "fixed1.8.0.8" because this was a regression from bug 355655. Checked in with 355655 on the 1.8.0 branch, but should be verified.
Keywords: fixed1.8.0.8

Comment 6

11 years ago
verified fixed in ff1.5.0.8 release in the browser and 1.8.0.8 20061108 windows/mac*/linux.
Keywords: fixed1.8.0.8 → verified1.8.0.8
You need to log in before you can comment on or make changes to this bug.