Fix the crash. IInternetBindInfo BINDSTRING_XDR_ORIGIN bug workaround.

src/plugin/PluginWbPassThrough.cpp

 #include "PluginStdAfx.h"
 
 #include "PluginWbPassThrough.h"
 #include "PluginClient.h"
 #include "PluginClientFactory.h"
 #ifdef SUPPORT_FILTER
 #include "PluginFilter.h"
 #endif
 #include "PluginSettings.h"
 #include "PluginClass.h"
@@ skipping 125 matching lines @@
   if (pOIBindInfo)
   {
     ULONG resLen = 0;
     LPOLESTR mime = 0;
     HRESULT hr = pOIBindInfo->GetBindString(BINDSTRING_ACCEPT_MIMES, &mime, 1, &resLen);
     if (mime && resLen > 0)
     {
       mimeType.SetString(mime);
     }
     LPOLESTR bindToObject = 0;
     hr = pOIBindInfo->GetBindString(BINDSTRING_FLAG_BIND_TO_OBJECT, &bindToObject, 1, &resLen);

[Felix Dahlke, 2013/08/09 07:12:33]

The hr we assign here is now unused.

     LPOLESTR domainRetrieved = 0;
     if (resLen == 0 || wcscmp(bindToObject, L"FALSE") == 0)
     {   
       hr = pOIBindInfo->GetBindString(BINDSTRING_XDR_ORIGIN, &domainRetrieved, 1, &resLen);
-    }
-    if ((hr == S_OK) && domainRetrieved && (resLen > 0))
-    {
-      boundDomain.SetString(domainRetrieved);
-      // Remove protocol
-      int pos = boundDomain.Find(L"://");
-      if (pos > 0)
+      if ((hr == S_OK) && domainRetrieved && (resLen > 0))
       {
-        boundDomain = boundDomain.Mid(pos + 3);
-      }
-      // Remove port
-      pos = boundDomain.Find(L":");
-      if (pos > 0)
-      {
-        boundDomain.Left(pos);
+        boundDomain.SetString(domainRetrieved);
+        // Remove protocol
+        int pos = boundDomain.Find(L"://");
+        if (pos > 0)
+        {
+          boundDomain = boundDomain.Mid(pos + 3);
+        }
+        // Remove port
+        pos = boundDomain.Find(L":");
+        if (pos > 0)
+        {
+          boundDomain.Left(pos);
+        }
       }
     }
   }
 
   CString cookie;
   ULONG len1 = 2048;
   ULONG len2 = 2048;
 
 #ifdef SUPPORT_FILTER
   int contentType = CFilter::contentTypeAny;
@@ skipping 191 matching lines @@
   CComPtr<IHttpNegotiate> spHttpNegotiate;
   QueryServiceFromClient(&spHttpNegotiate);
 
   return spHttpNegotiate ? spHttpNegotiate->OnResponse(dwResponseCode, szResponseHeaders, szRequestHeaders, pszAdditionalRequestHeaders) : S_OK;
 }
 
 STDMETHODIMP WBPassthruSink::ReportProgress(ULONG ulStatusCode, LPCWSTR szStatusText)
 {
   return m_spInternetProtocolSink ? m_spInternetProtocolSink->ReportProgress(ulStatusCode, szStatusText) : S_OK;
 }