Merge pull request #745 from pbalcer/fix-cov

igchor · web-flow · commit 2ffcda3f8a3a · 2023-07-25T11:22:54.000-07:00
[umf] fix the last remaining coverity issue
diff --git a/test/unified_malloc_framework/memoryPool.hpp b/test/unified_malloc_framework/memoryPool.hpp
@@ -284,7 +284,7 @@ TEST_P(umfMultiPoolTest, memoryTracking) {
         ASSERT_EQ(pool, expectedPool);
     }
 
-    for (auto p : ptrs) {
+    for (auto &p : ptrs) {
         umfFree(std::get<0>(p));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -284,7 +284,7 @@ TEST_P(umfMultiPoolTest, memoryTracking) {`
`284`	`284`	`ASSERT_EQ(pool, expectedPool);`
`285`	`285`	`}`
`286`	`286`
`287`		`- for (auto p : ptrs) {`
	`287`	`+ for (auto &p : ptrs) {`
`288`	`288`	`umfFree(std::get<0>(p));`
`289`	`289`	`}`
`290`	`290`	`}`