TI+JM: Kraken crypto regression

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jandem, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

764 bytes, application/javascript
Details
(Reporter)

Description

6 years ago
Created attachment 531406 [details]
Testcase

Attachment is reduced from Kraken crypto-aes and crypto-ccm. It should print:
OK
test.js:19: TypeError: d is undefined

With -n -m -a it prints nothing.
Created attachment 531436 [details]
Reduced testcase
Attachment #531406 - Attachment is obsolete: true
While analyzing a loop body's modset we could incorrectly mark the compilation as having failed without throwing an exception, which would bubble up to TryCompile and cause the interpreter to just exit, also without reporting an exception.  This adds an assertion to the compiler that if it produces an error there is a pending exception, and also fixes the loop body analysis to be infallible (it can allocate, but only into vectors with a CompilerAllocPolicy).

http://hg.mozilla.org/projects/jaegermonkey/rev/71e561e48de4
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.