commit c56f3e62766941004d685e102627d9904bb65d97
author Yanqiang Fan <yanqiang.fan@mediatek.com> Tue Sep 28 16:54:07 2021 +0800
committer chuanghua zhao <Chuanghua.Zhao@mediatek.com> Wed Jan 12 03:19:34 2022 +0000
tree 52245ba2564a879d45f909b1525a9402386ca677
parent 07f0cdfd5bf42367c7057a6ecc0e6689ea1488b5

fix mediaserver fatal error

app process died caused GraphicbufferSource in app process gone.
then call configure() through aidl interface in
GraphicBufferSourceWrapper::connect() will crash, as Failed HIDL return
status not checked.

add getStatus check the return status Error
EX_TRANSACTION_FAILED:DEAD_OBJECT

Bug: 202783241
Test: manually/monkey test

Change-Id: I0d7ff79e86ee00f105856615f2d6cc72c779fa34

media/codec2/sfplugin/CCodec.cpp

diff --git a/media/codec2/sfplugin/CCodec.cpp b/media/codec2/sfplugin/CCodec.cpp
index 5389339..2c00594 100644
--- a/media/codec2/sfplugin/CCodec.cpp
+++ b/media/codec2/sfplugin/CCodec.cpp
@@ -212,9 +212,8 @@
                 (OMX_INDEXTYPE)OMX_IndexParamConsumerUsageBits,
                 &usage, sizeof(usage));
 
-        mSource->configure(
-                mOmxNode, static_cast<hardware::graphics::common::V1_0::Dataspace>(mDataSpace));
-        return OK;
+        return GetStatus(mSource->configure(
+                mOmxNode, static_cast<hardware::graphics::common::V1_0::Dataspace>(mDataSpace)));
     }
 
     void disconnect() override {