[VP] Fix coverity low impact issue

Fix coverity low impact issue.
Jexu · Dec 14, 2023 · faa5cc3 · faa5cc3
1 parent 23eb02c
commit faa5cc3
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/media_driver/agnostic/common/os/mos_graphicsresource.cpp b/media_driver/agnostic/common/os/mos_graphicsresource.cpp
@@ -112,7 +112,6 @@ MOS_STATUS GraphicsResource::Dump(OsContext* osContextPtr, uint32_t overrideOffs
         nullptr)) != MOS_STATUS_SUCCESS)
     {
         MOS_OS_ASSERTMESSAGE("Failed to write to file '%s'.", sPath);
-        eStatus = MOS_STATUS_FILE_WRITE_FAILED;
         if (hFile != nullptr)
         {
             MosUtilities::MosCloseHandle(hFile);

diff --git a/media_driver/agnostic/common/os/mos_utilities_usersetting.cpp b/media_driver/agnostic/common/os/mos_utilities_usersetting.cpp
@@ -182,8 +182,7 @@ MOS_STATUS MosUtilities::MosWriteOneUserFeatureGroupToXML(MOS_USER_FEATURE_VALUE
         sOutBuf,
         sizeof(sOutBuf),
         sizeof(sOutBuf),
-        "  </Group>\n",
-        UserFeatureFilter.pcGroup);
+        "  </Group>\n");
     eStatus = MosAppendFileFromPtr(
         s_xmlFilePath,
         sOutBuf,

diff --git a/media_driver/agnostic/gen12/vp/hal/vphal_render_vebox_g12_base.cpp b/media_driver/agnostic/gen12/vp/hal/vphal_render_vebox_g12_base.cpp
@@ -614,6 +614,7 @@ MOS_STATUS VPHAL_VEBOX_STATE_G12_BASE::AllocateResources()
     InitValue               = 0;
     // change the init value when null hw is enabled
     VPHAL_RENDER_CHK_NULL(pOsInterface);
+    VPHAL_RENDER_CHK_NULL(pRenderData);
 
     if (pOsInterface->bNullHwIsEnabled)
     {