No issue - some cleanup in PluginWbPassThrough

src/plugin/PluginWbPassThrough.cpp

 #include "PluginStdAfx.h"
 
 #include "PluginWbPassThrough.h"
 #include "PluginClient.h"
 #include "PluginClientFactory.h"
 #include "PluginFilter.h"
 #include "PluginSettings.h"
 #include "PluginClass.h"
 #include "PluginSystem.h"
 #include <WinInet.h>
@@ skipping 256 matching lines @@
       )
     {
       return true;
     }
   }
   return false;
 }
 
 STDMETHODIMP WBPassthruSink::BeginningTransaction(LPCWSTR szURL, LPCWSTR szHeaders, DWORD dwReserved, LPWSTR* pszAdditionalHeaders)
 {
-  if (!szURL || !szHeaders)

[Oleksandr, 2014/11/19 02:34:59]

Why can't szHeaders be nullptr? I think it may happen, and since we don't care
we can only pass that to the original BeginningTransaction and let it deal with
this.

[sergei, 2014/11/19 11:05:02]

May be it can. I assumed that if it's [in] parameter and NULL is not mentioned
than it cannot be nullptr.
http://msdn.microsoft.com/en-us/library/ms775053(v=vs.85).aspx
Since we don't use it, it might be better to delegate it to the original
BeginningTransaction.

removed.

+  if (!szURL)
   {
     return E_POINTER;
   }
   std::wstring src = szURL;
   DEBUG_GENERAL(ToCString(src));
 
   std::string acceptHeader = ExtractHttpAcceptHeader(m_spTargetProtocol);
   m_contentType = GetContentTypeFromMimeType(ATL::CString(acceptHeader.c_str()));
 
   if (pszAdditionalHeaders)
@@ skipping 96 matching lines @@
   }
 
   return OnStart(szUrl, pOIProtSink, pOIBindInfo, grfPI, dwReserved, m_spInternetProtocol);
 }
 
 STDMETHODIMP WBPassthru::Read(/* [in, out] */ void *pv,/* [in] */ ULONG cb,/* [out] */ ULONG *pcbRead)
 {
   WBPassthruSink* pSink = GetSink();
   return pSink->OnRead(pv, cb, pcbRead);
 }