From 6a60db558bfe50fbcfc99cc3961121cf0302e102 Mon Sep 17 00:00:00 2001 From: Harald van Dijk Date: Fri, 8 Mar 2024 16:07:09 +0000 Subject: [PATCH] Fix uninitialised memory use. (#1909) CL_DEVICE_MAX_WORK_GROUP_SIZE is specified to fill in a size_t. By asking it to fill in an uninitialised cl_long, on platforms where size_t is smaller than cl_long, the high bits of the cl_long remain uninitialised. --- test_conformance/basic/test_async_copy2D.cpp | 2 +- test_conformance/basic/test_async_copy3D.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test_conformance/basic/test_async_copy2D.cpp b/test_conformance/basic/test_async_copy2D.cpp index 11ef84bd..eab9a35f 100644 --- a/test_conformance/basic/test_async_copy2D.cpp +++ b/test_conformance/basic/test_async_copy2D.cpp @@ -141,7 +141,7 @@ int test_copy2D(const cl_device_id deviceID, const cl_context context, test_error(error, "clGetDeviceInfo for CL_DEVICE_MAX_MEM_ALLOC_SIZE failed."); - cl_long max_work_group_size; + size_t max_work_group_size; error = clGetDeviceInfo(deviceID, CL_DEVICE_MAX_WORK_GROUP_SIZE, sizeof(max_work_group_size), &max_work_group_size, NULL); diff --git a/test_conformance/basic/test_async_copy3D.cpp b/test_conformance/basic/test_async_copy3D.cpp index aa22f3a2..976141b9 100644 --- a/test_conformance/basic/test_async_copy3D.cpp +++ b/test_conformance/basic/test_async_copy3D.cpp @@ -193,7 +193,7 @@ int test_copy3D(const cl_device_id deviceID, const cl_context context, test_error(error, "clGetDeviceInfo failed for CL_DEVICE_MAX_WORK_ITEM_SIZES"); - cl_long max_work_group_size; + size_t max_work_group_size; error = clGetDeviceInfo(deviceID, CL_DEVICE_MAX_WORK_GROUP_SIZE, sizeof(max_work_group_size), &max_work_group_size, NULL);