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

Commit d70c636a authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Fix issue which Ion handle are free twice."

parents cb5c02da 1fc35e65

media/jni/android_media_tv_Tuner.cpp

+38 −27
Original line number Diff line number Diff line
@@ -337,7 +337,10 @@ jobject MediaEvent::getLinearBlock() { 
    }

    mIonHandle = new C2HandleIon(dup(mAvHandle->data[0]), mDataLength);

    std::shared_ptr<C2LinearBlock> block = _C2BlockFactory::CreateLinearBlock(mIonHandle);



    if (block != nullptr) {

        // CreateLinearBlock delete mIonHandle after it create block successfully.

        // ToDo: coordinate who is response to delete mIonHandle

        mIonHandle = NULL;

        JNIEnv *env = AndroidRuntime::getJNIEnv();

        std::unique_ptr<JMediaCodecLinearBlock> context{new JMediaCodecLinearBlock};

        context->mBlock = block;
@@ -364,6 +367,14 @@ jobject MediaEvent::getLinearBlock() { 
        mLinearBlockObj = env->NewWeakGlobalRef(linearBlock);

        mAvHandleRefCnt++;

        return mLinearBlockObj;

    } else {

        native_handle_close(const_cast<native_handle_t*>(

                    reinterpret_cast<const native_handle_t*>(mIonHandle)));

        native_handle_delete(const_cast<native_handle_t*>(

                    reinterpret_cast<const native_handle_t*>(mIonHandle)));

        mIonHandle = NULL;

        return NULL;

    }

}



uint64_t MediaEvent::getAudioHandle() {