Bug 1328106 - check IPC state before sending event; r?jdm draft
authorLiang-Heng Chen <xeonchen@gmail.com>
Mon, 23 Jan 2017 16:23:34 +0800
changeset 464890 98e29837b0a409417d57fbe2da5e418bc1ca2516
parent 464862 d5e37c0a776f1f2c21ddac4612529f819e13733b
child 543035 c236484579039676698333834cad2a88c135b8e5
push id42476
push userbmo:xeonchen@mozilla.com
push dateMon, 23 Jan 2017 08:24:39 +0000
reviewersjdm
bugs1328106
milestone53.0a1
Bug 1328106 - check IPC state before sending event; r?jdm MozReview-Commit-ID: CeWCIG0dyKS
dom/network/TCPSocketParent.cpp
--- a/dom/network/TCPSocketParent.cpp
+++ b/dom/network/TCPSocketParent.cpp
@@ -355,18 +355,21 @@ TCPSocketParent::FireStringDataEvent(con
   MOZ_ASSERT(!mFilter, "Socket filtering doesn't support nsCString");
 
   SendEvent(NS_LITERAL_STRING("data"), data, aReadyState);
 }
 
 void
 TCPSocketParent::SendEvent(const nsAString& aType, CallbackData aData, TCPReadyState aReadyState)
 {
-  mozilla::Unused << PTCPSocketParent::SendCallback(nsString(aType), aData,
-                                                    static_cast<uint32_t>(aReadyState));
+  if (mIPCOpen) {
+    mozilla::Unused << PTCPSocketParent::SendCallback(nsString(aType),
+                                                      aData,
+                                                      static_cast<uint32_t>(aReadyState));
+  }
 }
 
 void
 TCPSocketParent::SetSocket(TCPSocket *socket)
 {
   mSocket = socket;
 }