Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "audio effect JNI: fix global ref leak." into mnc-dev

Browse Source
This commit is contained in:
Eric Laurent
2015-06-19 16:46:29 +00:00
committed by Android (Google) Code Review
parent 8f4ccd6c96 077999844f
commit 1f98e79abd
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
media/jni/audioeffect/android_media_AudioEffect.cpp
View File

@@ -472,6 +472,8 @@ static void android_media_AudioEffect_native_release(JNIEnv *env, jobject thiz)
if (lpJniStorage) { if (lpJniStorage) {
ALOGV("deleting pJniStorage: %p\n", lpJniStorage); ALOGV("deleting pJniStorage: %p\n", lpJniStorage);
env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_class);
env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_ref);
delete lpJniStorage; delete lpJniStorage;
} }
} }

2
media/jni/audioeffect/android_media_Visualizer.cpp
View File

@@ -450,6 +450,8 @@ static void android_media_visualizer_native_release(JNIEnv *env, jobject thiz)
if (lpJniStorage) { if (lpJniStorage) {
ALOGV("deleting pJniStorage: %p\n", lpJniStorage); ALOGV("deleting pJniStorage: %p\n", lpJniStorage);
env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_class);
env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_ref);
delete lpJniStorage; delete lpJniStorage;
} }
} }