commit 0321258252fba2f072cd8d12079d6d9d4ab465de
author huangli2018 <li.h.huang@intel.com> Tue Feb 28 16:13:16 2023 +0800
committer intel-mediadev <mediadev@intel.com> Tue Feb 28 16:45:04 2023 +0800
tree 2b30bb44c119c22b424831385e02c4412da7f135
parent 559a168b298e61f604d7819cbf20a411923c8b8c

[Decode] Add more conditons check for AV1 reference frame index

In order to fix memleak issue on VPL AV1,pr is intel-innersource/drivers.gpu.media.sdk.lib#6691, we will not allocate surfaces for 1s frame. Thus need driver need to do change , otherwise it will return invalid parameter for 1st frame

media_driver/linux/common/codec/ddi/media_ddi_decode_av1.cpp

diff --git a/media_driver/linux/common/codec/ddi/media_ddi_decode_av1.cpp b/media_driver/linux/common/codec/ddi/media_ddi_decode_av1.cpp
index 1be4ca2..cf982fe 100644
--- a/media_driver/linux/common/codec/ddi/media_ddi_decode_av1.cpp
+++ b/media_driver/linux/common/codec/ddi/media_ddi_decode_av1.cpp
@@ -227,9 +227,14 @@
         if (picParam->ref_frame_map[i] < mediaCtx->uiNumSurfaces)
         {
             frameIdx = GetRenderTargetID(&m_ddiDecodeCtx->RTtbl, refSurface);
-            if (frameIdx == DDI_CODEC_INVALID_FRAME_INDEX) {
-                return VA_STATUS_ERROR_INVALID_PARAMETER;
+       
+            if ((frameIdx == DDI_CODEC_INVALID_FRAME_INDEX) && 
+                (picParam->pic_info_fields.bits.frame_type != keyFrame) && 
+                (picParam->pic_info_fields.bits.frame_type != intraOnlyFrame))
+            {
+                 return VA_STATUS_ERROR_INVALID_PARAMETER;
             }
+
             picAV1Params->m_refFrameMap[i].FrameIdx = ((uint32_t)frameIdx >= CODECHAL_NUM_UNCOMPRESSED_SURFACE_AV1) ?
                                                       (CODECHAL_NUM_UNCOMPRESSED_SURFACE_AV1 - 1) : frameIdx;
         }