[Decode] Fix decode coverity issue

check status return to enhance roubustness.
intel · Dec 17, 2024 · a943748 · a943748
1 parent 585993d
commit a943748
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/...softlet/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_multipipe.cpp b/...softlet/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_multipipe.cpp
@@ -654,7 +654,7 @@ MOS_STATUS DecodeScalabilityMultiPipe::SendAttrWithFrameTracking(
     {
         PMOS_RESOURCE resource = nullptr;
         uint32_t      offset   = 0;
-        m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset);
+        SCALABILITY_CHK_STATUS_RETURN(m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset));
 
         cmdBuffer.Attributes.bEnableMediaFrameTracking    = true;
         cmdBuffer.Attributes.resMediaFrameTrackingSurface = resource;

diff --git a/...oftlet/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_singlepipe.cpp b/...oftlet/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_singlepipe.cpp
@@ -197,7 +197,7 @@ MOS_STATUS DecodeScalabilitySinglePipe::SendAttrWithFrameTracking(
     {
         PMOS_RESOURCE resource = nullptr;
         uint32_t      offset   = 0;
-        m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset);
+        SCALABILITY_CHK_STATUS_RETURN(m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset));
 
         cmdBuffer.Attributes.bEnableMediaFrameTracking    = true;
         cmdBuffer.Attributes.resMediaFrameTrackingSurface = resource;

diff --git a/...et/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_multipipe_next.cpp b/...et/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_multipipe_next.cpp
@@ -614,7 +614,7 @@ MOS_STATUS DecodeScalabilityMultiPipeNext::SendAttrWithFrameTracking(
     {
         PMOS_RESOURCE resource = nullptr;
         uint32_t      offset   = 0;
-        m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset);
+        SCALABILITY_CHK_STATUS_RETURN(m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset));
 
         cmdBuffer.Attributes.bEnableMediaFrameTracking    = true;
         cmdBuffer.Attributes.resMediaFrameTrackingSurface = resource;

diff --git a/...t/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_singlepipe_next.cpp b/...t/agnostic/common/codec/hal/dec/shared/scalability/decode_scalability_singlepipe_next.cpp
@@ -169,7 +169,7 @@ MOS_STATUS DecodeScalabilitySinglePipeNext::SendAttrWithFrameTracking(
     {
         PMOS_RESOURCE resource = nullptr;
         uint32_t      offset   = 0;
-        m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset);
+        SCALABILITY_CHK_STATUS_RETURN(m_statusReport->GetAddress(decode::statusReportGlobalCount, resource, offset));
 
         cmdBuffer.Attributes.bEnableMediaFrameTracking    = true;
         cmdBuffer.Attributes.resMediaFrameTrackingSurface = resource;