commit 95c51b3c3765124adf367047caa56f16bfb6eb6f
author Stephen Hines <srhines@google.com> Wed Nov 26 01:24:13 2014 -0800
committer Stephen Hines <srhines@google.com> Wed Nov 26 01:24:13 2014 -0800
tree fd22336d36ea556e4cf0a504b35d3da57a983e25
parent af6dad6cbb6685f92d7274f9e8c45baa98232c79

Add a missing SHARED_LOCKS_REQUIRED(mutator_lock_).

This was caught by Clang 3.6 using -Wthread-safety-analysis.

Change-Id: If2f25331f111ba6c67570e5aece1fba38f714d05

runtime/jni_internal.cc

diff --git a/runtime/jni_internal.cc b/runtime/jni_internal.cc
index 1dcfcab..4797e69 100644
--- a/runtime/jni_internal.cc
+++ b/runtime/jni_internal.cc
@@ -566,7 +566,8 @@
     return soa.AddLocalReference<jobject>(decoded_obj);
   }
 
-  static void DeleteLocalRef(JNIEnv* env, jobject obj) {
+  static void DeleteLocalRef(JNIEnv* env, jobject obj)
+      SHARED_LOCKS_REQUIRED(Locks::mutator_lock_) {
     if (obj == nullptr) {
       return;
     }