Closed
Bug 1258349
Opened 9 years ago
Closed 9 years ago
Assertion failure: size_t(opId.id()) <= 0xffui8, at CacheIR.h:150
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: cbook, Assigned: jandem)
References
()
Details
(Keywords: assertion)
Attachments
(2 files)
3.99 KB,
text/plain
|
Details | |
1.38 KB,
patch
|
efaust
:
review+
|
Details | Diff | Splinter Review |
Assertion failure: size_t(opId.id()) <= 0xffui8, at c:\Users\mozilla\debug-build s\mozilla-central\js\src\jit/CacheIR.h:150 Steps to reproduce: --> Load http://www.rentalcars.com/partners/integrations/stand-alone-app/?affiliateCode=jetstar&css=true&template=jetstar&forwardUrl=jetstar.rentalcars.com&preflang=en&prefcurrency=&cor=gb note: you might click ok on a slow script warning --> Assertion tested with todays m-c and windows 7 trunk debug build Regression of bug Bug 1255352 ?
Assignee | ||
Comment 1•9 years ago
|
||
Thanks. This is a bogus assert, will post a patch soon.
Flags: needinfo?(jdemooij)
Assignee | ||
Comment 2•9 years ago
|
||
Bogus assert. This website must be creating some weird proto chains.
Assignee: nobody → jdemooij
Status: NEW → ASSIGNED
Flags: needinfo?(jdemooij)
Attachment #8732838 -
Flags: review?(efaustbmo)
jsfunfuzz found this too (looking through backlog), but the testcase in comment 2 is very compact and concise, so I won't be filing a new bug.
Comment 4•9 years ago
|
||
Comment on attachment 8732838 [details] [diff] [review] Patch Review of attachment 8732838 [details] [diff] [review]: ----------------------------------------------------------------- Oh, indeed. Thanks for the static_assert.
Attachment #8732838 -
Flags: review?(efaustbmo) → review+
Comment 6•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f76214d65f75
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•