diff options
| author | mark@chromium.org <mark@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2012-07-03 18:31:08 +0000 |
|---|---|---|
| committer | mark@chromium.org <mark@chromium.org@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2012-07-03 18:31:08 +0000 |
| commit | 7db2fb7b93259c8b3a3a22addfb9125a474fdd5c (patch) | |
| tree | 206564c04a41e5c21a99bae708757c699dd6a1ea /src | |
| parent | Allow the minidump writer to collect handle data so that resulting dump contains (diff) | |
| download | breakpad-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')
| -rw-r--r-- | src/client/windows/crash_generation/minidump_generator.cc | 3 |
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. |