Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 83bf685e authored by Steve Kondik's avatar Steve Kondik Committed by Gerrit Code Review
Browse files

Merge "libaudioeffect_jni: GREF leak fix for JB" into jellybean

parents bd9fa42d ae0aa2bc

media/jni/audioeffect/android_media_AudioEffect.cpp

+5 −0
Original line number Diff line number Diff line
@@ -407,6 +407,8 @@ setup_failure: 
    env->SetIntField(thiz, fields.fidNativeAudioEffect, 0);



    if (lpJniStorage) {

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_class);

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_ref);

        delete lpJniStorage;

    }

    env->SetIntField(thiz, fields.fidJniData, 0);
@@ -439,6 +441,9 @@ static void android_media_AudioEffect_native_finalize(JNIEnv *env, jobject thiz 
    AudioEffectJniStorage* lpJniStorage = (AudioEffectJniStorage *)env->GetIntField(

        thiz, fields.fidJniData);

    if (lpJniStorage) {

        // delete global refs created in native_setup

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_class);

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.audioEffect_ref);

        ALOGV("deleting pJniStorage: %x\n", (int)lpJniStorage);

        delete lpJniStorage;

    }

media/jni/audioeffect/android_media_Visualizer.cpp

+5 −0
Original line number Diff line number Diff line
@@ -380,6 +380,8 @@ setup_failure: 
    env->SetIntField(thiz, fields.fidNativeVisualizer, 0);



    if (lpJniStorage) {

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_class);

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_ref);

        delete lpJniStorage;

    }

    env->SetIntField(thiz, fields.fidJniData, 0);
@@ -403,6 +405,9 @@ static void android_media_visualizer_native_finalize(JNIEnv *env, jobject thiz) 
    visualizerJniStorage* lpJniStorage = (visualizerJniStorage *)env->GetIntField(

        thiz, fields.fidJniData);

    if (lpJniStorage) {

        // delete global refs created in native_setup

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_class);

        env->DeleteGlobalRef(lpJniStorage->mCallbackData.visualizer_ref);

        ALOGV("deleting pJniStorage: %x\n", (int)lpJniStorage);

        delete lpJniStorage;

    }