From faa5cc3a51a6df2251f0ddd04d2bfb4a92d85b78 Mon Sep 17 00:00:00 2001 From: WenshengZhang Date: Thu, 14 Dec 2023 22:50:59 +0800 Subject: [PATCH] [VP] Fix coverity low impact issue Fix coverity low impact issue. --- media_driver/agnostic/common/os/mos_graphicsresource.cpp | 1 - media_driver/agnostic/common/os/mos_utilities_usersetting.cpp | 3 +-- .../agnostic/gen12/vp/hal/vphal_render_vebox_g12_base.cpp | 1 + 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/media_driver/agnostic/common/os/mos_graphicsresource.cpp b/media_driver/agnostic/common/os/mos_graphicsresource.cpp index 09b51236845..b34c92eaede 100644 --- 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 index e9560c6406e..294d2a63099 100644 --- 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), - " \n", - UserFeatureFilter.pcGroup); + " \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 index d4489165bbe..979848c55bd 100644 --- 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) {