January 2019 - Wine-commits

Zhiyi Zhang : vkd3d: Implement d3d12_query_heap_SetName().
by Alexandre Julliard 30 Jan '19

30 Jan '19

Module: vkd3d Branch: master Commit: d0e2802a3532b9c665528f2fd737ce2e2f6a9bc0 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=d0e2802a3532b9c665528f2… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Jan 30 01:02:31 2019 +0330 vkd3d: Implement d3d12_query_heap_SetName(). Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- Makefile.am | 4 +- include/private/vkd3d_utf8.h | 26 +++++++ libs/vkd3d-common/utf8.c | 162 +++++++++++++++++++++++++++++++++++++++++++ libs/vkd3d/device.c | 1 + libs/vkd3d/resource.c | 5 +- libs/vkd3d/utils.c | 29 ++++++++ libs/vkd3d/vkd3d_private.h | 5 ++ libs/vkd3d/vulkan_procs.h | 2 + 8 files changed, 231 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index 7869b2e..487d2ea 100644 --- a/Makefile.am +++ b/Makefile.am @@ -59,7 +59,8 @@ noinst_LTLIBRARIES = libvkd3d-common.la libvkd3d_common_la_SOURCES = \ include/private/vkd3d_debug.h \ libs/vkd3d-common/debug.c \ - libs/vkd3d-common/memory.c + libs/vkd3d-common/memory.c \ + libs/vkd3d-common/utf8.c lib_LTLIBRARIES = libvkd3d-shader.la libvkd3d.la libvkd3d-utils.la @@ -88,6 +89,7 @@ libvkd3d_la_SOURCES = \ include/private/vkd3d_common.h \ include/private/vkd3d_debug.h \ include/private/vkd3d_memory.h \ + include/private/vkd3d_utf8.h \ include/private/vkd3d_test.h \ include/vkd3d_d3d12.idl \ include/vkd3d_d3dcommon.idl \ diff --git a/include/private/vkd3d_utf8.h b/include/private/vkd3d_utf8.h new file mode 100644 index 0000000..ab32884 --- /dev/null +++ b/include/private/vkd3d_utf8.h @@ -0,0 +1,26 @@ +/* + * Copyright 2019 Zhiyi Zhang for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#ifndef __VKD3D_UTF8_H +#define __VKD3D_UTF8_H + +#include "vkd3d_common.h" + +char *vkd3d_strdup_w_utf8(const WCHAR *wstr, size_t wchar_size) DECLSPEC_HIDDEN; + +#endif /* __VKD3D_UTF8_H */ diff --git a/libs/vkd3d-common/utf8.c b/libs/vkd3d-common/utf8.c new file mode 100644 index 0000000..e48316d --- /dev/null +++ b/libs/vkd3d-common/utf8.c @@ -0,0 +1,162 @@ +/* + * Copyright 2000 Alexandre Julliard + * Copyright 2019 Zhiyi Zhang for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "vkd3d_memory.h" +#include "vkd3d_utf8.h" + +#include <inttypes.h> + +static size_t vkd3d_utf8_len(uint32_t c) +{ + /* 0x00-0x7f: 1 byte */ + if (c < 0x80) + return 1; + /* 0x80-0x7ff: 2 bytes */ + if (c < 0x800) + return 2; + /* 0x800-0xffff: 3 bytes */ + if (c < 0x10000) + return 3; + /* 0x10000-0x10ffff: 4 bytes */ + return 4; +} + +static void vkd3d_utf8_append(char **dst, uint32_t c) +{ + char *d = *dst; + + /* 0x00-0x7f: 1 byte */ + if (c < 0x80) + { + d[0] = c; + *dst += 1; + return; + } + + /* 0x80-0x7ff: 2 bytes */ + if (c < 0x800) + { + d[1] = 0x80 | (c & 0x3f); + c >>= 6; + d[0] = 0xc0 | c; + *dst += 2; + return; + } + + /* 0x800-0xffff: 3 bytes */ + if (c < 0x10000) /* 0x800-0xffff: 3 bytes */ + { + d[2] = 0x80 | (c & 0x3f); + c >>= 6; + d[1] = 0x80 | (c & 0x3f); + c >>= 6; + d[0] = 0xe0 | c; + *dst += 3; + return; + } + + /* 0x10000-0x10ffff: 4 bytes */ + d[3] = 0x80 | (c & 0x3f); + c >>= 6; + d[2] = 0x80 | (c & 0x3f); + c >>= 6; + d[1] = 0x80 | (c & 0x3f); + c >>= 6; + d[0] = 0xf0 | c; + *dst += 4; +} + +static uint32_t vkd3d_utf16_read(const uint16_t **src) +{ + const uint16_t *s = *src; + + if (s[0] < 0xd800 || s[0] > 0xdfff) /* Not a surrogate pair. */ + { + *src += 1; + return s[0]; + } + + if (s[0] > 0xdbff /* Invalid high surrogate. */ + || s[1] < 0xdc00 || s[1] > 0xdfff) /* Invalid low surrogate. */ + { + *src += 1; + return 0; + } + + *src += 2; + return 0x10000 + ((s[0] & 0x3ff) << 10) + (s[1] & 0x3ff); +} + +static char *vkd3d_strdup_w16_utf8(const uint16_t *wstr) +{ + const uint16_t *src = wstr; + size_t dst_size = 0; + uint32_t c; + char *dst; + + while (*src) + { + if (!(c = vkd3d_utf16_read(&src))) + continue; + dst_size += vkd3d_utf8_len(c); + } + ++dst_size; + + if (!(dst = vkd3d_malloc(dst_size))) + return NULL; + + src = wstr; + while (*src) + { + if (!(c = vkd3d_utf16_read(&src))) + continue; + vkd3d_utf8_append(&dst, c); + } + *dst = 0; + + return dst; +} + +static char *vkd3d_strdup_w32_utf8(const uint32_t *wstr) +{ + const uint32_t *src = wstr; + size_t dst_size = 0; + char *dst; + + while (*src) + dst_size += vkd3d_utf8_len(*src++); + ++dst_size; + + if (!(dst = vkd3d_malloc(dst_size))) + return NULL; + + src = wstr; + while (*src) + vkd3d_utf8_append(&dst, *src++); + *dst = 0; + + return dst; +} + +char *vkd3d_strdup_w_utf8(const WCHAR *wstr, size_t wchar_size) +{ + if (wchar_size == 2) + return vkd3d_strdup_w16_utf8((const uint16_t *)wstr); + return vkd3d_strdup_w32_utf8((const uint32_t *)wstr); +} diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c index 64c8391..4b5f522 100644 --- a/libs/vkd3d/device.c +++ b/libs/vkd3d/device.c @@ -84,6 +84,7 @@ static const char * const required_device_extensions[] = static const struct vkd3d_optional_extension_info optional_device_extensions[] = { {VK_KHR_PUSH_DESCRIPTOR_EXTENSION_NAME, offsetof(struct vkd3d_vulkan_info, KHR_push_descriptor)}, + {VK_EXT_DEBUG_MARKER_EXTENSION_NAME, offsetof(struct vkd3d_vulkan_info, EXT_debug_marker)}, {VK_EXT_TRANSFORM_FEEDBACK_EXTENSION_NAME, offsetof(struct vkd3d_vulkan_info, EXT_transform_feedback)}, {VK_EXT_VERTEX_ATTRIBUTE_DIVISOR_EXTENSION_NAME, diff --git a/libs/vkd3d/resource.c b/libs/vkd3d/resource.c index 6eba450..158240d 100644 --- a/libs/vkd3d/resource.c +++ b/libs/vkd3d/resource.c @@ -2730,9 +2730,10 @@ static HRESULT STDMETHODCALLTYPE d3d12_query_heap_SetName(ID3D12QueryHeap *iface { struct d3d12_query_heap *heap = impl_from_ID3D12QueryHeap(iface); - FIXME("iface %p, name %s stub!\n", iface, debugstr_w(name, heap->device->wchar_size)); + TRACE("iface %p, name %s.\n", iface, debugstr_w(name, heap->device->wchar_size)); - return E_NOTIMPL; + return vkd3d_set_vk_object_name(heap->device, (uint64_t)heap->vk_query_pool, + VK_DEBUG_REPORT_OBJECT_TYPE_QUERY_POOL_EXT, name); } static HRESULT STDMETHODCALLTYPE d3d12_query_heap_GetDevice(ID3D12QueryHeap *iface, diff --git a/libs/vkd3d/utils.c b/libs/vkd3d/utils.c index 0144fdc..27333f2 100644 --- a/libs/vkd3d/utils.c +++ b/libs/vkd3d/utils.c @@ -642,3 +642,32 @@ HRESULT vkd3d_set_private_data_interface(struct vkd3d_private_store *store, pthread_mutex_unlock(&store->mutex); return hr; } + +HRESULT vkd3d_set_vk_object_name(struct d3d12_device *device, uint64_t vk_object, + VkDebugReportObjectTypeEXT vk_object_type, const WCHAR *name) +{ + const struct vkd3d_vk_device_procs *vk_procs = &device->vk_procs; + VkDebugMarkerObjectNameInfoEXT info; + char *name_utf8; + VkResult vr; + + if (!name) + return E_INVALIDARG; + + if (!device->vk_info.EXT_debug_marker) + return S_OK; + + if (!(name_utf8 = vkd3d_strdup_w_utf8(name, device->wchar_size))) + return E_OUTOFMEMORY; + + info.sType = VK_STRUCTURE_TYPE_DEBUG_MARKER_OBJECT_NAME_INFO_EXT; + info.pNext = NULL; + info.objectType = vk_object_type; + info.object = vk_object; + info.pObjectName = name_utf8; + vr = VK_CALL(vkDebugMarkerSetObjectNameEXT(device->vk_device, &info)); + + vkd3d_free(name_utf8); + + return hresult_from_vk_result(vr); +} diff --git a/libs/vkd3d/vkd3d_private.h b/libs/vkd3d/vkd3d_private.h index 73825bb..69e2cf7 100644 --- a/libs/vkd3d/vkd3d_private.h +++ b/libs/vkd3d/vkd3d_private.h @@ -25,6 +25,7 @@ #include "vkd3d_common.h" #include "vkd3d_memory.h" +#include "vkd3d_utf8.h" #include "list.h" #include "rbtree.h" @@ -87,6 +88,7 @@ struct vkd3d_vulkan_info bool EXT_debug_report; /* device extensions */ bool KHR_push_descriptor; + bool EXT_debug_marker; bool EXT_transform_feedback; bool EXT_vertex_attribute_divisor; @@ -1026,4 +1028,7 @@ static inline void vkd3d_set_thread_name(const char *name) #endif } +HRESULT vkd3d_set_vk_object_name(struct d3d12_device *device, uint64_t vk_object, + VkDebugReportObjectTypeEXT vk_object_type, const WCHAR *name) DECLSPEC_HIDDEN; + #endif /* __VKD3D_PRIVATE_H */ diff --git a/libs/vkd3d/vulkan_procs.h b/libs/vkd3d/vulkan_procs.h index be983ec..b62af4e 100644 --- a/libs/vkd3d/vulkan_procs.h +++ b/libs/vkd3d/vulkan_procs.h @@ -174,6 +174,8 @@ VK_DEVICE_PFN(vkSetEvent) VK_DEVICE_PFN(vkUnmapMemory) VK_DEVICE_PFN(vkUpdateDescriptorSets) VK_DEVICE_PFN(vkWaitForFences) +/* VK_EXT_debug_marker */ +VK_DEVICE_EXT_PFN(vkDebugMarkerSetObjectNameEXT) /* VK_KHR_push_descriptor */ VK_DEVICE_EXT_PFN(vkCmdPushDescriptorSetKHR) /* VK_EXT_transform_feedback */

1 0

Zhiyi Zhang : vkd3d: Implement d3d12_resource_SetName().
by Alexandre Julliard 30 Jan '19

30 Jan '19

Module: vkd3d Branch: master Commit: 875577bb8998b16955a8433d04a19921486fb5b2 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=875577bb8998b16955a8433… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Jan 30 01:02:32 2019 +0330 vkd3d: Implement d3d12_resource_SetName(). Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d/resource.c | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/libs/vkd3d/resource.c b/libs/vkd3d/resource.c index 158240d..3f44712 100644 --- a/libs/vkd3d/resource.c +++ b/libs/vkd3d/resource.c @@ -842,10 +842,23 @@ static HRESULT STDMETHODCALLTYPE d3d12_resource_SetPrivateDataInterface(ID3D12Re static HRESULT STDMETHODCALLTYPE d3d12_resource_SetName(ID3D12Resource *iface, const WCHAR *name) { struct d3d12_resource *resource = impl_from_ID3D12Resource(iface); + HRESULT hr; - FIXME("iface %p, name %s stub!\n", iface, debugstr_w(name, resource->device->wchar_size)); + TRACE("iface %p, name %s.\n", iface, debugstr_w(name, resource->device->wchar_size)); - return E_NOTIMPL; + if (resource->vk_memory) + { + if (FAILED(hr = vkd3d_set_vk_object_name(resource->device, (uint64_t)resource->vk_memory, + VK_DEBUG_REPORT_OBJECT_TYPE_DEVICE_MEMORY_EXT, name))) + return hr; + } + + if (d3d12_resource_is_buffer(resource)) + return vkd3d_set_vk_object_name(resource->device, (uint64_t)resource->u.vk_buffer, + VK_DEBUG_REPORT_OBJECT_TYPE_BUFFER_EXT, name); + else + return vkd3d_set_vk_object_name(resource->device, (uint64_t)resource->u.vk_image, + VK_DEBUG_REPORT_OBJECT_TYPE_IMAGE_EXT, name); } static HRESULT STDMETHODCALLTYPE d3d12_resource_GetDevice(ID3D12Resource *iface,

1 0

Zhiyi Zhang : vkd3d: Implement d3d12_heap_SetName().
by Alexandre Julliard 30 Jan '19

30 Jan '19

Module: vkd3d Branch: master Commit: 1372bebc862b44d8c11730423280f33f0baabed1 URL: https://source.winehq.org/git/vkd3d.git/?a=commit;h=1372bebc862b44d8c117304… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Jan 30 01:02:33 2019 +0330 vkd3d: Implement d3d12_heap_SetName(). Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- libs/vkd3d/resource.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libs/vkd3d/resource.c b/libs/vkd3d/resource.c index 3f44712..ced232d 100644 --- a/libs/vkd3d/resource.c +++ b/libs/vkd3d/resource.c @@ -192,9 +192,10 @@ static HRESULT STDMETHODCALLTYPE d3d12_heap_SetName(ID3D12Heap *iface, const WCH { struct d3d12_heap *heap = impl_from_ID3D12Heap(iface); - FIXME("iface %p, name %s stub!\n", iface, debugstr_w(name, heap->device->wchar_size)); + TRACE("iface %p, name %s.\n", iface, debugstr_w(name, heap->device->wchar_size)); - return E_NOTIMPL; + return vkd3d_set_vk_object_name(heap->device, (uint64_t)heap->vk_memory, + VK_DEBUG_REPORT_OBJECT_TYPE_DEVICE_MEMORY_EXT, name); } static HRESULT STDMETHODCALLTYPE d3d12_heap_GetDevice(ID3D12Heap *iface,

1 0

Fabian Maurer : ucrtbase: Forward a few functions for dxil.dll and pkgmgr.exe.
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: e9c90afc065d7a3c57b532aa214b9b4b4546ee83 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e9c90afc065d7a3c57b532aa… Author: Fabian Maurer <dark.shadow4(a)web.de> Date: Sun Jan 27 16:06:01 2019 +0100 ucrtbase: Forward a few functions for dxil.dll and pkgmgr.exe. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=44469 Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-crt-private-l1-1-0.spec | 38 +++++++++++----------- dlls/ucrtbase/ucrtbase.spec | 38 +++++++++++----------- 2 files changed, 38 insertions(+), 38 deletions(-) diff --git a/dlls/api-ms-win-crt-private-l1-1-0/api-ms-win-crt-private-l1-1-0.spec b/dlls/api-ms-win-crt-private-l1-1-0/api-ms-win-crt-private-l1-1-0.spec index 9bddd3d..77be3a5 100644 --- a/dlls/api-ms-win-crt-private-l1-1-0/api-ms-win-crt-private-l1-1-0.spec +++ b/dlls/api-ms-win-crt-private-l1-1-0/api-ms-win-crt-private-l1-1-0.spec @@ -127,11 +127,11 @@ @ stub _o___libm_sse2_sinf @ stub _o___libm_sse2_tan @ stub _o___libm_sse2_tanf -@ stub _o___p___argc +@ cdecl _o___p___argc() ucrtbase._o___p___argc @ stub _o___p___argv -@ stub _o___p___wargv +@ cdecl _o___p___wargv() ucrtbase._o___p___wargv @ stub _o___p__acmdln -@ stub _o___p__commode +@ cdecl _o___p__commode() ucrtbase._o___p__commode @ stub _o___p__environ @ stub _o___p__fmode @ stub _o___p__mbcasemap @@ -144,7 +144,7 @@ @ stub _o___pwctype_func @ stub _o___std_exception_copy @ stub _o___std_exception_destroy -@ stub _o___std_type_info_destroy_list +@ cdecl _o___std_type_info_destroy_list(ptr) ucrtbase._o___std_type_info_destroy_list @ stub _o___std_type_info_name @ stub _o___stdio_common_vfprintf @ stub _o___stdio_common_vfprintf_p @@ -207,15 +207,15 @@ @ stub _o__chsize_s @ stub _o__close @ stub _o__commit -@ stub _o__configthreadlocale +@ cdecl _o__configthreadlocale(long) ucrtbase._o__configthreadlocale @ stub _o__configure_narrow_argv -@ stub _o__configure_wide_argv -@ stub _o__controlfp_s +@ cdecl _o__configure_wide_argv(long) ucrtbase._o__configure_wide_argv +@ cdecl _o__controlfp_s(ptr long long) ucrtbase._o__controlfp_s @ stub _o__cputs @ stub _o__cputws @ stub _o__creat @ stub _o__create_locale -@ stub _o__crt_atexit +@ cdecl _o__crt_atexit(ptr) ucrtbase._o__crt_atexit @ stub _o__ctime32_s @ stub _o__ctime64_s @ stub _o__cwait @@ -319,7 +319,7 @@ @ stub _o__get_fmode @ stub _o__get_heap_handle @ stub _o__get_initial_narrow_environment -@ stub _o__get_initial_wide_environment +@ cdecl _o__get_initial_wide_environment() ucrtbase._o__get_initial_wide_environment @ stub _o__get_invalid_parameter_handler @ stub _o__get_narrow_winmain_command_line @ stub _o__get_osfhandle @@ -366,7 +366,7 @@ @ stub _o__i64tow_s @ stub _o__initialize_narrow_environment @ cdecl _o__initialize_onexit_table(ptr) ucrtbase._o__initialize_onexit_table -@ stub _o__initialize_wide_environment +@ cdecl _o__initialize_wide_environment() ucrtbase._o__initialize_wide_environment @ stub _o__invalid_parameter_noinfo @ stub _o__invalid_parameter_noinfo_noreturn @ stub _o__isatty @@ -660,16 +660,16 @@ @ stub _o__scalbf @ stub _o__searchenv @ stub _o__searchenv_s -@ stub _o__seh_filter_dll -@ stub _o__seh_filter_exe +@ cdecl _o__seh_filter_dll(long ptr) ucrtbase._o__seh_filter_dll +@ cdecl _o__seh_filter_exe(long ptr) ucrtbase._o__seh_filter_exe @ stub _o__set_abort_behavior -@ stub _o__set_app_type +@ cdecl _o__set_app_type(long) ucrtbase._o__set_app_type @ stub _o__set_doserrno @ stub _o__set_errno -@ stub _o__set_fmode +@ cdecl _o__set_fmode(long) ucrtbase._o__set_fmode @ stub _o__set_invalid_parameter_handler @ stub _o__set_new_handler -@ stub _o__set_new_mode +@ cdecl _o__set_new_mode(long) ucrtbase._o__set_new_mode @ stub _o__set_thread_local_invalid_parameter_handler @ stub _o__seterrormode @ stub _o__setmbcp @@ -919,7 +919,7 @@ @ stub _o_bsearch @ stub _o_bsearch_s @ stub _o_btowc -@ stub _o_calloc +@ cdecl _o_calloc(long long) ucrtbase._o_calloc @ stub _o_cbrt @ stub _o_cbrtf @ stub _o_ceil @@ -936,7 +936,7 @@ @ stub _o_erfcl @ stub _o_erff @ stub _o_erfl -@ stub _o_exit +@ cdecl _o_exit(long) ucrtbase._o_exit @ stub _o_exp @ stub _o_exp2 @ stub _o_exp2f @@ -967,7 +967,7 @@ @ stub _o_fputws @ stub _o_fread @ stub _o_fread_s -@ stub _o_free +@ cdecl _o_free(ptr) ucrtbase._o_free @ stub _o_freopen @ stub _o_freopen_s @ stub _o_frexp @@ -1142,7 +1142,7 @@ @ stub _o_tmpfile_s @ stub _o_tmpnam_s @ stub _o_tolower -@ stub _o_toupper +@ cdecl _o_toupper(long) ucrtbase._o_toupper @ stub _o_towlower @ stub _o_towupper @ stub _o_ungetc diff --git a/dlls/ucrtbase/ucrtbase.spec b/dlls/ucrtbase/ucrtbase.spec index cb45aa5..2d1d8da 100644 --- a/dlls/ucrtbase/ucrtbase.spec +++ b/dlls/ucrtbase/ucrtbase.spec @@ -791,11 +791,11 @@ @ stub _o___libm_sse2_sinf @ stub _o___libm_sse2_tan @ stub _o___libm_sse2_tanf -@ stub _o___p___argc +@ cdecl _o___p___argc() MSVCRT___p___argc @ stub _o___p___argv -@ stub _o___p___wargv +@ cdecl _o___p___wargv() MSVCRT___p___wargv @ stub _o___p__acmdln -@ stub _o___p__commode +@ cdecl _o___p__commode() __p__commode @ stub _o___p__environ @ stub _o___p__fmode @ stub _o___p__mbcasemap @@ -808,7 +808,7 @@ @ stub _o___pwctype_func @ stub _o___std_exception_copy @ stub _o___std_exception_destroy -@ stub _o___std_type_info_destroy_list +@ cdecl _o___std_type_info_destroy_list(ptr) MSVCRT_type_info_destroy_list @ stub _o___std_type_info_name @ stub _o___stdio_common_vfprintf @ stub _o___stdio_common_vfprintf_p @@ -871,15 +871,15 @@ @ stub _o__chsize_s @ stub _o__close @ stub _o__commit -@ stub _o__configthreadlocale +@ cdecl _o__configthreadlocale(long) _configthreadlocale @ stub _o__configure_narrow_argv -@ stub _o__configure_wide_argv -@ stub _o__controlfp_s +@ cdecl _o__configure_wide_argv(long) _configure_wide_argv +@ cdecl _o__controlfp_s(ptr long long) _controlfp_s @ stub _o__cputs @ stub _o__cputws @ stub _o__creat @ stub _o__create_locale -@ stub _o__crt_atexit +@ cdecl _o__crt_atexit(ptr) MSVCRT__crt_atexit @ stub _o__ctime32_s @ stub _o__ctime64_s @ stub _o__cwait @@ -983,7 +983,7 @@ @ stub _o__get_fmode @ stub _o__get_heap_handle @ stub _o__get_initial_narrow_environment -@ stub _o__get_initial_wide_environment +@ cdecl _o__get_initial_wide_environment() _get_initial_wide_environment @ stub _o__get_invalid_parameter_handler @ stub _o__get_narrow_winmain_command_line @ stub _o__get_osfhandle @@ -1030,7 +1030,7 @@ @ stub _o__i64tow_s @ stub _o__initialize_narrow_environment @ cdecl _o__initialize_onexit_table(ptr) MSVCRT__initialize_onexit_table -@ stub _o__initialize_wide_environment +@ cdecl _o__initialize_wide_environment() _initialize_wide_environment @ stub _o__invalid_parameter_noinfo @ stub _o__invalid_parameter_noinfo_noreturn @ stub _o__isatty @@ -1324,16 +1324,16 @@ @ stub _o__scalbf @ stub _o__searchenv @ stub _o__searchenv_s -@ stub _o__seh_filter_dll -@ stub _o__seh_filter_exe +@ cdecl _o__seh_filter_dll(long ptr) __CppXcptFilter +@ cdecl _o__seh_filter_exe(long ptr) _XcptFilter @ stub _o__set_abort_behavior -@ stub _o__set_app_type +@ cdecl _o__set_app_type(long) MSVCRT___set_app_type @ stub _o__set_doserrno @ stub _o__set_errno -@ stub _o__set_fmode +@ cdecl _o__set_fmode(long) MSVCRT__set_fmode @ stub _o__set_invalid_parameter_handler @ stub _o__set_new_handler -@ stub _o__set_new_mode +@ cdecl _o__set_new_mode(long) MSVCRT__set_new_mode @ stub _o__set_thread_local_invalid_parameter_handler @ stub _o__seterrormode @ stub _o__setmbcp @@ -1583,7 +1583,7 @@ @ stub _o_bsearch @ stub _o_bsearch_s @ stub _o_btowc -@ stub _o_calloc +@ cdecl _o_calloc(long long) MSVCRT_calloc @ stub _o_cbrt @ stub _o_cbrtf @ stub _o_ceil @@ -1600,7 +1600,7 @@ @ stub _o_erfcl @ stub _o_erff @ stub _o_erfl -@ stub _o_exit +@ cdecl _o_exit(long) MSVCRT_exit @ stub _o_exp @ stub _o_exp2 @ stub _o_exp2f @@ -1631,7 +1631,7 @@ @ stub _o_fputws @ stub _o_fread @ stub _o_fread_s -@ stub _o_free +@ cdecl _o_free(ptr) MSVCRT_free @ stub _o_freopen @ stub _o_freopen_s @ stub _o_frexp @@ -1807,7 +1807,7 @@ @ stub _o_tmpfile_s @ stub _o_tmpnam_s @ stub _o_tolower -@ stub _o_toupper +@ cdecl _o_toupper(long) MSVCRT__toupper @ stub _o_towlower @ stub _o_towupper @ stub _o_ungetc

1 0

Sven Baars : wsdapi: Fix some memory leaks (Valgrind).
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: 18ac7e15aab3343b6c6d4dc56a33ea7d24e84f0f URL: https://source.winehq.org/git/wine.git/?a=commit;h=18ac7e15aab3343b6c6d4dc5… Author: Sven Baars <sven.wine(a)gmail.com> Date: Thu Jan 24 11:59:45 2019 +0100 wsdapi: Fix some memory leaks (Valgrind). Signed-off-by: Sven Baars <sven.wine(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/soap.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/wsdapi/soap.c b/dlls/wsdapi/soap.c index 2ecc013..aa8d15f 100644 --- a/dlls/wsdapi/soap.c +++ b/dlls/wsdapi/soap.c @@ -1426,6 +1426,9 @@ static HRESULT ws_element_to_wsdxml_element(WS_XML_READER *reader, IWSDXMLContex if (FAILED(ret)) goto cleanup; WSDXMLAddChild(cur_element, element); + WSDFreeLinkedMemory(name); + name = NULL; + cur_wsd_attrib = NULL; /* Add attributes */ @@ -1462,6 +1465,9 @@ static HRESULT ws_element_to_wsdxml_element(WS_XML_READER *reader, IWSDXMLContex new_wsd_attrib->Element = cur_element; new_wsd_attrib->Next = NULL; + WSDAttachLinkedMemory(new_wsd_attrib, name); + name = NULL; + if (cur_wsd_attrib == NULL) element->FirstAttribute = new_wsd_attrib; else @@ -1524,6 +1530,7 @@ outofmemory: cleanup: /* Free uri and element_name if applicable */ WSDFreeLinkedMemory(uri); + WSDFreeLinkedMemory(name); return ret; } @@ -1712,10 +1719,10 @@ HRESULT read_message(IWSDiscoveryPublisherImpl *impl, const char *xml, int xml_l IWSDXMLContext *context = NULL; WS_XML_STRING *soap_uri = NULL; const WS_XML_NODE *node; - WS_XML_READER *reader; + WS_XML_READER *reader = NULL; LPCWSTR value = NULL; LPWSTR uri, prefix; - WS_HEAP *heap; + WS_HEAP *heap = NULL; HRESULT ret; int i; @@ -1945,6 +1952,8 @@ cleanup: free_xml_string(soap_uri); WSDFreeLinkedMemory(soap_msg); if (context != NULL) IWSDXMLContext_Release(context); + if (reader != NULL) WsFreeReader(reader); + if (heap != NULL) WsFreeHeap(heap); return ret; }

1 0

Eric Bissonnette : dbghelp: Support full memory dumps.
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: b90bbcbe750f6af0ce13f2577984f4818e05a013 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b90bbcbe750f6af0ce13f257… Author: Eric Bissonnette <ebisso.dev(a)gmail.com> Date: Fri Jan 25 21:12:02 2019 -0500 dbghelp: Support full memory dumps. Add support of flag MinidumpWithFullMemory in function MinidumpWriteDump. A Memory64ListStream is added to the minidump streams and all memory regions of the process with MEM_COMMIT state are written to the last part of the minidump file. Signed-off-by: Eric Bissonnette <ebisso.dev(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dbghelp/dbghelp_private.h | 9 +++ dlls/dbghelp/minidump.c | 129 ++++++++++++++++++++++++++++++++++++++--- include/dbghelp.h | 13 +++++ 3 files changed, 144 insertions(+), 7 deletions(-) diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index ae8bc50..8083408 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -471,6 +471,12 @@ struct dump_memory ULONG rva; }; +struct dump_memory64 +{ + ULONG64 base; + ULONG64 size; +}; + struct dump_module { unsigned is_elf; @@ -509,6 +515,9 @@ struct dump_context struct dump_memory* mem; unsigned num_mem; unsigned alloc_mem; + struct dump_memory64* mem64; + unsigned num_mem64; + unsigned alloc_mem64; /* callback information */ MINIDUMP_CALLBACK_INFORMATION* cb; }; diff --git a/dlls/dbghelp/minidump.c b/dlls/dbghelp/minidump.c index 04dc775..b096476 100644 --- a/dlls/dbghelp/minidump.c +++ b/dlls/dbghelp/minidump.c @@ -294,6 +294,49 @@ static BOOL fetch_macho_module_info_cb(const WCHAR* name, unsigned long base, return TRUE; } +void minidump_add_memory64_block(struct dump_context* dc, ULONG64 base, ULONG64 size) +{ + if (!dc->mem64) + { + dc->alloc_mem64 = 32; + dc->mem64 = HeapAlloc(GetProcessHeap(), 0, dc->alloc_mem64 * sizeof(*dc->mem64)); + } + else if (dc->num_mem64 >= dc->alloc_mem64) + { + dc->alloc_mem64 *= 2; + dc->mem64 = HeapReAlloc(GetProcessHeap(), 0, dc->mem64, + dc->alloc_mem64 * sizeof(*dc->mem64)); + } + if (dc->mem64) + { + dc->mem64[dc->num_mem64].base = base; + dc->mem64[dc->num_mem64].size = size; + dc->num_mem64++; + } + else dc->num_mem64 = dc->alloc_mem64 = 0; +} + +static void fetch_memory64_info(struct dump_context* dc) +{ + ULONG_PTR addr; + MEMORY_BASIC_INFORMATION mbi; + + addr = 0; + while (VirtualQueryEx(dc->hProcess, (LPCVOID)addr, &mbi, sizeof(mbi)) != 0) + { + /* Memory regions with state MEM_COMMIT will be added to the dump */ + if (mbi.State == MEM_COMMIT) + { + minidump_add_memory64_block(dc, (ULONG_PTR)mbi.BaseAddress, mbi.RegionSize); + } + + if ((addr + mbi.RegionSize) < addr) + break; + + addr = (ULONG_PTR)mbi.BaseAddress + mbi.RegionSize; + } +} + static void fetch_modules_info(struct dump_context* dc) { EnumerateLoadedModulesW64(dc->hProcess, fetch_pe_module_info_cb, dc); @@ -831,6 +874,60 @@ static unsigned dump_memory_info(struct dump_context* dc) return sz; } +/****************************************************************** + * dump_memory64_info + * + * dumps information about the memory of the process (virtual memory) + */ +static unsigned dump_memory64_info(struct dump_context* dc) +{ + MINIDUMP_MEMORY64_LIST mdMem64List; + MINIDUMP_MEMORY_DESCRIPTOR64 mdMem64; + DWORD written; + unsigned i, len, sz; + RVA rva_base; + char tmp[1024]; + ULONG64 pos; + LARGE_INTEGER filepos; + + sz = sizeof(mdMem64List.NumberOfMemoryRanges) + + sizeof(mdMem64List.BaseRva) + + dc->num_mem64 * sizeof(mdMem64); + + mdMem64List.NumberOfMemoryRanges = dc->num_mem64; + mdMem64List.BaseRva = dc->rva + sz; + + append(dc, &mdMem64List.NumberOfMemoryRanges, + sizeof(mdMem64List.NumberOfMemoryRanges)); + append(dc, &mdMem64List.BaseRva, + sizeof(mdMem64List.BaseRva)); + + rva_base = dc->rva; + dc->rva += dc->num_mem64 * sizeof(mdMem64); + + /* dc->rva is not updated past this point. The end of the dump + * is just the full memory data. */ + filepos.QuadPart = dc->rva; + for (i = 0; i < dc->num_mem64; i++) + { + mdMem64.StartOfMemoryRange = dc->mem64[i].base; + mdMem64.DataSize = dc->mem64[i].size; + SetFilePointerEx(dc->hFile, filepos, NULL, FILE_BEGIN); + for (pos = 0; pos < dc->mem64[i].size; pos += sizeof(tmp)) + { + len = min(dc->mem64[i].size - pos, sizeof(tmp)); + if (ReadProcessMemory(dc->hProcess, + (void*)(ULONG_PTR)(dc->mem64[i].base + pos), + tmp, len, NULL)) + WriteFile(dc->hFile, tmp, len, &written, NULL); + } + filepos.QuadPart += mdMem64.DataSize; + writeat(dc, rva_base + i * sizeof(mdMem64), &mdMem64, sizeof(mdMem64)); + } + + return sz; +} + static unsigned dump_misc_info(struct dump_context* dc) { MINIDUMP_MISC_INFO mmi; @@ -876,6 +973,9 @@ BOOL WINAPI MiniDumpWriteDump(HANDLE hProcess, DWORD pid, HANDLE hFile, dc.mem = NULL; dc.num_mem = 0; dc.alloc_mem = 0; + dc.mem64 = NULL; + dc.num_mem64 = 0; + dc.alloc_mem64 = 0; dc.rva = 0; if (!fetch_process_info(&dc)) return FALSE; @@ -890,8 +990,6 @@ BOOL WINAPI MiniDumpWriteDump(HANDLE hProcess, DWORD pid, HANDLE hFile, if (DumpType & MiniDumpWithDataSegs) FIXME("NIY MiniDumpWithDataSegs\n"); - if (DumpType & MiniDumpWithFullMemory) - FIXME("NIY MiniDumpWithFullMemory\n"); if (DumpType & MiniDumpWithHandleData) FIXME("NIY MiniDumpWithHandleData\n"); if (DumpType & MiniDumpFilterMemory) @@ -940,11 +1038,15 @@ BOOL WINAPI MiniDumpWriteDump(HANDLE hProcess, DWORD pid, HANDLE hFile, writeat(&dc, mdHead.StreamDirectoryRva + idx_stream++ * sizeof(mdDir), &mdDir, sizeof(mdDir)); - mdDir.StreamType = MemoryListStream; - mdDir.Location.Rva = dc.rva; - mdDir.Location.DataSize = dump_memory_info(&dc); - writeat(&dc, mdHead.StreamDirectoryRva + idx_stream++ * sizeof(mdDir), - &mdDir, sizeof(mdDir)); + + if (!(DumpType & MiniDumpWithFullMemory)) + { + mdDir.StreamType = MemoryListStream; + mdDir.Location.Rva = dc.rva; + mdDir.Location.DataSize = dump_memory_info(&dc); + writeat(&dc, mdHead.StreamDirectoryRva + idx_stream++ * sizeof(mdDir), + &mdDir, sizeof(mdDir)); + } mdDir.StreamType = MiscInfoStream; mdDir.Location.Rva = dc.rva; @@ -977,12 +1079,25 @@ BOOL WINAPI MiniDumpWriteDump(HANDLE hProcess, DWORD pid, HANDLE hFile, } } + /* 3.4) write full memory (if requested) */ + if (DumpType & MiniDumpWithFullMemory) + { + fetch_memory64_info(&dc); + + mdDir.StreamType = Memory64ListStream; + mdDir.Location.Rva = dc.rva; + mdDir.Location.DataSize = dump_memory64_info(&dc); + writeat(&dc, mdHead.StreamDirectoryRva + idx_stream++ * sizeof(mdDir), + &mdDir, sizeof(mdDir)); + } + /* fill the remaining directory entries with 0's (unused stream types) */ /* NOTE: this should always come last in the dump! */ for (i = idx_stream; i < nStreams; i++) writeat(&dc, mdHead.StreamDirectoryRva + i * sizeof(emptyDir), &emptyDir, sizeof(emptyDir)); HeapFree(GetProcessHeap(), 0, dc.mem); + HeapFree(GetProcessHeap(), 0, dc.mem64); HeapFree(GetProcessHeap(), 0, dc.modules); HeapFree(GetProcessHeap(), 0, dc.threads); diff --git a/include/dbghelp.h b/include/dbghelp.h index 6e8d375..4210616 100644 --- a/include/dbghelp.h +++ b/include/dbghelp.h @@ -747,6 +747,19 @@ typedef struct _MINIDUMP_MEMORY_LIST MINIDUMP_MEMORY_DESCRIPTOR MemoryRanges[1]; /* FIXME: 0-sized array not supported */ } MINIDUMP_MEMORY_LIST, *PMINIDUMP_MEMORY_LIST; +typedef struct _MINIDUMP_MEMORY_DESCRIPTOR64 +{ + ULONG64 StartOfMemoryRange; + ULONG64 DataSize; +} MINIDUMP_MEMORY_DESCRIPTOR64, *PMINIDUMP_MEMORY_DESCRIPTOR64; + +typedef struct _MINIDUMP_MEMORY64_LIST +{ + ULONG64 NumberOfMemoryRanges; + RVA64 BaseRva; + MINIDUMP_MEMORY_DESCRIPTOR64 MemoryRanges[1]; /* FIXME: 0-sized array not supported */ +} MINIDUMP_MEMORY64_LIST, *PMINIDUMP_MEMORY64_LIST; + #define MINIDUMP_MISC1_PROCESS_ID 0x00000001 #define MINIDUMP_MISC1_PROCESS_TIMES 0x00000002

1 0

Jon Doron : kernel32: Fix MoveFileWithProgressW from closing same handle twice.
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: 69d3c7a00f37badf6a5661ab0c176963fe63dc42 URL: https://source.winehq.org/git/wine.git/?a=commit;h=69d3c7a00f37badf6a5661ab… Author: Jon Doron <arilou(a)gmail.com> Date: Fri Jan 11 10:24:44 2019 +0200 kernel32: Fix MoveFileWithProgressW from closing same handle twice. Signed-off-by: Jon Doron <arilou(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/path.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/kernel32/path.c b/dlls/kernel32/path.c index 41f0d34..554f96a 100644 --- a/dlls/kernel32/path.c +++ b/dlls/kernel32/path.c @@ -1392,6 +1392,7 @@ BOOL WINAPI MoveFileWithProgressW( LPCWSTR source, LPCWSTR dest, } NtClose( dest_handle ); + dest_handle = NULL; } else if (status != STATUS_OBJECT_NAME_NOT_FOUND) {

1 0

Jon Doron : kernel32: Fix MOVEFILE_REPLACE_EXISTING between devices.
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: aa5107b16542a2b236224cc4974731691d77a20c URL: https://source.winehq.org/git/wine.git/?a=commit;h=aa5107b16542a2b236224cc4… Author: Jon Doron <arilou(a)gmail.com> Date: Fri Jan 11 10:24:45 2019 +0200 kernel32: Fix MOVEFILE_REPLACE_EXISTING between devices. Fix the case of rename fails because when done between 2 different devices and the MOVEFILE_REPLACE_EXISTING is set. Signed-off-by: Jon Doron <arilou(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/path.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/path.c b/dlls/kernel32/path.c index 554f96a..cf1c768 100644 --- a/dlls/kernel32/path.c +++ b/dlls/kernel32/path.c @@ -1418,8 +1418,9 @@ BOOL WINAPI MoveFileWithProgressW( LPCWSTR source, LPCWSTR dest, NtClose( source_handle ); RtlFreeAnsiString( &source_unix ); RtlFreeAnsiString( &dest_unix ); - if (!CopyFileExW( source, dest, fnProgress, - param, NULL, COPY_FILE_FAIL_IF_EXISTS )) + if (!CopyFileExW( source, dest, fnProgress, param, NULL, + flag & MOVEFILE_REPLACE_EXISTING ? + 0 : COPY_FILE_FAIL_IF_EXISTS )) return FALSE; return DeleteFileW( source ); }

1 0

Sven Baars : dmime/tests: Fix a memory leak (Valgrind).
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: 8f3a474655387845d2381f39f7addf526c92c2c8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8f3a474655387845d2381f39… Author: Sven Baars <sven.wine(a)gmail.com> Date: Tue Jan 29 00:10:58 2019 +0100 dmime/tests: Fix a memory leak (Valgrind). Signed-off-by: Sven Baars <sven.wine(a)gmail.com> Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmime/tests/performance.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/dmime/tests/performance.c b/dlls/dmime/tests/performance.c index c4cbb32..567de1e 100644 --- a/dlls/dmime/tests/performance.c +++ b/dlls/dmime/tests/performance.c @@ -315,6 +315,9 @@ static void test_createport(void) ok(port == NULL, "Get IDirectMusicPort pointer? %p\n", port); ok(portparams.dwValidParams == 0, "portparams struct was filled in?\n"); + hr = IDirectMusicPerformance8_CloseDown(perf); + ok(hr == S_OK, "CloseDown failed: %08x\n", hr); + IDirectMusic_Release(music); IDirectMusicPerformance_Release(perf); }

1 0

Sven Baars : dinput/tests: Fix some memory leaks (Valgrind).
by Alexandre Julliard 29 Jan '19

29 Jan '19

Module: wine Branch: master Commit: 2543295a15f2e52236faf0b874cf8462c7fd256a URL: https://source.winehq.org/git/wine.git/?a=commit;h=2543295a15f2e52236faf0b8… Author: Sven Baars <sven.wine(a)gmail.com> Date: Tue Jan 29 00:10:59 2019 +0100 dinput/tests: Fix some memory leaks (Valgrind). Signed-off-by: Sven Baars <sven.wine(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dinput/tests/dinput.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/dlls/dinput/tests/dinput.c b/dlls/dinput/tests/dinput.c index cb72689..23b20d2 100644 --- a/dlls/dinput/tests/dinput.c +++ b/dlls/dinput/tests/dinput.c @@ -604,10 +604,14 @@ static void test_DirectInputJoyConfig8(void) "IDirectInputJoyConfig8_GetConfig returned 0x%08x\n", hr); if (SUCCEEDED(hr)) - ok (SUCCEEDED(IDirectInput_CreateDevice(pDI, &info.guidInstance, &pDID, NULL)), - "IDirectInput_CreateDevice failed with guid from GetConfig hr = 0x%08x\n", hr); + { + hr = IDirectInput_CreateDevice(pDI, &info.guidInstance, &pDID, NULL); + ok (SUCCEEDED(hr), "IDirectInput_CreateDevice failed with guid from GetConfig hr = 0x%08x\n", hr); + IDirectInputDevice_Release(pDID); + } } + IDirectInputJoyConfig8_Release(pDIJC); IDirectInput_Release(pDI); }

1 0