Bug 902158 - don't null out the socket security callbacks on the main thread; r=rkent,hiro a=jcranmer
authorIrving Reid <irving@mozilla.com>
Mon, 20 Oct 2014 22:34:10 -0400
changeset 21150 1e2c95e9daf3f88abaa4d2ab80985ad99ba736c4
parent 21149 ca3044b26a30a8db228a9c16c595bc1d8bbd862b
child 21151 b9477885bcd6b96895d9fdaafa7e3d8041df8580
push id1274
push usermbanner@mozilla.com
push dateMon, 12 Jan 2015 19:54:49 +0000
treeherdercomm-beta@baea280adc1c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrkent, hiro, jcranmer
bugs902158
Bug 902158 - don't null out the socket security callbacks on the main thread; r=rkent,hiro a=jcranmer
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;