diff options
author | Mathieu Chartier <mathieuc@google.com> | 2015-07-30 11:41:04 -0700 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2015-07-30 11:41:04 -0700 |
commit | 2ada67b3145d38b352297bdb9d0546f58882e208 (patch) | |
tree | cd81388615831741556f4e1cf3ca1ae9910e0968 | |
parent | 8f96df846403703e14016590b4c0c3af870561d9 (diff) | |
download | android_art-2ada67b3145d38b352297bdb9d0546f58882e208.tar.gz android_art-2ada67b3145d38b352297bdb9d0546f58882e208.tar.bz2 android_art-2ada67b3145d38b352297bdb9d0546f58882e208.zip |
Don't dump stack when deleting an invalid local ref
Bug: 21724724
Change-Id: Ifa99df9ee9f3265038a0c0f7944e834e42c498d4
-rw-r--r-- | runtime/indirect_reference_table.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/runtime/indirect_reference_table.cc b/runtime/indirect_reference_table.cc index 55632bad55..d54c1a653e 100644 --- a/runtime/indirect_reference_table.cc +++ b/runtime/indirect_reference_table.cc @@ -29,6 +29,8 @@ namespace art { +static constexpr bool kDumpStackOnNonLocalReference = false; + template<typename T> class MutatorLockedDumpable { public: @@ -184,7 +186,9 @@ bool IndirectReferenceTable::Remove(uint32_t cookie, IndirectRef iref) { if (env->check_jni) { ScopedObjectAccess soa(self); LOG(WARNING) << "Attempt to remove non-JNI local reference, dumping thread"; - self->Dump(LOG(WARNING)); + if (kDumpStackOnNonLocalReference) { + self->Dump(LOG(WARNING)); + } } return true; } |