diff options
author | Mathieu Chartier <mathieuc@google.com> | 2014-11-15 13:07:39 -0800 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2014-11-17 10:16:34 -0800 |
commit | 41da59665556e8dd5601178185c68d48d645500b (patch) | |
tree | c1f8b5844e3243907a060f663c35370464328609 /runtime/jni_internal.cc | |
parent | 410709facb361c5503c7825f01edd264f4450666 (diff) | |
download | art-41da59665556e8dd5601178185c68d48d645500b.tar.gz art-41da59665556e8dd5601178185c68d48d645500b.tar.bz2 art-41da59665556e8dd5601178185c68d48d645500b.zip |
Fix reference leaks in ToReflectedMethod and ToReflectedField
Used ScopedLocalRef to fix, added regression tests.
Bug: 18396311
(cherry picked from commit 8c41753e5eda8322b4d992fe88855058f4c0c2e1)
Change-Id: I0ddfc20ac2384c90ff18586242c1f1d9a6d6eb6d
Diffstat (limited to 'runtime/jni_internal.cc')
-rw-r--r-- | runtime/jni_internal.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/runtime/jni_internal.cc b/runtime/jni_internal.cc index 3b0656e4c6..1dcfcabf9d 100644 --- a/runtime/jni_internal.cc +++ b/runtime/jni_internal.cc @@ -360,7 +360,7 @@ class JNI { ScopedObjectAccess soa(env); mirror::ArtMethod* m = soa.DecodeMethod(mid); CHECK(!kMovingMethods); - jobject art_method = soa.AddLocalReference<jobject>(m); + ScopedLocalRef<jobject> art_method(env, soa.AddLocalReference<jobject>(m)); jobject reflect_method; if (m->IsConstructor()) { reflect_method = env->AllocObject(WellKnownClasses::java_lang_reflect_Constructor); @@ -371,7 +371,7 @@ class JNI { return nullptr; } SetObjectField(env, reflect_method, - WellKnownClasses::java_lang_reflect_AbstractMethod_artMethod, art_method); + WellKnownClasses::java_lang_reflect_AbstractMethod_artMethod, art_method.get()); return reflect_method; } @@ -379,13 +379,13 @@ class JNI { CHECK_NON_NULL_ARGUMENT(fid); ScopedObjectAccess soa(env); mirror::ArtField* f = soa.DecodeField(fid); - jobject art_field = soa.AddLocalReference<jobject>(f); + ScopedLocalRef<jobject> art_field(env, soa.AddLocalReference<jobject>(f)); jobject reflect_field = env->AllocObject(WellKnownClasses::java_lang_reflect_Field); if (env->ExceptionCheck()) { return nullptr; } SetObjectField(env, reflect_field, - WellKnownClasses::java_lang_reflect_Field_artField, art_field); + WellKnownClasses::java_lang_reflect_Field_artField, art_field.get()); return reflect_field; } |