Closed Bug 1447964 Opened 4 years ago Closed 4 years ago

CurrentThreadCanAccessZone() allows main thread context to access helper thread zones

Categories

(Core :: JavaScript Engine, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

Currently this allows access to any zone for the main thread context even if it's in use by a helper thread zone.
Patch to check helper thread zone access separately and fix the one place this caused an assertion.
Attachment #8961330 - Flags: review?(jdemooij)
Comment on attachment 8961330 [details] [diff] [review]
bug1447964-can-access-zone

Review of attachment 8961330 [details] [diff] [review]:
-----------------------------------------------------------------

Good find.
Attachment #8961330 - Flags: review?(jdemooij) → review+
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6df204b3970
Improve CurrentThreadCanAccessZone() check r=jandem
Priority: -- → P1
https://hg.mozilla.org/mozilla-central/rev/e6df204b3970
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.