Fix crash on file upload

src/plugin/ProtocolImpl.inl

 // Copyright 2007 Igor Tandetnik
 // 
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //    http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 454 matching lines @@
     ATLASSERT(pOIBindInfo != 0);
     ATLASSERT(pTargetProtocol != 0);
     if (!pOIProtSink || !pOIBindInfo || !pTargetProtocol)
     {
       return E_POINTER;
     }
 
     // This method should only be called once, and be the only source
     // of target interface pointers.
     ATLASSERT(m_spInternetProtocolSink == 0);
-    ATLASSERT(m_spInternetBindInfo == 0);
     ATLASSERT(m_spTargetProtocol == 0);
-    if (m_spInternetProtocolSink || m_spInternetBindInfo || m_spTargetProtocol)
+    if (m_spInternetProtocolSink || m_spTargetProtocol)
     {
       return E_UNEXPECTED;
     }
 
     ATLASSERT(m_spServiceProvider == 0);
 
     m_spInternetProtocolSink = pOIProtSink;
-    m_spInternetBindInfo = pOIBindInfo;
     m_spTargetProtocol = pTargetProtocol;
     return S_OK;
   }
 
   inline void IInternetProtocolSinkImpl::ReleaseAll()
   {
     m_spInternetProtocolSink.Release();
     m_spServiceProvider.Release();
-    m_spInternetBindInfo.Release();
     m_spTargetProtocol.Release();
   }
 
   inline IServiceProvider* IInternetProtocolSinkImpl::GetClientServiceProvider()
   {
     return m_spServiceProvider;
   }
 
   inline HRESULT IInternetProtocolSinkImpl::QueryServiceFromClient(
     REFGUID guidService, REFIID riid, void** ppvObject)
@@ skipping 74 matching lines @@
     /* [in] */ REFGUID guidService,
     /* [in] */ REFIID riid,
     /* [out] */ void** ppvObject)
   {
     ATLASSERT(m_spServiceProvider != 0);
     return m_spServiceProvider ?
       m_spServiceProvider->QueryService(guidService, riid, ppvObject) :
       E_UNEXPECTED;
   }
 
-  // IInternetBindInfo
-  inline STDMETHODIMP IInternetProtocolSinkImpl::GetBindInfo(
-    /* [out] */ DWORD *grfBINDF,
-    /* [in, out] */ BINDINFO *pbindinfo)
-  {
-    ATLASSERT(m_spInternetBindInfo != 0);
-    return m_spInternetBindInfo ?
-      m_spInternetBindInfo->GetBindInfo(grfBINDF, pbindinfo) :
-      E_UNEXPECTED;
-  }
-
-  inline STDMETHODIMP IInternetProtocolSinkImpl::GetBindString(
-    /* [in] */ ULONG ulStringType,
-    /* [in, out] */ LPOLESTR *ppwzStr,
-    /* [in] */ ULONG cEl,
-    /* [in, out] */ ULONG *pcElFetched)
-  {
-    ATLASSERT(m_spInternetBindInfo != 0);
-    return m_spInternetBindInfo ?
-      m_spInternetBindInfo->GetBindString(ulStringType, ppwzStr, cEl,
-      pcElFetched) :
-    E_UNEXPECTED;
-  }
-
   // ===== CInternetProtocolSinkWithSP =====
 
   template <class T, class ThreadModel>
   inline HRESULT CInternetProtocolSinkWithSP<T, ThreadModel>::OnStart(
     LPCWSTR szUrl, IInternetProtocolSink *pOIProtSink,
     IInternetBindInfo *pOIBindInfo,     DWORD grfPI, DWORD dwReserved,
     IInternetProtocol* pTargetProtocol)
   {
     ATLASSERT(m_spServiceProvider == 0);
     if (m_spServiceProvider)
@@ skipping 101 matching lines @@
     ATLASSERT(pTargetProtocol);
     HRESULT hr = m_internetSink.OnStart(szUrl, pOIProtSink, pOIBindInfo,
       grfPI, dwReserved, pTargetProtocol);
 
     //We return INET_E_REDIRECT_FAILED in case we have iframe blocking.
     if (hr == INET_E_REDIRECT_FAILED)
     {
       return S_OK;
     }
     CComPtr<IInternetProtocolSink> spSink;
-    CComPtr<IInternetBindInfo> spBindInfo;
     if (SUCCEEDED(hr))
     {
       hr = m_internetSink.QueryInterface(&spSink);
       ATLASSERT(SUCCEEDED(hr) && spSink != 0);
     }
     if (SUCCEEDED(hr))
     {
-      hr = m_internetSink.QueryInterface(&spBindInfo);
-      ATLASSERT(SUCCEEDED(hr) && spBindInfo != 0);
-    }
-    if (SUCCEEDED(hr))
-    {
-      hr = pTargetProtocol->Start(szUrl, spSink, spBindInfo, grfPI,
-        dwReserved);
-    } 
-    else
-    {
-
+      hr = pTargetProtocol->Start(szUrl, spSink, pOIBindInfo, grfPI, dwReserved);
     }
     return hr;
   }
   template <class Sink>
   inline HRESULT CustomSinkStartPolicy<Sink>::Read(/* [in, out] */ void *pv,/* [in] */ ULONG cb,/* [out] */ ULONG *pcbRead)
   {
     return m_internetSink.Read(pv, cb, pcbRead);
   }
 
   // ===== CInternetProtocol =====
@@ skipping 23 matching lines @@
   inline STDMETHODIMP CInternetProtocol<StartPolicy, ThreadModel>::Read(
     /* [in, out] */ void *pv,
     /* [in] */ ULONG cb,
     /* [out] */ ULONG *pcbRead)
   {
     return StartPolicy::Read(pv, cb, pcbRead);
   }
 } // end namespace PassthroughAPP
 
 #endif // PASSTHROUGHAPP_PROTOCOLIMPL_INL