Bug 552864 part 1 - Move DLL blocklist in XRE, and inline NS_SetDllDirectory and environment sanitization, which now needs to be called manually. r=ehsan,r=bsmedberg

Author: glandium

browser/app/nsBrowserApp.cpp

@@ -54,8 +54,6 @@
 #include "nsStringGlue.h"
 
 #ifdef XP_WIN
-// we want to use the DLL blocklist if possible
-#define XRE_WANT_DLL_BLOCKLIST
 // we want a wmain entry point
 #include "nsWindowsWMain.cpp"
 #endif
@@ -110,6 +108,10 @@ int main(int argc, char* argv[])
 {
   ScopedLogging log;
 
+#ifdef XRE_HAS_DLL_BLOCKLIST
+  XRE_SetupDllBlocklist();
+#endif
+
   nsCOMPtr<nsILocalFile> appini;
   nsresult rv = XRE_GetBinaryPath(argv[0], getter_AddRefs(appini));
   if (NS_FAILED(rv)) {

dom/plugins/ipc/PluginProcessChild.cpp

@@ -146,6 +146,7 @@ PluginProcessChild::Init()
         protectCurrentDirectory = false;
     }
     if (protectCurrentDirectory) {
+        SanitizeEnvironmentVariables();
         NS_SetDllDirectory(L"");
     }
 

ipc/app/MozillaRuntimeMain.cpp

@@ -73,6 +73,7 @@ main(int argc, char* argv[])
     // avoid it for unsupported plugins.  See PluginProcessChild::Init for
     // the details.
     if (proctype != GeckoProcessType_Plugin) {
+        mozilla::SanitizeEnvironmentVariables();
         mozilla::NS_SetDllDirectory(L"");
     }
 #endif

toolkit/xre/Makefile.in

@@ -92,6 +92,7 @@ endif
 
 ifeq ($(MOZ_WIDGET_TOOLKIT),windows)
 CPPSRCS += nsNativeAppSupportWin.cpp
+CPPSRCS += nsWindowsDllBlocklist.cpp
 DEFINES += -DWIN32_LEAN_AND_MEAN -DUNICODE -D_UNICODE
 EXPORTS = nsWindowsDllInterceptor.h
 else

toolkit/xre/nsWindowsDllBlocklist.cpp

@@ -40,6 +40,12 @@
 
 #include <stdio.h>
 
+#ifdef XRE_WANT_DLL_BLOCKLIST
+#define XRE_SetupDllBlocklist SetupDllBlocklist
+#else
+#include "nsXULAppAPI.h"
+#endif
+
 #include "nsAutoPtr.h"
 
 #include "prlog.h"
@@ -208,7 +214,7 @@ patched_LdrLoadDll (PWCHAR filePath, PULONG flags, PUNICODE_STRING moduleFileNam
 WindowsDllInterceptor NtDllIntercept;
 
 void
-SetupDllBlocklist()
+XRE_SetupDllBlocklist()
 {
   NtDllIntercept.Init("ntdll.dll");
 

toolkit/xre/nsWindowsWMain.cpp

@@ -76,6 +76,7 @@ FreeAllocStrings(int argc, char **argv)
 int wmain(int argc, WCHAR **argv)
 {
 #ifndef XRE_DONT_PROTECT_DLL_LOAD
+  mozilla::SanitizeEnvironmentVariables();
   mozilla::NS_SetDllDirectory(L"");
 #endif
 

xpcom/base/Makefile.in

@@ -98,7 +98,6 @@ EXPORTS_mozilla = \
 	$(NULL)
 
 ifeq (windows,$(MOZ_WIDGET_TOOLKIT))
-CPPSRCS += nsSetDllDirectory.cpp
 CPPSRCS += nsCrashOnException.cpp
 endif
 

xpcom/base/nsSetDllDirectory.cpp

@@ -43,13 +43,48 @@
 #error This file only makes sense on Windows.
 #endif
 
+#include <windows.h>
 #include <nscore.h>
+#include <stdlib.h>
 
 namespace mozilla {
 
+static void SanitizeEnvironmentVariables()
+{
+  DWORD bufferSize = GetEnvironmentVariableW(L"PATH", NULL, 0);
+  if (bufferSize) {
+    wchar_t* originalPath = new wchar_t[bufferSize];
+    if (bufferSize - 1 == GetEnvironmentVariableW(L"PATH", originalPath, bufferSize)) {
+      bufferSize = ExpandEnvironmentStringsW(originalPath, NULL, 0);
+      if (bufferSize) {
+        wchar_t* newPath = new wchar_t[bufferSize];
+        if (ExpandEnvironmentStringsW(originalPath,
+                                      newPath,
+                                      bufferSize)) {
+          SetEnvironmentVariableW(L"PATH", newPath);
+        }
+        delete[] newPath;
+      }
+    }
+    delete[] originalPath;
+  }
+}
+
 // Sets the directory from which DLLs can be loaded if the SetDllDirectory OS
 // API is available.
-XPCOM_API(void) NS_SetDllDirectory(const WCHAR *aDllDirectory);
+// You must call SanitizeEnvironmentVariables before this function when calling
+// it the first time.
+static inline void NS_SetDllDirectory(const WCHAR *aDllDirectory)
+{
+  typedef BOOL
+  (WINAPI *pfnSetDllDirectory) (LPCWSTR);
+  pfnSetDllDirectory setDllDirectory = nsnull;
+  setDllDirectory = reinterpret_cast<pfnSetDllDirectory>
+      (GetProcAddress(GetModuleHandleW(L"kernel32.dll"), "SetDllDirectoryW"));
+  if (setDllDirectory) {
+    setDllDirectory(aDllDirectory);
+  }
+}
 
 }
 

xpcom/base/nsSetDllDirectory.h

@@ -562,4 +562,11 @@ XRE_API(bool,
 
 XRE_API(void,
         XRE_InstallX11ErrorHandler, ())
+
+#if defined(_MSC_VER) && defined(_M_IX86)
+#define XRE_HAS_DLL_BLOCKLIST
+XRE_API(void,
+        XRE_SetupDllBlocklist, ())
+#endif
+
 #endif // _nsXULAppAPI_h__