From 52edb12505ca450b1e68ad67a86154d6edb54579 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?K=C3=A9vin=20Petit?= Date: Sat, 1 Jul 2023 13:41:37 +0100 Subject: [PATCH] Fix CL_KERNEL_ATTRIBUTES query (#567) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The api_query_string is destroyed before we copy from it. Should have been caught in testing/code review. Change-Id: I4998d9b2518f10e2e1efbee4b42edc0014561432 Signed-off-by: Kévin Petit --- src/api.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/api.cpp b/src/api.cpp index 29a59aed..70874563 100644 --- a/src/api.cpp +++ b/src/api.cpp @@ -2861,6 +2861,7 @@ cl_int CLVK_API_CALL clGetKernelInfo(cl_kernel kern, cl_kernel_info param_name, cl_uint val_uint; cl_context val_context; cl_program val_program; + api_query_string val_string; auto kernel = icd_downcast(kern); @@ -2894,9 +2895,9 @@ cl_int CLVK_API_CALL clGetKernelInfo(cl_kernel kern, cl_kernel_info param_name, ret_size = sizeof(val_program); break; case CL_KERNEL_ATTRIBUTES: { - const api_query_string attrs = kernel->attributes(); - copy_ptr = attrs.c_str(); - ret_size = attrs.size_with_null(); + val_string = kernel->attributes(); + copy_ptr = val_string.c_str(); + ret_size = val_string.size_with_null(); break; } default: