Fix named pipe security on Windows XP

src/plugin/AdblockPlusClient.cpp

 #include "PluginStdAfx.h"
 #include "PluginSettings.h"
 #include "PluginSystem.h"
 #include "PluginFilter.h"
 #include "PluginClientFactory.h"
 #include "PluginMutex.h"
 #include "PluginClass.h"
 
 #include "AdblockPlusClient.h"
 
@@ skipping 24 matching lines @@
           GetTokenInformation(token, TokenAppContainerSid, acs, length, &length);
         }
         else
         {
           throw std::runtime_error("Out of memory");
         }
     }
 
     BOOL createProcRes = 0;
     // Running inside AppContainer or in Windows XP
-    if ((acs != NULL && acs->TokenAppContainer != NULL) || (!IsWindowsVistaOrLater()))

[Felix Dahlke, 2014/06/24 12:25:05]

Redundant parentheses around |!IsWindowsVistaOrLater()|

+    if ((acs != NULL && acs->TokenAppContainer != NULL) || !IsWindowsVistaOrLater())
     {
       // We need to break out from AppContainer. Launch with default security - registry entry will eat the user prompt
       // See http://msdn.microsoft.com/en-us/library/bb250462(v=vs.85).aspx#wpm_elebp
       createProcRes = CreateProcessW(engineExecutablePath.c_str(), params.GetBuffer(params.GetLength() + 1),
                               0, 0, false, 0, 0, 0, (STARTUPINFOW*)&startupInfo, &processInformation);
     }
     else
     {
       // Launch with Low Integrity explicitly
       HANDLE newToken;
@@ skipping 463 matching lines @@
   Communication::OutputBuffer request;
   request << Communication::PROC_GET_HOST << ToUtf8String(url);
 
   Communication::InputBuffer response;
   if (!CallEngine(request, response)) 
     return L"";
   std::string host;
   response >> host;
   return ToUtf16String(host);
 }