aboutsummaryrefslogtreecommitdiff
path: root/src/client
diff options
context:
space:
mode:
authormark@chromium.org <mark@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e>2012-07-03 18:31:08 +0000
committermark@chromium.org <mark@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e>2012-07-03 18:31:08 +0000
commit7db2fb7b93259c8b3a3a22addfb9125a474fdd5c (patch)
tree206564c04a41e5c21a99bae708757c699dd6a1ea /src/client
parentAllow the minidump writer to collect handle data so that resulting dump contains (diff)
downloadbreakpad-7db2fb7b93259c8b3a3a22addfb9125a474fdd5c.tar.xz
Do not fail crash dump generation if handle operations tracing is not enabled
and yet the fatal exception was STATUS_INVALID_HANDLE. BUG=131699 Patch by Alex Pakhunov <alexeypa@chromium.org> Review URL: https://breakpad.appspot.com/409003/ git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@980 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/client')
-rw-r--r--src/client/windows/crash_generation/minidump_generator.cc3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/client/windows/crash_generation/minidump_generator.cc b/src/client/windows/crash_generation/minidump_generator.cc
index 84922940..fe4937aa 100644
--- a/src/client/windows/crash_generation/minidump_generator.cc
+++ b/src/client/windows/crash_generation/minidump_generator.cc
@@ -149,7 +149,8 @@ bool HandleTraceData::CollectHandleData(
AvrfResourceHandleTrace,
&RecordHandleOperations,
this) != ERROR_SUCCESS) {
- return false;
+ // The handle tracing must have not been enabled.
+ return true;
}
// Now that |handle_| is initialized, purge all irrelevant operations.