From e29d0fd3a1bceefe9c34bd7361a9a2a91971b0d6 Mon Sep 17 00:00:00 2001 From: Sven van Haastregt Date: Tue, 1 Aug 2023 09:49:43 +0100 Subject: [PATCH] conversions: fix memory leak from excess gMTdata initialization (#1787) `gMTdata` was initialized twice, but freed only once. Drop the first initialization with a local seed, and initialize with `gRandomSeed` instead. Signed-off-by: Sven van Haastregt --- test_conformance/conversions/test_conversions.cpp | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/test_conformance/conversions/test_conversions.cpp b/test_conformance/conversions/test_conversions.cpp index a8be2098..dab61dc5 100644 --- a/test_conformance/conversions/test_conversions.cpp +++ b/test_conformance/conversions/test_conversions.cpp @@ -118,7 +118,6 @@ const int test_num = ARRAY_SIZE(test_list); int main(int argc, const char **argv) { int error; - cl_uint seed = (cl_uint)time(NULL); argc = parseCustomParam(argc, argv); if (argc == -1) @@ -145,8 +144,8 @@ int main(int argc, const char **argv) #endif vlog("===========================================================\n"); - vlog("Random seed: %u\n", seed); - gMTdata = init_genrand(seed); + vlog("Random seed: %u\n", gRandomSeed); + gMTdata = init_genrand(gRandomSeed); const char *arg[] = { argv[0] }; int ret = @@ -475,8 +474,6 @@ test_status InitCL(cl_device_id device) } } - gMTdata = init_genrand(gRandomSeed); - char c[1024]; static const char *no_yes[] = { "NO", "YES" }; vlog("\nCompute Device info:\n");