Closed
Bug 754374
Opened 13 years ago
Closed 8 years ago
Call bad cert handler and SSL error listener asynchronously to avoid using SyncRunnableBase
Categories
(Core :: Security: PSM, defect)
Core
Security: PSM
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: briansmith, Unassigned)
References
Details
AFAICT, we do not have to call the nsISSLErrorListener synchronously; it is OK to use normal event dispatching to the main thread. This will solve the second most common cause of deadlocks due to bug 729536.
Reporter | ||
Comment 1•13 years ago
|
||
This must be done for the bad cert handler too, at least in the synchronous cert verification case (i.e. for Thunderbird), to avoid bug 729536.
Summary: Call SSL error listener asynchronously to avoid using SyncRunnableBase → Call bad cert handler and SSL error listener asynchronously to avoid using SyncRunnableBase
Reporter | ||
Updated•11 years ago
|
Assignee: brian → nobody
nsISSLErrorListener was removed in bug 844351 and nsIBadCertListener2 dispatch only happens in CertErrorRunnable::CheckCertOverrides(), which only happens as part of running a CertErrorRunnable, which only happens on the main thread (as part of CertErrorRunnableRunnable running on the STS thread and dispatching the given CertErrorRunnable to the main thread and then waiting for it to finish). So, doesn't look like there's anything more to do here.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•