From 72bafd378d836d9a1fe81a1f26c91d51b2a7892b Mon Sep 17 00:00:00 2001 From: doshimun Date: Thu, 15 May 2008 04:12:34 +0000 Subject: Microsoft broke PROCESS_ALL_ACCESS when we build with WINVER=0x600. See http://nsylvain.blogspot.com/2008/01/winverwin32winnt-mayhem.html for details (second half of the post). Changing PROCESS_ALL_ACCESS to GENERIC_ALL in ClientInfo class when opening the client process handle on the OOP server side. A minor change to pipe name in the test app also. git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@274 4c0a9323-5329-0410-9bdc-e9ce6186880e --- src/client/windows/crash_generation/client_info.cc | 2 +- src/client/windows/tests/crash_generation_app/crash_generation_app.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'src/client/windows') diff --git a/src/client/windows/crash_generation/client_info.cc b/src/client/windows/crash_generation/client_info.cc index 6d602744..7b90561d 100644 --- a/src/client/windows/crash_generation/client_info.cc +++ b/src/client/windows/crash_generation/client_info.cc @@ -54,7 +54,7 @@ ClientInfo::ClientInfo(CrashGenerationServer* crash_server, } bool ClientInfo::Initialize() { - process_handle_ = OpenProcess(PROCESS_ALL_ACCESS, FALSE, pid_); + process_handle_ = OpenProcess(GENERIC_ALL, FALSE, pid_); if (!process_handle_) { return false; } diff --git a/src/client/windows/tests/crash_generation_app/crash_generation_app.cc b/src/client/windows/tests/crash_generation_app/crash_generation_app.cc index 7ed1021b..a3c560bd 100644 --- a/src/client/windows/tests/crash_generation_app/crash_generation_app.cc +++ b/src/client/windows/tests/crash_generation_app/crash_generation_app.cc @@ -35,7 +35,7 @@ namespace google_breakpad { const int kMaxLoadString = 100; -const wchar_t kPipeName[] = L"\\\\.\\pipe\\GoogleCrashServices\\S-1-5-21-39260824-743453154-142223018-195347"; +const wchar_t kPipeName[] = L"\\\\.\\pipe\\BreakpadCrashServices\\TestServer"; const DWORD kEditBoxStyles = WS_CHILD | WS_VISIBLE | -- cgit v1.2.1