TM: bizarre results for ((1 * (1)) | "")

VERIFIED FIXED in mozilla1.9.1b2

Status

()

Core
JavaScript Engine
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: gal)

Tracking

({testcase, verified1.9.1})

Trunk
mozilla1.9.1b2
x86
Mac OS X
testcase, verified1.9.1
Points:
---
Bug Flags:
blocking1.9.1 +
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
e.js:

empty = [];
out = [];
for (var j=0;j<10;++j) { empty[42]; out.push((1 * (1)) | ""); }
print(uneval(out));

./js -j ~/e.js
[1, 1, 1965522873, 9, 16, 25, 36, 49, 64, 81]
(Reporter)

Comment 1

10 years ago
Why do I need the "empty[42]" part?  That's an artifact of how the fuzzer works, rather than something the fuzzer generated :(
(Assignee)

Comment 2

10 years ago
Created attachment 348523 [details] [diff] [review]
patch
Assignee: general → gal
Status: NEW → ASSIGNED
Attachment #348523 - Flags: review?(edwsmith)
(Assignee)

Comment 3

10 years ago
This is a super evil bug. We can't do CSE across in-trace branch labels. Will push with r=me until someone + or - it.
(Assignee)

Updated

10 years ago
Attachment #348523 - Flags: review?(danderson)
Attachment #348523 - Flags: review?(danderson) → review+
(Assignee)

Comment 4

10 years ago
http://hg.mozilla.org/tracemonkey/rev/55e30117ecee
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

10 years ago
ccing ed and graydon

Updated

10 years ago
Attachment #348523 - Flags: review?(edwsmith) → review+
reopening, marking blocking beta2, will close once landed on m-c.
Status: RESOLVED → REOPENED
Flags: blocking1.9.1+
Resolution: FIXED → ---

Updated

10 years ago
Target Milestone: --- → mozilla1.9.1b2
Fixed in the merge pushed by vlad on Nov 18 14:11:14 2008 -0800:
http://hg.mozilla.org/mozilla-central/rev/e8ed5d4bf531
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED

Comment 8

10 years ago
Checking in js1_5/extensions/regress-465276.js;
/cvsroot/mozilla/js/tests/js1_5/extensions/regress-465276.js,v  <--  regress-465276.js
initial revision: 1.1
Flags: in-testsuite+
Flags: in-litmus-
Keywords: fixed1.9.1

Comment 10

10 years ago
v 1.9.1, 1.9.2
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
You need to log in before you can comment on or make changes to this bug.