Closed
Bug 1747457
Opened 4 years ago
Closed 4 years ago
Fix a number of missing locks in worker code
Categories
(Core :: DOM: Workers, defect, P2)
Core
DOM: Workers
Tracking
()
RESOLVED
FIXED
97 Branch
People
(Reporter: jesup, Assigned: jesup)
References
(Blocks 1 open bug)
Details
(Keywords: csectype-race, sec-moderate, Whiteboard: [post-critsmash-triage][adv-main97+r])
Attachments
(1 file)
There are a number of missing locks in worker code; also mThread is access from multiple threads, but only set on one thread, so did some cleanup around that which will let the clang thread-safety patches be happy with it (mostly taking a local copy at the start of the RunLoop). Added comments as well. Annotations will land later, after base thread-safety support is landed.
|
Assignee | |
Comment 1•4 years ago
|
||
|
||
Updated•4 years ago
|
Assignee: nobody → rjesup
Status: NEW → ASSIGNED
|
||
Updated•4 years ago
|
Severity: -- → S3
|
||
Comment 3•4 years ago
|
||
dom/workers cleanup r=dom-worker-reviewers,smaug
https://hg.mozilla.org/integration/autoland/rev/8fa4dac9a6c1ddd86a78e22e2efc31c4d233e3aa
https://hg.mozilla.org/mozilla-central/rev/8fa4dac9a6c1
Group: core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox97:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
Flags: qe-verify-
Whiteboard: [post-critsmash-triage]
|
||
Updated•4 years ago
|
Whiteboard: [post-critsmash-triage] → [post-critsmash-triage][adv-main97+r]
|
||
Updated•3 years ago
|
Group: core-security-release
You need to log in
before you can comment on or make changes to this bug.
Description
•