Don't simulate OOM in ExceptionHandlerBailout()

RESOLVED FIXED in Firefox 44

Status

()

Core
JavaScript Engine
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jonco, Assigned: jonco)

Tracking

unspecified
mozilla44
Points:
---

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Following on from bug 1171909 and bug 1201057, update ExceptionHandlerBailout() so that we don't simulate OOM while bailing out
(Assignee)

Comment 1

2 years ago
BailoutIonToBaseline() is called elsewhere where we do simulate OOM so there should be some coverage of this.

This change is motivated by the unhandlable OOM crash in ExceptionHandlerBailout() happening when testing other OOM handling patches.
(Assignee)

Comment 2

2 years ago
Created attachment 8669032 [details] [diff] [review]
bug1210760-oom-in-exception-handler

Patch to add AutoEnterOOMUnsafeRegion around where we call ExceptionHandlerBailout().
Attachment #8669032 - Flags: review?(terrence)
Attachment #8669032 - Flags: review?(terrence) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/5118292ae7f3
https://hg.mozilla.org/mozilla-central/rev/5118292ae7f3
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.