commit 5a832258726e3dfa148a1ddf63e987ca0cbab17a
author Mathieu Chartier <mathieuc@google.com> Thu Mar 28 07:31:16 2019 -0700
committer Mathieu Chartier <mathieuc@google.com> Thu Mar 28 16:24:36 2019 +0000
tree 3a8f5deece4fe5992a3bb79d362b59d9ddfdf754
parent 08417350b78b975a7fed36d3360e0f926abae385

Address comments for aog/934603

Test: TH
Change-Id: I6b54cfbaa49d320c3c9e97ebaeeb97e285f9bf43

runtime/thread_list.cc

diff --git a/runtime/thread_list.cc b/runtime/thread_list.cc
index f07dca1..ed6b2c9 100644
--- a/runtime/thread_list.cc
+++ b/runtime/thread_list.cc
@@ -203,6 +203,8 @@
  public:
   DumpCheckpoint(std::ostream* os, bool dump_native_stack)
       : os_(os),
+        // Avoid verifying count in case a thread doesn't end up passing through the barrier.
+        // This avoids a SIGABRT that would otherwise happen in the destructor.
         barrier_(0, /*verify_count_on_shutdown=*/false),
         backtrace_map_(dump_native_stack ? BacktraceMap::Create(getpid()) : nullptr),
         dump_native_stack_(dump_native_stack) {