commit 7b98edabb61c6f06199fc22ffc9ebff580dfefc3
author Joshua Peraza <jperaza@chromium.org> Mon Aug 06 10:27:13 2018 -0700
committer Joshua Peraza <jperaza@chromium.org> Mon Aug 06 18:02:41 2018 +0000
tree f651edfc9a443fb7c24c82b651b6759dc2bfab5b
parent f12b6897897fb5b7e74437371fdf2b36a36f5661

Set new ARM64 context flags

Change-Id: I4749459ec37b076b226fa734824380a7254f1064
Reviewed-on: https://chromium-review.googlesource.com/1163826
Reviewed-by: Mark Mentovai <mark@chromium.org>

src/processor/minidump.cc

diff --git a/src/processor/minidump.cc b/src/processor/minidump.cc
index c86a813..d8a45dc 100644
--- a/src/processor/minidump.cc
+++ b/src/processor/minidump.cc
@@ -753,18 +753,10 @@
         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;