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

Refine DeleteLocalRef invocation

Browse Source 
Bug: 254803449
Test: atest android.media.tv.tuner.cts on cf_x86_tv-userdebug
Change-Id: I3ade90126cc4cfd56e8f9f2547f351e7daf2b441
This commit is contained in:
Ray Chin
2022-11-17 19:01:38 +08:00
parent 91f9f9d7a0
commit 0b8d90d23e
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
media/jni/android_media_tv_Tuner.cpp
View File

@@ -380,6 +380,7 @@ void LnbClientCallbackImpl::onDiseqcMessage(const vector<uint8_t> &diseqcMessage
gFields.onLnbDiseqcMessageID,
array);
env->DeleteLocalRef(lnb);
env->DeleteLocalRef(array);
} else {
ALOGE("LnbClientCallbackImpl::onDiseqcMessage:"
"Lnb object has been freed. Ignoring callback.");
@@ -848,6 +849,7 @@ void FilterClientCallbackImpl::getRestartEvent(jobjectArray &arr, const int size
const int32_t &startId = event.get<DemuxFilterEvent::Tag::startId>();
jobject obj = env->NewObject(eventClazz, eventInit, startId);
env->SetObjectArrayElement(arr, size, obj);
env->DeleteLocalRef(obj);
}
void FilterClientCallbackImpl::onFilterEvent(const vector<DemuxFilterEvent> &events) {
@@ -2181,7 +2183,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
env->NewObject(longClazz, initLong,
static_cast<long>(s.get<FrontendStatus::Tag::innerFec>()));
env->SetObjectField(statusObj, field, newLongObj);
env->DeleteLocalRef(longClazz);
env->DeleteLocalRef(newLongObj);
break;
}
@@ -2347,7 +2348,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
}
env->SetObjectField(statusObj, field, valObj);
env->DeleteLocalRef(plpClazz);
env->DeleteLocalRef(valObj);
break;
}
@@ -2758,7 +2758,6 @@ jobject JTuner::getFrontendStatus(jintArray types) {
}
env->SetObjectField(statusObj, field, valObj);
env->DeleteLocalRef(plpClazz);
env->DeleteLocalRef(valObj);
break;
}