Open
Bug 921682
Opened 11 years ago
Updated 2 years ago
imgStatusTracker should use a safe weak pointer implementation
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
NEW
People
(Reporter: seth, Unassigned)
References
Details
Bug 867755 introduced the use of SupportsWeakPtr in imgStatusTracker. Unfortunately, as mentioned in that bug, the interface of SupportsWeakPtr is unsafe for multithreaded use (and questionable in general). imgStatusTracker should use a safe weak pointer implementation that involves acquiring a strong pointer to the object pointed to by the weak pointer, rather than just checking whether the weak pointer is null once and hoping that result stays valid for the rest of the function. Sadly, such an API does not exist at the time of this writing, so step one is probably to fix SupportsWeakPtr or create a new one.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•