[coverity] Fix 1566349 Resource leak

Move `ur_loader_config_handle_t` cleanup to the text fixture's `TearDown()`.
oneapi-src · Jul 18, 2023 · 38141b3 · 38141b3
1 parent 19fdd60
commit 38141b3
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/test/loader/loader_config/urLoaderConfigCreate.cpp b/test/loader/loader_config/urLoaderConfigCreate.cpp
@@ -5,12 +5,18 @@
 
 #include "fixtures.hpp"
 
-struct LoaderConfigCreateTest : ::testing::Test {};
+struct LoaderConfigCreateTest : ::testing::Test {
+ void TearDown() override {
+ if (loaderConfig) {
+ ASSERT_SUCCESS(urLoaderConfigRelease(loaderConfig));
+ }
+ }
 
-TEST_F(LoaderConfigCreateTest, Success) {
  ur_loader_config_handle_t loaderConfig = nullptr;
+};
+
+TEST_F(LoaderConfigCreateTest, Success) {
  ASSERT_SUCCESS(urLoaderConfigCreate(&loaderConfig));
- ASSERT_SUCCESS(urLoaderConfigRelease(loaderConfig));
 }
 
 TEST_F(LoaderConfigCreateTest, InvalidNullPointerLoaderConfig) {