November 2022 - Wine-commits

Jacek Caban : winevulkan: Convert all 32-bit structs containing pNext pointer.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: af2f0c790bc19f391008ef0c1ef1ae80ecd450d9 URL: https://gitlab.winehq.org/wine/wine/-/commit/af2f0c790bc19f391008ef0c1ef1ae… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 11 18:38:06 2022 +0100 winevulkan: Convert all 32-bit structs containing pNext pointer. --- dlls/winevulkan/make_vulkan | 9 + dlls/winevulkan/vulkan_thunks.c | 24742 ++++++++++++++++++++++++++++---------- 2 files changed, 18241 insertions(+), 6510 deletions(-)

1 0

Jacek Caban : winevulkan: Convert all 32-bit structs containing pointer or pointer-size members.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 89310c035d07e8f2ca512f0c63454f7f91b2d132 URL: https://gitlab.winehq.org/wine/wine/-/commit/89310c035d07e8f2ca512f0c63454f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 11 21:00:44 2022 +0100 winevulkan: Convert all 32-bit structs containing pointer or pointer-size members. --- dlls/winevulkan/make_vulkan | 20 +- dlls/winevulkan/vulkan_thunks.c | 420 +++++++++++++++++++++++++++++++++++++--- 2 files changed, 405 insertions(+), 35 deletions(-)

1 0

Jacek Caban : winevulkan: Convert dispatchable handle arrays in 32-bit thunks.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 3ceae1dcaf32e1cfcfd1f327b7226d31b006c7a9 URL: https://gitlab.winehq.org/wine/wine/-/commit/3ceae1dcaf32e1cfcfd1f327b7226d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 11 23:14:44 2022 +0100 winevulkan: Convert dispatchable handle arrays in 32-bit thunks. --- dlls/winevulkan/make_vulkan | 5 ++-- dlls/winevulkan/vulkan_thunks.c | 54 ++++++++++++++++++++++++++++++++++++++--- 2 files changed, 54 insertions(+), 5 deletions(-) diff --git a/dlls/winevulkan/make_vulkan b/dlls/winevulkan/make_vulkan index aa0f14f5ac8..26a4f986c49 100755 --- a/dlls/winevulkan/make_vulkan +++ b/dlls/winevulkan/make_vulkan @@ -1697,6 +1697,8 @@ class VkParam(VkVariable): if unwrap and self.handle.is_wrapped(): return True + if conv and self.handle.is_dispatchable(): + return True return False @@ -2373,12 +2375,11 @@ class ArrayConversionFunction(object): body += " }\n" body += " else\n" body += " out[i] = NULL;\n" - elif self.array.is_handle() and self.direction == Direction.INPUT: + elif self.array.is_handle() and self.direction == Direction.INPUT and self.unwrap: if self.array.pointer_array: LOGGER.error("Unhandled handle pointer arrays") body += " out[i] = " + self.array.handle.driver_handle("in[i]") + ";\n" else: - LOGGER.warning("Unhandled conversion operand type") body += " out[i] = in[i];\n" body += " }\n" diff --git a/dlls/winevulkan/vulkan_thunks.c b/dlls/winevulkan/vulkan_thunks.c index bc14e4e26de..25cfc4bed82 100644 --- a/dlls/winevulkan/vulkan_thunks.c +++ b/dlls/winevulkan/vulkan_thunks.c @@ -5873,6 +5873,24 @@ static inline void convert_VkCommandBufferAllocateInfo_win32_to_unwrapped_host(c } #endif /* USE_STRUCT_CONVERSION */ +#if defined(USE_STRUCT_CONVERSION) +static inline VkCommandBuffer *convert_VkCommandBuffer_array_win32_to_unwrapped_host(struct conversion_context *ctx, const VkCommandBuffer *in, uint32_t count) +{ + VkCommandBuffer *out; + unsigned int i; + + if (!in || !count) return NULL; + + out = conversion_context_alloc(ctx, count * sizeof(*out)); + for (i = 0; i < count; i++) + { + out[i] = in[i]; + } + + return out; +} +#endif /* USE_STRUCT_CONVERSION */ + #if defined(USE_STRUCT_CONVERSION) static inline void convert_VkDescriptorSetAllocateInfo_win32_to_host(struct conversion_context *ctx, const VkDescriptorSetAllocateInfo32 *in, VkDescriptorSetAllocateInfo *out) { @@ -16162,6 +16180,20 @@ static inline void convert_VkPerformanceCounterDescriptionKHR_array_host_to_win3 } #endif /* USE_STRUCT_CONVERSION */ +#if defined(USE_STRUCT_CONVERSION) +static inline void convert_VkPhysicalDevice_array_unwrapped_host_to_win32(const VkPhysicalDevice *in, VkPhysicalDevice *out, uint32_t count) +{ + unsigned int i; + + if (!in) return; + + for (i = 0; i < count; i++) + { + out[i] = in[i]; + } +} +#endif /* USE_STRUCT_CONVERSION */ + #if defined(USE_STRUCT_CONVERSION) static inline void convert_VkMappedMemoryRange_win32_to_host(const VkMappedMemoryRange32 *in, VkMappedMemoryRange *out) { @@ -24680,11 +24712,16 @@ static NTSTATUS thunk32_vkAllocateCommandBuffers(void *args) VkResult result; } *params = args; VkCommandBufferAllocateInfo pAllocateInfo_host; + VkCommandBuffer *pCommandBuffers_host; + struct conversion_context ctx; TRACE("%p, %p, %p\n", params->device, params->pAllocateInfo, params->pCommandBuffers); + init_conversion_context(&ctx); convert_VkCommandBufferAllocateInfo_win32_to_unwrapped_host(params->pAllocateInfo, &pAllocateInfo_host); - params->result = wine_vkAllocateCommandBuffers(params->device, &pAllocateInfo_host, params->pCommandBuffers); + pCommandBuffers_host = convert_VkCommandBuffer_array_win32_to_unwrapped_host(&ctx, params->pCommandBuffers, params->pAllocateInfo->commandBufferCount); + params->result = wine_vkAllocateCommandBuffers(params->device, &pAllocateInfo_host, pCommandBuffers_host); + free_conversion_context(&ctx); return STATUS_SUCCESS; } @@ -36023,10 +36060,16 @@ static NTSTATUS thunk32_vkEnumeratePhysicalDevices(void *args) VkPhysicalDevice *pPhysicalDevices; VkResult result; } *params = args; + VkPhysicalDevice *pPhysicalDevices_host; + struct conversion_context ctx; TRACE("%p, %p, %p\n", params->instance, params->pPhysicalDeviceCount, params->pPhysicalDevices); - params->result = wine_vkEnumeratePhysicalDevices(params->instance, params->pPhysicalDeviceCount, params->pPhysicalDevices); + init_conversion_context(&ctx); + pPhysicalDevices_host = (params->pPhysicalDevices && *params->pPhysicalDeviceCount) ? conversion_context_alloc(&ctx, sizeof(*pPhysicalDevices_host) * *params->pPhysicalDeviceCount) : NULL; + params->result = wine_vkEnumeratePhysicalDevices(params->instance, params->pPhysicalDeviceCount, pPhysicalDevices_host); + convert_VkPhysicalDevice_array_unwrapped_host_to_win32(pPhysicalDevices_host, params->pPhysicalDevices, *params->pPhysicalDeviceCount); + free_conversion_context(&ctx); return STATUS_SUCCESS; } @@ -36092,10 +36135,15 @@ static NTSTATUS thunk32_vkFreeCommandBuffers(void *args) uint32_t commandBufferCount; const VkCommandBuffer *pCommandBuffers; } *params = args; + const VkCommandBuffer *pCommandBuffers_host; + struct conversion_context ctx; TRACE("%p, 0x%s, %u, %p\n", params->device, wine_dbgstr_longlong(params->commandPool), params->commandBufferCount, params->pCommandBuffers); - wine_vkFreeCommandBuffers(params->device, params->commandPool, params->commandBufferCount, params->pCommandBuffers); + init_conversion_context(&ctx); + pCommandBuffers_host = convert_VkCommandBuffer_array_win32_to_unwrapped_host(&ctx, params->pCommandBuffers, params->commandBufferCount); + wine_vkFreeCommandBuffers(params->device, params->commandPool, params->commandBufferCount, pCommandBuffers_host); + free_conversion_context(&ctx); return STATUS_SUCCESS; }

1 0

Jacek Caban : winevulkan: Convert pointers to pointer-sized variables in 32-bit thunks.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: a8539fc979884a09edf96d40705bd97f7833fe29 URL: https://gitlab.winehq.org/wine/wine/-/commit/a8539fc979884a09edf96d40705bd9… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Nov 11 23:31:20 2022 +0100 winevulkan: Convert pointers to pointer-sized variables in 32-bit thunks. --- dlls/winevulkan/make_vulkan | 14 ++++++++++---- dlls/winevulkan/vulkan_thunks.c | 35 ++++++++++++++++++++++++++++------- 2 files changed, 38 insertions(+), 11 deletions(-) diff --git a/dlls/winevulkan/make_vulkan b/dlls/winevulkan/make_vulkan index 26a4f986c49..d7349120e40 100755 --- a/dlls/winevulkan/make_vulkan +++ b/dlls/winevulkan/make_vulkan @@ -1617,16 +1617,20 @@ class VkParam(VkVariable): ret += " convert_{0}_{3}_to_{5}host({4}{1}{2}, {2}_host);\n".format(self.type, prefix, self.name, win_type, ctx_param, wrap_part) ret += " }\n" return ret - else: + elif self.is_struct(): return " convert_{0}_{3}_to_{5}host({4}{1}{2}, &{2}_host);\n".format(self.type, prefix, self.name, win_type, ctx_param, wrap_part) + else: + return " {1}_host = *{0}{1};\n".format(prefix, self.name) else: if self.is_dynamic_array(): return " convert_{0}_array_{1}host_to_{2}({3}_host, {4}{3}, {5});\n".format( self.type, wrap_part, win_type, self.name, prefix, self.get_dyn_array_len(prefix)) - else: + elif self.is_struct(): ref_part = "" if self.optional else "&" return " convert_{0}_host_to_{3}({4}{2}_host, {1}{2});\n".format( self.type, prefix, self.name, win_type, ref_part) + else: + return " *{0}{1} = {1}_host;\n".format(prefix, self.name) def definition(self, postfix=None, is_member=False, conv=False): """ Return prototype for the parameter. E.g. 'const char *foo' """ @@ -1684,9 +1688,9 @@ class VkParam(VkVariable): return self.struct.needs_conversion(conv, unwrap, direction, self.is_const()) if self.is_handle(): - # non-array handles are handled inline in thunks + # non-pointer handles are handled inline in thunks if not self.is_dynamic_array() and not self.is_static_array(): - return False + return conv and self.is_pointer() and self.handle.is_dispatchable() # vkAllocateCommandBuffers is a special case, we use it in our private thunk as an input param param_direction = (Direction.INPUT if self.is_const() else Direction.OUTPUT) @@ -1699,6 +1703,8 @@ class VkParam(VkVariable): return True if conv and self.handle.is_dispatchable(): return True + elif self.is_pointer() and self.is_pointer_size(): + return conv return False diff --git a/dlls/winevulkan/vulkan_thunks.c b/dlls/winevulkan/vulkan_thunks.c index 25cfc4bed82..32553aba432 100644 --- a/dlls/winevulkan/vulkan_thunks.c +++ b/dlls/winevulkan/vulkan_thunks.c @@ -33296,13 +33296,16 @@ static NTSTATUS thunk32_vkCreateDevice(void *args) VkResult result; } *params = args; VkDeviceCreateInfo pCreateInfo_host; + VkDevice pDevice_host; struct conversion_context ctx; TRACE("%p, %p, %p, %p\n", params->physicalDevice, params->pCreateInfo, params->pAllocator, params->pDevice); init_conversion_context(&ctx); convert_VkDeviceCreateInfo_win32_to_host(&ctx, params->pCreateInfo, &pCreateInfo_host); - params->result = wine_vkCreateDevice(params->physicalDevice, &pCreateInfo_host, params->pAllocator, params->pDevice, params->client_ptr); + pDevice_host = *params->pDevice; + params->result = wine_vkCreateDevice(params->physicalDevice, &pCreateInfo_host, params->pAllocator, &pDevice_host, params->client_ptr); + *params->pDevice = pDevice_host; free_conversion_context(&ctx); return STATUS_SUCCESS; } @@ -33610,13 +33613,16 @@ static NTSTATUS thunk32_vkCreateInstance(void *args) VkResult result; } *params = args; VkInstanceCreateInfo pCreateInfo_host; + VkInstance pInstance_host; struct conversion_context ctx; TRACE("%p, %p, %p\n", params->pCreateInfo, params->pAllocator, params->pInstance); init_conversion_context(&ctx); convert_VkInstanceCreateInfo_win32_to_host(&ctx, params->pCreateInfo, &pCreateInfo_host); - params->result = wine_vkCreateInstance(&pCreateInfo_host, params->pAllocator, params->pInstance, params->client_ptr); + pInstance_host = *params->pInstance; + params->result = wine_vkCreateInstance(&pCreateInfo_host, params->pAllocator, &pInstance_host, params->client_ptr); + *params->pInstance = pInstance_host; free_conversion_context(&ctx); return STATUS_SUCCESS; } @@ -37477,10 +37483,13 @@ static NTSTATUS thunk32_vkGetDeviceQueue(void *args) uint32_t queueIndex; VkQueue *pQueue; } *params = args; + VkQueue pQueue_host; TRACE("%p, %u, %u, %p\n", params->device, params->queueFamilyIndex, params->queueIndex, params->pQueue); - wine_vkGetDeviceQueue(params->device, params->queueFamilyIndex, params->queueIndex, params->pQueue); + pQueue_host = *params->pQueue; + wine_vkGetDeviceQueue(params->device, params->queueFamilyIndex, params->queueIndex, &pQueue_host); + *params->pQueue = pQueue_host; return STATUS_SUCCESS; } @@ -37509,11 +37518,14 @@ static NTSTATUS thunk32_vkGetDeviceQueue2(void *args) VkQueue *pQueue; } *params = args; VkDeviceQueueInfo2 pQueueInfo_host; + VkQueue pQueue_host; TRACE("%p, %p, %p\n", params->device, params->pQueueInfo, params->pQueue); convert_VkDeviceQueueInfo2_win32_to_host(params->pQueueInfo, &pQueueInfo_host); - wine_vkGetDeviceQueue2(params->device, &pQueueInfo_host, params->pQueue); + pQueue_host = *params->pQueue; + wine_vkGetDeviceQueue2(params->device, &pQueueInfo_host, &pQueue_host); + *params->pQueue = pQueue_host; return STATUS_SUCCESS; } @@ -39818,10 +39830,13 @@ static NTSTATUS thunk32_vkGetPipelineCacheData(void *args) void *pData; VkResult result; } *params = args; + size_t pDataSize_host; TRACE("%p, 0x%s, %p, %p\n", params->device, wine_dbgstr_longlong(params->pipelineCache), params->pDataSize, params->pData); - params->result = wine_device_from_handle(params->device)->funcs.p_vkGetPipelineCacheData(wine_device_from_handle(params->device)->device, params->pipelineCache, params->pDataSize, params->pData); + pDataSize_host = *params->pDataSize; + params->result = wine_device_from_handle(params->device)->funcs.p_vkGetPipelineCacheData(wine_device_from_handle(params->device)->device, params->pipelineCache, &pDataSize_host, params->pData); + *params->pDataSize = pDataSize_host; return STATUS_SUCCESS; } @@ -40420,10 +40435,13 @@ static NTSTATUS thunk32_vkGetShaderInfoAMD(void *args) void *pInfo; VkResult result; } *params = args; + size_t pInfoSize_host; TRACE("%p, 0x%s, %#x, %#x, %p, %p\n", params->device, wine_dbgstr_longlong(params->pipeline), params->shaderStage, params->infoType, params->pInfoSize, params->pInfo); - params->result = wine_device_from_handle(params->device)->funcs.p_vkGetShaderInfoAMD(wine_device_from_handle(params->device)->device, params->pipeline, params->shaderStage, params->infoType, params->pInfoSize, params->pInfo); + pInfoSize_host = *params->pInfoSize; + params->result = wine_device_from_handle(params->device)->funcs.p_vkGetShaderInfoAMD(wine_device_from_handle(params->device)->device, params->pipeline, params->shaderStage, params->infoType, &pInfoSize_host, params->pInfo); + *params->pInfoSize = pInfoSize_host; return STATUS_SUCCESS; } @@ -40559,10 +40577,13 @@ static NTSTATUS thunk32_vkGetValidationCacheDataEXT(void *args) void *pData; VkResult result; } *params = args; + size_t pDataSize_host; TRACE("%p, 0x%s, %p, %p\n", params->device, wine_dbgstr_longlong(params->validationCache), params->pDataSize, params->pData); - params->result = wine_device_from_handle(params->device)->funcs.p_vkGetValidationCacheDataEXT(wine_device_from_handle(params->device)->device, params->validationCache, params->pDataSize, params->pData); + pDataSize_host = *params->pDataSize; + params->result = wine_device_from_handle(params->device)->funcs.p_vkGetValidationCacheDataEXT(wine_device_from_handle(params->device)->device, params->validationCache, &pDataSize_host, params->pData); + *params->pDataSize = pDataSize_host; return STATUS_SUCCESS; }

1 0

Rémi Bernon : opengl32: Generate wow64 thunks.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 531b8bc48326d3fb047c5b335cd5eb77a4ce4244 URL: https://gitlab.winehq.org/wine/wine/-/commit/531b8bc48326d3fb047c5b335cd5eb… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Nov 5 01:22:12 2022 +0100 opengl32: Generate wow64 thunks. --- dlls/opengl32/make_opengl | 170 +- dlls/opengl32/unix_thunks.c | 36969 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 37136 insertions(+), 3 deletions(-)

1 0

Rémi Bernon : opengl32: Manually write glPathGlyphIndexRangeNV wow64 thunk.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 076ac8484ea00a5336f0e149f70a94adbaebc0c1 URL: https://gitlab.winehq.org/wine/wine/-/commit/076ac8484ea00a5336f0e149f70a94… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Oct 7 18:12:51 2022 +0200 opengl32: Manually write glPathGlyphIndexRangeNV wow64 thunk. --- dlls/opengl32/make_opengl | 40 +++++++++++++++++++++++++++++++++------- dlls/opengl32/unix_thunks.c | 27 ++------------------------- dlls/opengl32/unix_wgl.c | 35 +++++++++++++++++++++++++++++++++++ 3 files changed, 70 insertions(+), 32 deletions(-) diff --git a/dlls/opengl32/make_opengl b/dlls/opengl32/make_opengl index 9eff8e83bcc..f0cdd136bcf 100755 --- a/dlls/opengl32/make_opengl +++ b/dlls/opengl32/make_opengl @@ -167,6 +167,10 @@ my %manual_win_thunks = "wglGetProcAddress" => 1, "wglSwapBuffers" => 1, ); +my %manual_wow64_thunks = + ( + "glPathGlyphIndexRangeNV" => 1, + ); # # Used to convert some types @@ -258,7 +262,7 @@ sub generate_unix_thunk($$$) my $func_ret = get_func_ret( $func, 0 ); my $ret = ""; - $ret .= "static NTSTATUS $prefix\_$name( void *args )\n"; + $ret .= "NTSTATUS $prefix\_$name( void *args )\n"; $ret .= "{\n"; $ret .= " struct $name\_params *params = args;\n" unless !$call_args && is_void_func( $func ); # special case for functions that take an HDC as first parameter @@ -1058,18 +1062,21 @@ foreach (sort keys %wgl_functions) { next if defined $manual_win_functions{$_}; next if needs_wrapper( $_, $wgl_functions{$_} ); + print OUT "static " unless defined $manual_wow64_thunks{$_}; print OUT generate_unix_thunk($_, $wgl_functions{$_}, "wgl"); } foreach (sort keys %norm_functions) { next if defined $manual_win_functions{$_}; next if needs_wrapper( $_, $norm_functions{$_} ); + print OUT "static " unless defined $manual_wow64_thunks{$_}; print OUT generate_unix_thunk($_, $norm_functions{$_}, "gl"); } foreach (sort keys %ext_functions) { next if defined $manual_win_functions{$_}; next if needs_wrapper( $_, $ext_functions{$_} ); + print OUT "static " unless defined $manual_wow64_thunks{$_}; print OUT generate_unix_thunk($_, $ext_functions{$_}, "ext"); } @@ -1099,29 +1106,48 @@ print OUT "typedef ULONG PTR32;\n\n"; foreach (sort keys %wgl_functions) { next if defined $manual_win_functions{$_}; - next if !needs_wow64_thunk($_, $wgl_functions{$_}); + next if !needs_wow64_thunk($_, $wgl_functions{$_}) || defined $manual_wow64_thunks{$_}; print OUT generate_wow64_thunk($_, $wgl_functions{$_}, "wgl"); } foreach (sort keys %norm_functions) { next if defined $manual_win_functions{$_}; - next if !needs_wow64_thunk($_, $norm_functions{$_}); + next if !needs_wow64_thunk($_, $norm_functions{$_}) || defined $manual_wow64_thunks{$_}; print OUT generate_wow64_thunk($_, $norm_functions{$_}, "gl"); } foreach (sort keys %ext_functions) { next if defined $manual_win_functions{$_}; - next if !needs_wow64_thunk($_, $ext_functions{$_}); + next if !needs_wow64_thunk($_, $ext_functions{$_}) || defined $manual_wow64_thunks{$_}; print OUT generate_wow64_thunk($_, $ext_functions{$_}, "ext"); } +foreach (sort keys %wgl_functions) +{ + next if defined $manual_win_functions{$_}; + next unless defined $manual_wow64_thunks{$_}; + print OUT "extern NTSTATUS wow64_wgl_$_( void *args ) DECLSPEC_HIDDEN;\n"; +} +foreach (sort keys %norm_functions) +{ + next if defined $manual_win_functions{$_}; + next unless defined $manual_wow64_thunks{$_}; + print OUT "extern NTSTATUS wow64_gl_$_( void *args ) DECLSPEC_HIDDEN;\n"; +} +foreach (sort keys %ext_functions) +{ + next if defined $manual_win_functions{$_}; + next unless defined $manual_wow64_thunks{$_}; + print OUT "extern NTSTATUS wow64_ext_$_( void *args ) DECLSPEC_HIDDEN;\n"; +} + print OUT "\nconst unixlib_entry_t __wine_unix_call_wow64_funcs[] =\n"; print OUT "{\n"; print OUT " &thread_attach,\n"; foreach (sort keys %wgl_functions) { next if defined $manual_win_functions{$_}; - if (needs_wow64_thunk($_, $wgl_functions{$_})) + if (needs_wow64_thunk($_, $wgl_functions{$_}) || defined $manual_wow64_thunks{$_}) { printf OUT " &wow64_wgl_%s,\n", $_; } @@ -1133,7 +1159,7 @@ foreach (sort keys %wgl_functions) foreach (sort keys %norm_functions) { next if defined $manual_win_functions{$_}; - if (needs_wow64_thunk($_, $norm_functions{$_})) + if (needs_wow64_thunk($_, $norm_functions{$_}) || defined $manual_wow64_thunks{$_}) { printf OUT " &wow64_gl_%s,\n", $_; } @@ -1145,7 +1171,7 @@ foreach (sort keys %norm_functions) foreach (sort keys %ext_functions) { next if defined $manual_win_functions{$_}; - if (needs_wow64_thunk($_, $ext_functions{$_})) + if (needs_wow64_thunk($_, $ext_functions{$_}) || defined $manual_wow64_thunks{$_}) { printf OUT " &wow64_ext_%s,\n", $_; } diff --git a/dlls/opengl32/unix_thunks.c b/dlls/opengl32/unix_thunks.c index c46618d4694..b424bac922b 100644 --- a/dlls/opengl32/unix_thunks.c +++ b/dlls/opengl32/unix_thunks.c @@ -14753,7 +14753,7 @@ static NTSTATUS ext_glPathGlyphIndexArrayNV( void *args ) return STATUS_SUCCESS; } -static NTSTATUS ext_glPathGlyphIndexRangeNV( void *args ) +NTSTATUS ext_glPathGlyphIndexRangeNV( void *args ) { struct glPathGlyphIndexRangeNV_params *params = args; const struct opengl_funcs *funcs = NtCurrentTeb()->glTable; @@ -48613,30 +48613,6 @@ static NTSTATUS wow64_ext_glPathGlyphIndexArrayNV( void *args ) return status; } -static NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) -{ - struct - { - GLenum fontTarget; - PTR32 fontName; - GLbitfield fontStyle; - GLuint pathParameterTemplate; - GLfloat emScale; - PTR32 baseAndCount; - GLenum ret; - } *params32 = args; - struct glPathGlyphIndexRangeNV_params params = - { - .fontTarget = params32->fontTarget, - .fontName = ULongToPtr(params32->fontName), - .fontStyle = params32->fontStyle, - .pathParameterTemplate = params32->pathParameterTemplate, - .emScale = params32->emScale, - }; - FIXME( "params32 %p, params %p stub!\n", params32, &params ); - return STATUS_NOT_IMPLEMENTED; -} - static NTSTATUS wow64_ext_glPathGlyphRangeNV( void *args ) { struct @@ -61112,6 +61088,7 @@ static NTSTATUS wow64_ext_wglSetPixelFormatWINE( void *args ) return status; } +extern NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; const unixlib_entry_t __wine_unix_call_wow64_funcs[] = { diff --git a/dlls/opengl32/unix_wgl.c b/dlls/opengl32/unix_wgl.c index 54f4b0709dd..0daaf597ff3 100644 --- a/dlls/opengl32/unix_wgl.c +++ b/dlls/opengl32/unix_wgl.c @@ -1050,3 +1050,38 @@ NTSTATUS WINAPI thread_attach( void *args ) NtCurrentTeb()->glTable = &null_opengl_funcs; return STATUS_SUCCESS; } + +#ifdef _WIN64 + +typedef ULONG PTR32; + +extern NTSTATUS ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; + +NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) +{ + struct + { + GLenum fontTarget; + PTR32 fontName; + GLbitfield fontStyle; + GLuint pathParameterTemplate; + GLfloat emScale; + GLuint baseAndCount[2]; + GLenum ret; + } *params32 = args; + struct glPathGlyphIndexRangeNV_params params = + { + .fontTarget = params32->fontTarget, + .fontName = ULongToPtr(params32->fontName), + .fontStyle = params32->fontStyle, + .pathParameterTemplate = params32->pathParameterTemplate, + .emScale = params32->emScale, + .baseAndCount = {params32->baseAndCount[0], params32->baseAndCount[1]}, + }; + NTSTATUS status; + if ((status = ext_glPathGlyphIndexRangeNV( &params ))) return status; + params32->ret = params.ret; + return status; +} + +#endif

1 0

Rémi Bernon : opengl32: Implement wow64 thunk for wglGetProcAddress.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: e9db6bca44921b47bb5454a295f1fa208a6cebaa URL: https://gitlab.winehq.org/wine/wine/-/commit/e9db6bca44921b47bb5454a295f1fa… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Nov 14 22:51:31 2022 +0100 opengl32: Implement wow64 thunk for wglGetProcAddress. --- dlls/opengl32/make_opengl | 1 + dlls/opengl32/unix_thunks.c | 16 +--------------- dlls/opengl32/unix_wgl.c | 20 ++++++++++++++++++++ 3 files changed, 22 insertions(+), 15 deletions(-) diff --git a/dlls/opengl32/make_opengl b/dlls/opengl32/make_opengl index f0cdd136bcf..312e7acc964 100755 --- a/dlls/opengl32/make_opengl +++ b/dlls/opengl32/make_opengl @@ -170,6 +170,7 @@ my %manual_win_thunks = my %manual_wow64_thunks = ( "glPathGlyphIndexRangeNV" => 1, + "wglGetProcAddress" => 1, ); # diff --git a/dlls/opengl32/unix_thunks.c b/dlls/opengl32/unix_thunks.c index b424bac922b..c710d2141ec 100644 --- a/dlls/opengl32/unix_thunks.c +++ b/dlls/opengl32/unix_thunks.c @@ -27294,21 +27294,6 @@ static NTSTATUS wow64_wgl_wglGetPixelFormat( void *args ) return status; } -static NTSTATUS wow64_wgl_wglGetProcAddress( void *args ) -{ - struct - { - PTR32 lpszProc; - PTR32 ret; - } *params32 = args; - struct wglGetProcAddress_params params = - { - .lpszProc = ULongToPtr(params32->lpszProc), - }; - FIXME( "params32 %p, params %p stub!\n", params32, &params ); - return STATUS_NOT_IMPLEMENTED; -} - static NTSTATUS wow64_wgl_wglMakeCurrent( void *args ) { struct @@ -61088,6 +61073,7 @@ static NTSTATUS wow64_ext_wglSetPixelFormatWINE( void *args ) return status; } +extern NTSTATUS wow64_wgl_wglGetProcAddress( void *args ) DECLSPEC_HIDDEN; extern NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; const unixlib_entry_t __wine_unix_call_wow64_funcs[] = diff --git a/dlls/opengl32/unix_wgl.c b/dlls/opengl32/unix_wgl.c index 0daaf597ff3..7fd044182d7 100644 --- a/dlls/opengl32/unix_wgl.c +++ b/dlls/opengl32/unix_wgl.c @@ -542,6 +542,9 @@ static PROC WINAPI wrap_wglGetProcAddress( LPCSTR name ) *func_ptr = driver_func; } + /* Return the index into the extension registry instead of a useless + * function pointer, PE side will returns its own function pointers. + */ return (void *)(UINT_PTR)(found - extension_registry); } @@ -1057,6 +1060,23 @@ typedef ULONG PTR32; extern NTSTATUS ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; +NTSTATUS wow64_wgl_wglGetProcAddress( void *args ) +{ + struct + { + PTR32 lpszProc; + PTR32 ret; + } *params32 = args; + struct wglGetProcAddress_params params = + { + .lpszProc = ULongToPtr(params32->lpszProc), + }; + NTSTATUS status; + if ((status = wgl_wglGetProcAddress( &params ))) return status; + params32->ret = (UINT_PTR)params.ret; + return STATUS_SUCCESS; +} + NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) { struct

1 0

Rémi Bernon : opengl32: Implement wow64 thunk for wglMakeCurrent (et al.).
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: c24d7a23ad066e444cc3555d5f159b7b77c466b4 URL: https://gitlab.winehq.org/wine/wine/-/commit/c24d7a23ad066e444cc3555d5f159b… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Nov 14 22:51:46 2022 +0100 opengl32: Implement wow64 thunk for wglMakeCurrent (et al.). Keeping the 32bit TEB context and private data in sync. --- dlls/opengl32/make_opengl | 4 +++ dlls/opengl32/unix_thunks.c | 76 +++------------------------------------ dlls/opengl32/unix_wgl.c | 87 +++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 95 insertions(+), 72 deletions(-) diff --git a/dlls/opengl32/make_opengl b/dlls/opengl32/make_opengl index 312e7acc964..7803dd21287 100755 --- a/dlls/opengl32/make_opengl +++ b/dlls/opengl32/make_opengl @@ -170,7 +170,11 @@ my %manual_win_thunks = my %manual_wow64_thunks = ( "glPathGlyphIndexRangeNV" => 1, + "wglCreateContext" => 1, + "wglDeleteContext" => 1, "wglGetProcAddress" => 1, + "wglMakeContextCurrentARB" => 1, + "wglMakeCurrent" => 1, ); # diff --git a/dlls/opengl32/unix_thunks.c b/dlls/opengl32/unix_thunks.c index c710d2141ec..5accad6fc0b 100644 --- a/dlls/opengl32/unix_thunks.c +++ b/dlls/opengl32/unix_thunks.c @@ -27222,38 +27222,6 @@ static NTSTATUS wow64_wgl_wglCopyContext( void *args ) return status; } -static NTSTATUS wow64_wgl_wglCreateContext( void *args ) -{ - struct - { - PTR32 hDc; - PTR32 ret; - } *params32 = args; - struct wglCreateContext_params params = - { - .hDc = ULongToPtr(params32->hDc), - }; - FIXME( "params32 %p, params %p stub!\n", params32, &params ); - return STATUS_NOT_IMPLEMENTED; -} - -static NTSTATUS wow64_wgl_wglDeleteContext( void *args ) -{ - struct - { - PTR32 oldContext; - BOOL ret; - } *params32 = args; - struct wglDeleteContext_params params = - { - .oldContext = ULongToPtr(params32->oldContext), - }; - NTSTATUS status; - status = wgl_wglDeleteContext( &params ); - params32->ret = params.ret; - return status; -} - static NTSTATUS wow64_wgl_wglDescribePixelFormat( void *args ) { struct @@ -27294,25 +27262,6 @@ static NTSTATUS wow64_wgl_wglGetPixelFormat( void *args ) return status; } -static NTSTATUS wow64_wgl_wglMakeCurrent( void *args ) -{ - struct - { - PTR32 hDc; - PTR32 newContext; - BOOL ret; - } *params32 = args; - struct wglMakeCurrent_params params = - { - .hDc = ULongToPtr(params32->hDc), - .newContext = ULongToPtr(params32->newContext), - }; - NTSTATUS status; - status = wgl_wglMakeCurrent( &params ); - params32->ret = params.ret; - return status; -} - static NTSTATUS wow64_wgl_wglSetPixelFormat( void *args ) { struct @@ -60879,27 +60828,6 @@ static NTSTATUS wow64_ext_wglGetPixelFormatAttribivARB( void *args ) return status; } -static NTSTATUS wow64_ext_wglMakeContextCurrentARB( void *args ) -{ - struct - { - PTR32 hDrawDC; - PTR32 hReadDC; - PTR32 hglrc; - BOOL ret; - } *params32 = args; - struct wglMakeContextCurrentARB_params params = - { - .hDrawDC = ULongToPtr(params32->hDrawDC), - .hReadDC = ULongToPtr(params32->hReadDC), - .hglrc = ULongToPtr(params32->hglrc), - }; - NTSTATUS status; - status = ext_wglMakeContextCurrentARB( &params ); - params32->ret = params.ret; - return status; -} - static NTSTATUS wow64_ext_wglQueryCurrentRendererIntegerWINE( void *args ) { struct @@ -61073,8 +61001,12 @@ static NTSTATUS wow64_ext_wglSetPixelFormatWINE( void *args ) return status; } +extern NTSTATUS wow64_wgl_wglCreateContext( void *args ) DECLSPEC_HIDDEN; +extern NTSTATUS wow64_wgl_wglDeleteContext( void *args ) DECLSPEC_HIDDEN; extern NTSTATUS wow64_wgl_wglGetProcAddress( void *args ) DECLSPEC_HIDDEN; +extern NTSTATUS wow64_wgl_wglMakeCurrent( void *args ) DECLSPEC_HIDDEN; extern NTSTATUS wow64_ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; +extern NTSTATUS wow64_ext_wglMakeContextCurrentARB( void *args ) DECLSPEC_HIDDEN; const unixlib_entry_t __wine_unix_call_wow64_funcs[] = { diff --git a/dlls/opengl32/unix_wgl.c b/dlls/opengl32/unix_wgl.c index 7fd044182d7..7ba2045e384 100644 --- a/dlls/opengl32/unix_wgl.c +++ b/dlls/opengl32/unix_wgl.c @@ -1060,6 +1060,93 @@ typedef ULONG PTR32; extern NTSTATUS ext_glPathGlyphIndexRangeNV( void *args ) DECLSPEC_HIDDEN; +static inline void update_teb32_context(void) +{ + TEB *teb = NtCurrentTeb(); + void *teb32; + + if (!teb->WowTebOffset) return; + teb32 = (char *)teb + teb->WowTebOffset; + + ((TEB32 *)teb32)->glCurrentRC = (UINT_PTR)teb->glCurrentRC; + ((TEB32 *)teb32)->glReserved1[0] = (UINT_PTR)teb->glReserved1[0]; + ((TEB32 *)teb32)->glReserved1[1] = (UINT_PTR)teb->glReserved1[1]; +} + +NTSTATUS wow64_wgl_wglCreateContext( void *args ) +{ + struct + { + PTR32 hDc; + PTR32 ret; + } *params32 = args; + struct wglCreateContext_params params = + { + .hDc = ULongToPtr(params32->hDc), + }; + NTSTATUS status; + if ((status = wgl_wglCreateContext( &params ))) return status; + params32->ret = (UINT_PTR)params.ret; + return STATUS_SUCCESS; +} + +NTSTATUS wow64_wgl_wglDeleteContext( void *args ) +{ + struct + { + PTR32 oldContext; + BOOL ret; + } *params32 = args; + struct wglDeleteContext_params params = + { + .oldContext = ULongToPtr(params32->oldContext), + }; + NTSTATUS status; + if (!(status = wgl_wglDeleteContext( &params ))) update_teb32_context(); + params32->ret = params.ret; + return status; +} + +NTSTATUS wow64_wgl_wglMakeCurrent( void *args ) +{ + struct + { + PTR32 hDc; + PTR32 newContext; + BOOL ret; + } *params32 = args; + struct wglMakeCurrent_params params = + { + .hDc = ULongToPtr(params32->hDc), + .newContext = ULongToPtr(params32->newContext), + }; + NTSTATUS status; + if (!(status = wgl_wglMakeCurrent( &params ))) update_teb32_context(); + params32->ret = params.ret; + return status; +} + +NTSTATUS wow64_ext_wglMakeContextCurrentARB( void *args ) +{ + struct + { + PTR32 hDrawDC; + PTR32 hReadDC; + PTR32 hglrc; + BOOL ret; + } *params32 = args; + struct wglMakeContextCurrentARB_params params = + { + .hDrawDC = ULongToPtr(params32->hDrawDC), + .hReadDC = ULongToPtr(params32->hReadDC), + .hglrc = ULongToPtr(params32->hglrc), + }; + NTSTATUS status; + if (!(status = ext_wglMakeContextCurrentARB( &params ))) update_teb32_context(); + params32->ret = params.ret; + return status; +} + NTSTATUS wow64_wgl_wglGetProcAddress( void *args ) { struct

1 0

Alex Henrie : inetcomm: Use standard C functions for memory allocation in protocol.c.
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 02bb8401d7890f66f3f70e7a8010c7aa109034fa URL: https://gitlab.winehq.org/wine/wine/-/commit/02bb8401d7890f66f3f70e7a8010c7… Author: Alex Henrie <alexhenrie24(a)gmail.com> Date: Wed Nov 16 22:16:27 2022 -0700 inetcomm: Use standard C functions for memory allocation in protocol.c. --- dlls/inetcomm/protocol.c | 31 +++++++------------------------ 1 file changed, 7 insertions(+), 24 deletions(-) diff --git a/dlls/inetcomm/protocol.c b/dlls/inetcomm/protocol.c index 5cb0d53d253..c98139ad49e 100644 --- a/dlls/inetcomm/protocol.c +++ b/dlls/inetcomm/protocol.c @@ -25,7 +25,6 @@ #include "inetcomm_private.h" #include "wine/debug.h" -#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(inetcomm); @@ -62,22 +61,6 @@ typedef struct { static const WCHAR mhtml_prefixW[] = L"mhtml:"; static const WCHAR mhtml_separatorW[] = L"!x-usc:"; -static inline LPWSTR heap_strdupW(LPCWSTR str) -{ - LPWSTR ret = NULL; - - if(str) { - DWORD size; - - size = (lstrlenW(str)+1)*sizeof(WCHAR); - ret = heap_alloc(size); - if(ret) - memcpy(ret, str, size); - } - - return ret; -} - static HRESULT parse_mhtml_url(const WCHAR *url, mhtml_url_t *r) { const WCHAR *p; @@ -247,7 +230,7 @@ static ULONG WINAPI BindStatusCallback_Release(IBindStatusCallback *iface) IInternetProtocol_Release(&This->protocol->IInternetProtocol_iface); if(This->stream) IStream_Release(This->stream); - heap_free(This); + free(This); } return ref; @@ -419,8 +402,8 @@ static ULONG WINAPI MimeHtmlProtocol_Release(IUnknown *iface) IInternetProtocolSink_Release(This->sink); if(This->stream) IStream_Release(This->stream); - heap_free(This->location); - heap_free(This); + free(This->location); + free(This); } return ref; @@ -475,7 +458,7 @@ static HRESULT WINAPI MimeHtmlProtocol_Start(IInternetProtocol *iface, const WCH if(FAILED(hres)) return hres; - if(url.location && !(This->location = heap_strdupW(url.location))) + if(url.location && !(This->location = wcsdup(url.location))) return E_OUTOFMEMORY; hres = IInternetBindInfo_GetBindInfo(pOIBindInfo, &bindf, &bindinfo); @@ -488,7 +471,7 @@ static HRESULT WINAPI MimeHtmlProtocol_Start(IInternetProtocol *iface, const WCH IInternetProtocolSink_AddRef(This->sink = pOIProtSink); - binding = heap_alloc(FIELD_OFFSET(MimeHtmlBinding, url[url.mhtml_len+1])); + binding = malloc(FIELD_OFFSET(MimeHtmlBinding, url[url.mhtml_len+1])); if(!binding) return E_OUTOFMEMORY; memcpy(binding->url, url.mhtml, url.mhtml_len*sizeof(WCHAR)); @@ -496,7 +479,7 @@ static HRESULT WINAPI MimeHtmlProtocol_Start(IInternetProtocol *iface, const WCH hres = CreateURLMoniker(NULL, binding->url, &mon); if(FAILED(hres)) { - heap_free(binding); + free(binding); return hres; } @@ -729,7 +712,7 @@ HRESULT MimeHtmlProtocol_create(IUnknown *outer, void **obj) { MimeHtmlProtocol *protocol; - protocol = heap_alloc(sizeof(*protocol)); + protocol = malloc(sizeof(*protocol)); if(!protocol) return E_OUTOFMEMORY;

1 0

Zebediah Figura : setupapi/tests: Add more tests for buffer size handling in SetupDiGetDeviceInterfaceDetail().
by Alexandre Julliard 17 Nov '22

17 Nov '22

Module: wine Branch: master Commit: 60af599bb916260e8bb8a2f5d0111815f741fef4 URL: https://gitlab.winehq.org/wine/wine/-/commit/60af599bb916260e8bb8a2f5d01118… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Nov 16 19:28:25 2022 -0600 setupapi/tests: Add more tests for buffer size handling in SetupDiGetDeviceInterfaceDetail(). --- dlls/setupapi/tests/devinst.c | 34 +++++++++++++++++++++++++++------- 1 file changed, 27 insertions(+), 7 deletions(-) diff --git a/dlls/setupapi/tests/devinst.c b/dlls/setupapi/tests/devinst.c index c4eebdf0ab8..9d6a61c08c8 100644 --- a/dlls/setupapi/tests/devinst.c +++ b/dlls/setupapi/tests/devinst.c @@ -1192,7 +1192,7 @@ static void test_device_iface_detail(void) SP_DEVICE_INTERFACE_DETAIL_DATA_A *detail; SP_DEVICE_INTERFACE_DATA iface = {sizeof(iface)}; SP_DEVINFO_DATA device = {sizeof(device)}; - DWORD size = 0, expectedsize; + DWORD size = 0, expected_size; HDEVINFO set; BOOL ret; @@ -1231,37 +1231,57 @@ static void test_device_iface_detail(void) ok(!ret, "Expected failure.\n"); ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "Got unexpected error %#lx.\n", GetLastError()); - detail = heap_alloc(size); - expectedsize = FIELD_OFFSET(SP_DEVICE_INTERFACE_DETAIL_DATA_W, DevicePath[strlen(path) + 1]); + expected_size = FIELD_OFFSET(SP_DEVICE_INTERFACE_DETAIL_DATA_A, DevicePath[strlen(path) + 1]); + todo_wine ok(size == expected_size, "Expected size %lu, got %lu.\n", expected_size, size); + detail = heap_alloc(size * 2); detail->cbSize = 0; SetLastError(0xdeadbeef); - ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, size, &size, NULL); + size = 0xdeadbeef; + ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, expected_size * 2, &size, NULL); ok(!ret, "Expected failure.\n"); ok(GetLastError() == ERROR_INVALID_USER_BUFFER, "Got unexpected error %#lx.\n", GetLastError()); + ok(size == 0xdeadbeef, "Expected size %lu, got %lu.\n", expected_size, size); detail->cbSize = size; SetLastError(0xdeadbeef); - ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, size, &size, NULL); + size = 0xdeadbeef; + ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, expected_size * 2, &size, NULL); ok(!ret, "Expected failure.\n"); ok(GetLastError() == ERROR_INVALID_USER_BUFFER, "Got unexpected error %#lx.\n", GetLastError()); + ok(size == 0xdeadbeef, "Expected size %lu, got %lu.\n", expected_size, size); detail->cbSize = sizeof(SP_DEVICE_INTERFACE_DETAIL_DATA_A); SetLastError(0xdeadbeef); - ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, size, &size, NULL); + size = 0xdeadbeef; + ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, expected_size, &size, NULL); + todo_wine ok(ret, "Failed to get interface detail, error %#lx.\n", GetLastError()); + if (ret) + ok(!strcasecmp(path, detail->DevicePath), "Got unexpected path %s.\n", detail->DevicePath); + todo_wine ok(size == expected_size, "Expected size %lu, got %lu.\n", expected_size, size); + + SetLastError(0xdeadbeef); + size = 0xdeadbeef; + ret = SetupDiGetDeviceInterfaceDetailA(set, &iface, detail, expected_size * 2, &size, NULL); ok(ret, "Failed to get interface detail, error %#lx.\n", GetLastError()); ok(!strcasecmp(path, detail->DevicePath), "Got unexpected path %s.\n", detail->DevicePath); + todo_wine ok(size == expected_size, "Expected size %lu, got %lu.\n", expected_size, size); + + expected_size = FIELD_OFFSET(SP_DEVICE_INTERFACE_DETAIL_DATA_W, DevicePath[strlen(path) + 1]); + size = 0xdeadbeef; ret = SetupDiGetDeviceInterfaceDetailW(set, &iface, NULL, 0, &size, NULL); ok(!ret, "Expected failure.\n"); ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "Got unexpected error %#lx.\n", GetLastError()); - ok(size == expectedsize, "Got unexpected size %ld.\n", size); + ok(size == expected_size, "Expected size %lu, got %lu.\n", expected_size, size); memset(&device, 0, sizeof(device)); device.cbSize = sizeof(device); + size = 0xdeadbeef; ret = SetupDiGetDeviceInterfaceDetailW(set, &iface, NULL, 0, &size, &device); ok(!ret, "Expected failure.\n"); ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "Got unexpected error %#lx.\n", GetLastError()); + ok(size == expected_size, "Expected size %lu, got %lu.\n", expected_size, size); ok(IsEqualGUID(&device.ClassGuid, &guid), "Got unexpected class %s.\n", wine_dbgstr_guid(&device.ClassGuid)); heap_free(detail);

1 0