diff options
| author | Joshua Peraza <jperaza@chromium.org> | 2018-08-06 10:27:13 -0700 |
|---|---|---|
| committer | Joshua Peraza <jperaza@chromium.org> | 2018-08-06 18:02:41 +0000 |
| commit | 7b98edabb61c6f06199fc22ffc9ebff580dfefc3 (patch) | |
| tree | f651edfc9a443fb7c24c82b651b6759dc2bfab5b | |
| parent | processor: Fix memory leak after 88d8114f (diff) | |
| download | breakpad-7b98edabb61c6f06199fc22ffc9ebff580dfefc3.tar.xz | |
Set new ARM64 context flags
Change-Id: I4749459ec37b076b226fa734824380a7254f1064
Reviewed-on: https://chromium-review.googlesource.com/1163826
Reviewed-by: Mark Mentovai <mark@chromium.org>
| -rw-r--r-- | src/processor/minidump.cc | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/src/processor/minidump.cc b/src/processor/minidump.cc index c86a813f..d8a45dc2 100644 --- a/src/processor/minidump.cc +++ b/src/processor/minidump.cc @@ -753,18 +753,10 @@ bool MinidumpContext::Read(uint32_t expected_size) { Swap(&context_arm64->float_save.regs[fpr_index]); } } - SetContextFlags(static_cast<uint32_t>(context_arm64->context_flags)); - - // Check for data loss when converting context flags from uint64_t into - // uint32_t - if (static_cast<uint64_t>(GetContextFlags()) != - context_arm64->context_flags) { - BPLOG(ERROR) << "Data loss detected when converting ARM64 context_flags"; - return false; - } scoped_ptr<MDRawContextARM64> new_context(new MDRawContextARM64()); ConvertOldARM64Context(*context_arm64.get(), new_context.get()); + SetContextFlags(new_context->context_flags); SetContextARM64(new_context.release()); } else { uint32_t context_flags; |