Bug 902158 - don't null out the socket security callbacks on the main thread; r=rkent,hiro a=standard8
authorIrving Reid <irving@mozilla.com>
Tue, 21 Oct 2014 14:52:45 -0400
changeset 25499 0781ca74f44a36221c45bddcc7e7d5f7d6d6b658
parent 25498 d22cbdff2a6f26d9589c4b052b0e5cae81cf876a
child 25501 02b8a1d209a1a7db1cec780e2dd41af5da57f2e7
push id1850
push userclokep@gmail.com
push dateWed, 08 Mar 2017 19:29:12 +0000
treeherdercomm-esr52@028df196b2d9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrkent, hiro, standard8
bugs902158
Bug 902158 - don't null out the socket security callbacks on the main thread; r=rkent,hiro a=standard8
mailnews/base/util/nsMsgProtocol.cpp
--- a/mailnews/base/util/nsMsgProtocol.cpp
+++ b/mailnews/base/util/nsMsgProtocol.cpp
@@ -303,17 +303,16 @@ nsresult nsMsgProtocol::CloseSocket()
   nsresult rv = NS_OK;
   // release all of our socket state
   m_socketIsOpen = false;
   m_inputStream = nullptr;
   m_outputStream = nullptr;
   if (m_transport) {
     nsCOMPtr<nsISocketTransport> strans = do_QueryInterface(m_transport);
     if (strans) {
-      strans->SetSecurityCallbacks(nullptr);
       strans->SetEventSink(nullptr, nullptr); // break cyclic reference!
     }
   }
   // we need to call Cancel so that we remove the socket transport from the mActiveTransportList.  see bug #30648
   if (m_request) {
     rv = m_request->Cancel(NS_BINDING_ABORTED);
   }
   m_request = 0;