Add static analysis fixes

Signed-off-by: Jemale Lockett <jemale.lockett@intel.com>
oneapi-src · Sep 26, 2024 · cad0645 · cad0645
1 parent e0eb0fa
commit cad0645
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/utils/image/test/image_integration_tests.cpp b/utils/image/test/image_integration_tests.cpp
@@ -36,7 +36,10 @@ TEST(ImageIntegrationTests, WritesPNGFile) {
 
   level_zero_tests::ImagePNG32Bit output("output.png");
   EXPECT_EQ(output.get_pixels(), pixels);
-  std::remove("output.png");
+  auto result = std::remove("output.png");
+  if (result != 0) {
+    std::cerr << "Error deleting file output.png" << std::endl;
+  }
 
   EXPECT_EQ(image.get_pixels(), pixels);
 }
@@ -72,7 +75,10 @@ TEST(ImageIntegrationTests, WritesGrayscaleBMPFile) {
 
   level_zero_tests::ImageBMP8Bit output("output.bmp");
   EXPECT_EQ(output.get_pixels(), pixels);
-  std::remove("output.bmp");
+  auto result = std::remove("output.bmp");
+  if (result != 0) {
+    std::cerr << "Error deleting file output.bmp" << std::endl;
+  }
 
   EXPECT_EQ(image.get_pixels(), pixels);
 }
@@ -104,7 +110,10 @@ TEST(ImageIntegrationTests, WritesColorBMPFile) {
 
   level_zero_tests::ImageBMP32Bit output("output.bmp");
   EXPECT_EQ(output.get_pixels(), pixels);
-  std::remove("output.bmp");
+  auto result = std::remove("output.bmp");
+  if (result != 0) {
+    std::cerr << "Error deleting file output.bmp" << std::endl;
+  }
 
   EXPECT_EQ(image.get_pixels(), pixels);
 }
diff --git a/utils/test_harness/src/test_harness_cmdqueue.cpp b/utils/test_harness/src/test_harness_cmdqueue.cpp
@@ -82,7 +82,10 @@ ze_command_queue_handle_t create_command_queue(
 zeCommandQueue::zeCommandQueue() { command_queue_ = create_command_queue(); }
 
 zeCommandQueue::~zeCommandQueue() {
-  EXPECT_EQ(ZE_RESULT_SUCCESS, zeCommandQueueDestroy(command_queue_));
+  auto result = zeCommandQueueDestroy(command_queue_);
+  if (result != ZE_RESULT_SUCCESS) {
+    LOG_ERROR << "zeCommandQueueDestroy failed: " << lzt::to_string(result);
+  }
 }
 
 void execute_command_lists(ze_command_queue_handle_t cq,