Make Crashpad use the user data dir, rather than always default location

chrome_elf/chrome_elf_main.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome_elf/chrome_elf_main.h"
 
 #include <assert.h>
 #include <windows.h>
 
 #include "chrome/install_static/install_details.h"
 #include "chrome/install_static/install_util.h"
 #include "chrome/install_static/product_install_details.h"
+#include "chrome/install_static/user_data_dir.h"
 #include "chrome_elf/blacklist/blacklist.h"
 #include "chrome_elf/crash/crash_helper.h"
 
 // This function is a temporary workaround for https://crbug.com/655788. We
 // need to come up with a better way to initialize crash reporting that can
 // happen inside DllMain().
 void SignalInitializeCrashReporting() {
   if (!elf_crash::InitializeCrashReporting()) {
 #ifdef _DEBUG
     assert(false);
 #endif  // _DEBUG
   }
 }
 
 void SignalChromeElf() {
   blacklist::ResetBeacon();
 }
 
+extern "C" void GetUserDataDirectoryThunk(wchar_t* user_data_dir,
+                                          size_t user_data_dir_length,
+                                          wchar_t* invalid_user_data_dir,
+                                          size_t invalid_user_data_dir_length) {
+  std::wstring user_data_dir_str, invalid_user_data_dir_str;
+  bool ret = install_static::GetUserDataDirectory(&user_data_dir_str,
+                                                  &invalid_user_data_dir_str);
+  assert(ret);
+  wcscpy_s(user_data_dir, user_data_dir_length, user_data_dir_str.c_str());

[grt (UTC plus 2), 2016/11/23 09:47:37]

looks like this is counting on the crt's parameter validation to kick in if the
destination buffers are somehow too small. could you take a look at
base/win/process_startup_helper.cc to be sure that it will do the right thing
(comments there still reference breakpad -- is it correct)?

[scottmg, 2016/11/23 17:59:03]

I think process_startup_helper.cc is fine (other than the comment referring to
breakpad). wcsncpy_s(..., _TRUNCATE) is probably more sensible here though, so
switched to that.

+  wcscpy_s(invalid_user_data_dir, invalid_user_data_dir_length,
+           invalid_user_data_dir_str.c_str());
+}
+
 // Returns the payload for the ELF's InstallDetails instance. For use by
 // install_static::InstallDetails::InitializeFromPrimaryModule.
 extern "C" const install_static::InstallDetails::Payload*
 GetInstallDetailsPayload() {
   return install_static::InstallDetails::Get().GetPayload();
 }
 
 BOOL APIENTRY DllMain(HMODULE module, DWORD reason, LPVOID reserved) {
   if (reason == DLL_PROCESS_ATTACH) {
     install_static::InitializeProductDetailsForPrimaryModule();
 
-// CRT on initialization installs an exception filter which calls
-// TerminateProcess. We need to hook CRT's attempt to set an exception.
-// NOTE: Do not hook if ASan is present, or ASan will fail to install
-// its own unhandled exception filter.
 #if !defined(ADDRESS_SANITIZER)
+    // CRT on initialization installs an exception filter which calls
+    // TerminateProcess. We need to hook CRT's attempt to set an exception.
+    // NOTE: Do not hook if ASan is present, or ASan will fail to install
+    // its own unhandled exception filter.
     elf_crash::DisableSetUnhandledExceptionFilter();
 #endif  // !defined (ADDRESS_SANITIZER)
 
     install_static::InitializeProcessType();
 
     __try {
       blacklist::Initialize(false);  // Don't force, abort if beacon is present.
     } __except (elf_crash::GenerateCrashDump(GetExceptionInformation())) {
     }
   } else if (reason == DLL_PROCESS_DETACH) {
     elf_crash::ShutdownCrashReporting();
   }
   return TRUE;
 }