Fixed nsHTTPChannel::GetEventSink(...) to addref the event sink being returned. This is consistant with XPCom guidelines..

netwerk/protocol/http/src/nsHTTPChannel.cpp

@@ -295,8 +295,16 @@ nsHTTPChannel::GetResponseString(char* *o_String)
 NS_IMETHODIMP
 nsHTTPChannel::GetEventSink(nsIHTTPEventSink* *o_EventSink) 
 {
-    *o_EventSink = m_pEventSink; 
-    return NS_OK;
+    nsresult rv;
+
+    if (o_EventSink) {
+        *o_EventSink = m_pEventSink;
+        NS_IF_ADDREF(*o_EventSink);
+        rv = NS_OK;
+    } else {
+        rv = NS_ERROR_NULL_POINTER;
+    }
+    return rv;
 }
 
 

netwerk/protocol/http/src/nsHTTPResponseListener.cpp

@@ -242,10 +242,10 @@ nsresult nsHTTPResponseListener::FireOnHeadersAvailable()
     if (m_bHeadersDone) {
 
         // Notify the event sink that response headers are available...
-        nsIHTTPEventSink* pSink= nsnull;
-        m_pConnection->GetEventSink(&pSink);
-        if (pSink) {
-            pSink->OnHeadersAvailable(m_pConnection);
+        nsCOMPtr<nsIHTTPEventSink> sink;
+        m_pConnection->GetEventSink(getter_AddRefs(sink));
+        if (sink) {
+            sink->OnHeadersAvailable(m_pConnection);
         }
 
         // Check for any modules that want to receive headers once they've arrived.