Closed
Bug 1521869
Opened 7 years ago
Closed 7 years ago
QuotaManager::ShutdownObserver::Observe() has a data race
Categories
(Core :: Storage: Quota Manager, enhancement, P3)
Core
Storage: Quota Manager
Tracking
()
RESOLVED
INVALID
People
(Reporter: ytausky, Unassigned)
Details
This line reads done in the main thread, waiting for a write from the background thread, without synchronization.
|
Reporter | |
Updated•7 years ago
|
Priority: -- → P3
|
|
Reporter | |
Comment 1•7 years ago
|
||
Looking deeper, this is not a race condition: https://searchfox.org/mozilla-central/source/dom/quota/ActorsParent.cpp#2460
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
|
||
Comment 2•7 years ago
|
||
Yes, so this line:
https://searchfox.org/mozilla-central/rev/6c784c93cfbd5119ed07773a170b59fbce1377ea/dom/quota/ActorsParent.cpp#2501
works correctly, because:
https://searchfox.org/mozilla-central/rev/6c784c93cfbd5119ed07773a170b59fbce1377ea/xpcom/threads/nsThreadUtils.h#334
checks the condition after the event/runnable is processed.
You need to log in
before you can comment on or make changes to this bug.
Description
•