| commit | 5809caff94f4b05a59f887ad65b9e88f2ab35e0c | [log] [tgz] |
|---|---|---|
| author | David Li <dvdli@google.com> | Mon Mar 22 03:10:37 2021 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Mon Mar 22 03:10:37 2021 +0000 |
| tree | 1ea017f7606af9c0133255a7582c5d17000a3775 | |
| parent | 022c080e62036847e7fcac1ab4ee40417318cc1f [diff] | |
| parent | 06b7deb347af6e7665b3580fe950d7fdb26f759f [diff] |
Merge "fix null dereference on aeli" am: 00112c5a0a am: 06b7deb347 Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/1646947 Change-Id: I94372e2f4144639039dabb39c6bf36e0ffc6b435
diff --git a/media/libeffects/proxy/EffectProxy.cpp b/media/libeffects/proxy/EffectProxy.cpp index c010d68..be9f8c0 100644 --- a/media/libeffects/proxy/EffectProxy.cpp +++ b/media/libeffects/proxy/EffectProxy.cpp
@@ -116,6 +116,16 @@ pContext->sube[SUB_FX_OFFLOAD] = sube[1]; pContext->desc[SUB_FX_OFFLOAD] = desc[1]; pContext->aeli[SUB_FX_OFFLOAD] = aeli[1]; + } else { + ALOGE("Both effects have (or don't have) EFFECT_FLAG_HW_ACC_TUNNEL flag"); + delete[] sube; + delete[] desc; + delete[] aeli; + delete[] pContext->sube; + delete[] pContext->desc; + delete[] pContext->aeli; + delete pContext; + return -EINVAL; } delete[] desc; delete[] aeli;