From 7898721cce41e93e09c3d055e618754ef064f1ae Mon Sep 17 00:00:00 2001 From: Vinson Lee Date: Sun, 24 Apr 2022 21:28:50 -0700 Subject: [PATCH] panfrost: Fix memory leaks on unit test failures. Change ASSERT_EQ to EXPECT_EQ to avoid aborting before freeing memory. Fix defects reported by Coverity Scan. Resource leak (RESOURCE_LEAK) leaked_storage: Variable tiled going out of scope leaks the storage it points to. leaked_storage: Variable linear going out of scope leaks the storage it points to. leaked_storage: Variable ref going out of scope leaks the storage it points to. Fixes: bb6c14a6978 ("panfrost: Unit test u-interleaved tiling routines") Suggested-by: Alyssa Rosenzweig Signed-off-by: Vinson Lee Part-of: --- src/panfrost/shared/test/test-tiling.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/panfrost/shared/test/test-tiling.cpp b/src/panfrost/shared/test/test-tiling.cpp index b06954485d0..e6b9fe669f5 100644 --- a/src/panfrost/shared/test/test-tiling.cpp +++ b/src/panfrost/shared/test/test-tiling.cpp @@ -160,9 +160,9 @@ test(unsigned width, unsigned height, unsigned rx, unsigned ry, dst_stride, src_stride, format, store); if (store) - ASSERT_EQ(memcmp(ref, tiled, bpp * tiled_width * tiled_height), 0); + EXPECT_EQ(memcmp(ref, tiled, bpp * tiled_width * tiled_height), 0); else - ASSERT_EQ(memcmp(ref, linear, bpp * rw * linear_stride), 0); + EXPECT_EQ(memcmp(ref, linear, bpp * rw * linear_stride), 0); free(ref); free(tiled);