Bug 1334465 - Set mIPCClosed to true before calling SendDeleteSelf in order to avoid race r=bagder

In the previous code, a race condition could cause us to call SendSetPriority() after calling SendDeleteSelf.

For example:
T1: SendDeleteSelf()
T2: if (!mIPCClosed) SendSetPriority()
T1: mIPCClosed = true

MozReview-Commit-ID: 3XOwCaphb2o

--HG--
extra : source : 4ebdab0e332892378558817e30d0138c95199ce5
This commit is contained in:
Valentin Gosu 2018-02-11 03:51:09 +01:00
parent 0038490a71
commit e4bb1bbe71

View File

@ -2292,8 +2292,8 @@ HttpChannelParent::UpdateAndSerializeSecurityInfo(nsACString& aSerializedSecurit
bool
HttpChannelParent::DoSendDeleteSelf()
{
bool rv = SendDeleteSelf();
mIPCClosed = true;
bool rv = SendDeleteSelf();
CleanupBackgroundChannel();