This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: mediatek: vcodec: Handle invalid decoder vsi
Author:  Irui Wang <irui.w...@mediatek.com>
Date:    Thu Mar 21 09:47:54 2024 +0800

Handle an invalid decoder vsi in vpu_dec_init to ensure the decoder vsi
is valid for future use.

Fixes: 590577a4e525 ("[media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder 
Driver")

Signed-off-by: Irui Wang <irui.w...@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno 
<angelogioacchino.delre...@collabora.com>
Signed-off-by: Sebastian Fricke <sebastian.fri...@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c | 6 ++++++
 1 file changed, 6 insertions(+)

---

diff --git a/drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c 
b/drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c
index da6be556727b..145958206e38 100644
--- a/drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c
+++ b/drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c
@@ -233,6 +233,12 @@ int vpu_dec_init(struct vdec_vpu_inst *vpu)
        mtk_vdec_debug(vpu->ctx, "vdec_inst=%p", vpu);
 
        err = vcodec_vpu_send_msg(vpu, (void *)&msg, sizeof(msg));
+
+       if (IS_ERR_OR_NULL(vpu->vsi)) {
+               mtk_vdec_err(vpu->ctx, "invalid vdec vsi, status=%d", err);
+               return -EINVAL;
+       }
+
        mtk_vdec_debug(vpu->ctx, "- ret=%d", err);
        return err;
 }

Reply via email to