March 2022 - Wine-commits - list.winehq.org

Nikolay Sivov : comsvcs: Use CRT memory allocation functions.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 0d8ebada7fef1b40d95514384617ed158f963611 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0d8ebada7fef1b40d9551438… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Mar 31 19:57:56 2022 +0300 comsvcs: Use CRT memory allocation functions. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comsvcs/main.c | 24 +++++++++++------------- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/dlls/comsvcs/main.c b/dlls/comsvcs/main.c index b34c7d21309..43803f60632 100644 --- a/dlls/comsvcs/main.c +++ b/dlls/comsvcs/main.c @@ -19,7 +19,6 @@ */ #include "comsvcs_private.h" -#include "wine/heap.h" #include "wine/debug.h" #include "initguid.h" #include "comsvcs_classes.h" @@ -109,7 +108,7 @@ static ULONG WINAPI holder_Release(IHolder *iface) if (!ref) { - heap_free(This); + free(This); } return ref; @@ -221,7 +220,7 @@ static HRESULT create_holder(IDispenserDriver *driver, IHolder **object) TRACE("(%p)\n", object); - hold = heap_alloc(sizeof(*hold)); + hold = malloc(sizeof(*hold)); if (!hold) { *object = NULL; @@ -277,7 +276,7 @@ static ULONG WINAPI dismanager_Release(IDispenserManager *iface) { if (This->mta_cookie) CoDecrementMTAUsage(This->mta_cookie); - heap_free(This); + free(This); } return ref; @@ -330,7 +329,7 @@ static HRESULT WINAPI dispenser_manager_cf_CreateInstance(IClassFactory *iface, TRACE("(%p %s %p)\n", outer, debugstr_guid(riid), object); - dismanager = heap_alloc_zero(sizeof(*dismanager)); + dismanager = calloc(1, sizeof(*dismanager)); if (!dismanager) { *object = NULL; @@ -443,8 +442,8 @@ static ULONG WINAPI new_moniker_Release(IMoniker* iface) if (!refcount) { - heap_free(moniker->progid); - heap_free(moniker); + free(moniker->progid); + free(moniker); } return refcount; @@ -488,7 +487,7 @@ static HRESULT WINAPI new_moniker_Load(IMoniker *iface, IStream *stream) if (SUCCEEDED(hr) && progid_len) { - if (!(progid = heap_alloc(progid_len))) + if (!(progid = malloc(progid_len))) return E_OUTOFMEMORY; hr = IStream_Read(stream, progid, progid_len, &len); } @@ -500,12 +499,12 @@ static HRESULT WINAPI new_moniker_Load(IMoniker *iface, IStream *stream) if (SUCCEEDED(hr) && pad == 0) { moniker->clsid = clsid; - heap_free(moniker->progid); + free(moniker->progid); moniker->progid = progid; progid = NULL; } - heap_free(progid); + free(progid); return hr; } @@ -871,7 +870,7 @@ static HRESULT new_moniker_parse_displayname(IBindCtx *pbc, LPOLESTR name, ULONG progid = str; } - moniker = heap_alloc_zero(sizeof(*moniker)); + moniker = calloc(1, sizeof(*moniker)); if (!moniker) return E_OUTOFMEMORY; @@ -881,12 +880,11 @@ static HRESULT new_moniker_parse_displayname(IBindCtx *pbc, LPOLESTR name, ULONG moniker->clsid = guid; if (progid) { - if (!(moniker->progid = heap_alloc((lstrlenW(progid) + 1) * sizeof(WCHAR)))) + if (!(moniker->progid = wcsdup(progid))) { IMoniker_Release(&moniker->IMoniker_iface); return E_OUTOFMEMORY; } - lstrcpyW(moniker->progid, progid); } *ret = &moniker->IMoniker_iface;

1 0

Rémi Bernon : kernelbase: Check handle validity in unsafe_mem_from_HLOCAL.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 458f49dee1210335e8a99e5cef604aef45d931a6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=458f49dee1210335e8a99e5c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 31 12:06:37 2022 +0200 kernelbase: Check handle validity in unsafe_mem_from_HLOCAL. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/memory.c | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/dlls/kernelbase/memory.c b/dlls/kernelbase/memory.c index 3cb596e5814..655778fe79f 100644 --- a/dlls/kernelbase/memory.c +++ b/dlls/kernelbase/memory.c @@ -598,9 +598,12 @@ struct mem_entry * the output jpeg's > 1 MB if not */ #define HLOCAL_STORAGE (sizeof(HLOCAL) * 2) -static inline struct mem_entry *mem_from_HLOCAL( HLOCAL handle ) +static inline struct mem_entry *unsafe_mem_from_HLOCAL( HLOCAL handle ) { - return (struct mem_entry *)((char *)handle - 2); + struct mem_entry *mem = CONTAINING_RECORD( handle, struct mem_entry, ptr ); + if (!((ULONG_PTR)handle & 2)) return NULL; + if (mem->magic != MAGIC_LOCAL_USED) return NULL; + return mem; } static inline HLOCAL HLOCAL_from_mem( struct mem_entry *mem ) @@ -709,8 +712,7 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalFree( HLOCAL handle ) } else /* HANDLE */ { - mem = mem_from_HLOCAL( handle ); - if (mem->magic == MAGIC_LOCAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { mem->magic = 0xdead; if (mem->ptr) @@ -745,6 +747,7 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalFree( HLOCAL handle ) */ LPVOID WINAPI DECLSPEC_HOTPATCH LocalLock( HLOCAL handle ) { + struct mem_entry *mem; void *ret = NULL; TRACE_(globalmem)( "handle %p\n", handle ); @@ -767,8 +770,7 @@ LPVOID WINAPI DECLSPEC_HOTPATCH LocalLock( HLOCAL handle ) RtlLockHeap( GetProcessHeap() ); __TRY { - struct mem_entry *mem = mem_from_HLOCAL( handle ); - if (mem->magic == MAGIC_LOCAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { ret = mem->ptr; if (!mem->ptr) SetLastError( ERROR_DISCARDED ); @@ -826,10 +828,9 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT f LocalFree( handle ); } } - else if (!is_pointer( handle ) && (flags & LMEM_DISCARDABLE)) + else if ((mem = unsafe_mem_from_HLOCAL( handle )) && (flags & LMEM_DISCARDABLE)) { /* change the flags to make our block "discardable" */ - mem = mem_from_HLOCAL( handle ); mem->flags |= LMEM_DISCARDABLE >> 8; ret = handle; } @@ -843,10 +844,9 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT f if (!(flags & LMEM_MOVEABLE)) heap_flags |= HEAP_REALLOC_IN_PLACE_ONLY; ret = HeapReAlloc( GetProcessHeap(), heap_flags, handle, size ); } - else + else if ((mem = unsafe_mem_from_HLOCAL( handle ))) { /* reallocate a moveable block */ - mem = mem_from_HLOCAL( handle ); if (size != 0) { if (size <= INT_MAX - HLOCAL_STORAGE) @@ -886,6 +886,7 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT f else WARN_(globalmem)( "not freeing memory associated with locked handle\n" ); } } + else SetLastError( ERROR_INVALID_HANDLE ); } RtlUnlockHeap( GetProcessHeap() ); return ret; @@ -897,6 +898,7 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT f */ BOOL WINAPI DECLSPEC_HOTPATCH LocalUnlock( HLOCAL handle ) { + struct mem_entry *mem; BOOL ret = FALSE; TRACE_(globalmem)( "handle %p\n", handle ); @@ -910,8 +912,7 @@ BOOL WINAPI DECLSPEC_HOTPATCH LocalUnlock( HLOCAL handle ) RtlLockHeap( GetProcessHeap() ); __TRY { - struct mem_entry *mem = mem_from_HLOCAL( handle ); - if (mem->magic == MAGIC_LOCAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { if (mem->lock) {

1 0

Rémi Bernon : kernelbase: Check pointer validity in unsafe_ptr_from_HLOCAL.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 82bab8366da2053e62a2f8cb66e8b3a4c262e162 URL: https://source.winehq.org/git/wine.git/?a=commit;h=82bab8366da2053e62a2f8cb… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 31 12:06:38 2022 +0200 kernelbase: Check pointer validity in unsafe_ptr_from_HLOCAL. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/memory.c | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/dlls/kernelbase/memory.c b/dlls/kernelbase/memory.c index 655778fe79f..e81d38564e7 100644 --- a/dlls/kernelbase/memory.c +++ b/dlls/kernelbase/memory.c @@ -611,9 +611,10 @@ static inline HLOCAL HLOCAL_from_mem( struct mem_entry *mem ) return &mem->ptr; } -static inline BOOL is_pointer( HLOCAL handle ) +static inline void *unsafe_ptr_from_HLOCAL( HLOCAL handle ) { - return !((ULONG_PTR)handle & 2); + if ((ULONG_PTR)handle & 2) return NULL; + return handle; } /*********************************************************************** @@ -695,6 +696,7 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalFree( HLOCAL handle ) { struct mem_entry *mem; HLOCAL ret; + void *ptr; TRACE_(globalmem)( "handle %p\n", handle ); @@ -702,9 +704,9 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalFree( HLOCAL handle ) __TRY { ret = 0; - if (is_pointer( handle )) /* POINTER */ + if ((ptr = unsafe_ptr_from_HLOCAL( handle ))) { - if (!HeapFree( GetProcessHeap(), HEAP_NO_SERIALIZE, handle )) + if (!HeapFree( GetProcessHeap(), HEAP_NO_SERIALIZE, ptr )) { SetLastError( ERROR_INVALID_HANDLE ); ret = handle; @@ -752,11 +754,11 @@ LPVOID WINAPI DECLSPEC_HOTPATCH LocalLock( HLOCAL handle ) TRACE_(globalmem)( "handle %p\n", handle ); - if (is_pointer( handle )) + if ((ret = unsafe_ptr_from_HLOCAL( handle ))) { __TRY { - volatile char *p = handle; + volatile char *p = ret; *p |= 0; } __EXCEPT_PAGE_FAULT @@ -764,7 +766,7 @@ LPVOID WINAPI DECLSPEC_HOTPATCH LocalLock( HLOCAL handle ) return NULL; } __ENDTRY - return handle; + return ret; } RtlLockHeap( GetProcessHeap() ); @@ -799,16 +801,16 @@ LPVOID WINAPI DECLSPEC_HOTPATCH LocalLock( HLOCAL handle ) HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT flags ) { struct mem_entry *mem; - void *ptr; HLOCAL ret = 0; DWORD heap_flags = (flags & LMEM_ZEROINIT) ? HEAP_ZERO_MEMORY : 0; + void *ptr; TRACE_(globalmem)( "handle %p, size %#Ix, flags %#x\n", handle, size, flags ); RtlLockHeap( GetProcessHeap() ); if (flags & LMEM_MODIFY) /* modify flags */ { - if (is_pointer( handle ) && (flags & LMEM_MOVEABLE)) + if (unsafe_ptr_from_HLOCAL( handle ) && (flags & LMEM_MOVEABLE)) { /* make a fixed block moveable * actually only NT is able to do this. But it's soo simple @@ -838,11 +840,11 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalReAlloc( HLOCAL handle, SIZE_T size, UINT f } else { - if (is_pointer( handle )) + if ((ptr = unsafe_ptr_from_HLOCAL( handle ))) { /* reallocate fixed memory */ if (!(flags & LMEM_MOVEABLE)) heap_flags |= HEAP_REALLOC_IN_PLACE_ONLY; - ret = HeapReAlloc( GetProcessHeap(), heap_flags, handle, size ); + ret = HeapReAlloc( GetProcessHeap(), heap_flags, ptr, size ); } else if ((mem = unsafe_mem_from_HLOCAL( handle ))) { @@ -903,7 +905,7 @@ BOOL WINAPI DECLSPEC_HOTPATCH LocalUnlock( HLOCAL handle ) TRACE_(globalmem)( "handle %p\n", handle ); - if (is_pointer( handle )) + if (unsafe_ptr_from_HLOCAL( handle )) { SetLastError( ERROR_NOT_LOCKED ); return FALSE;

1 0

Rémi Bernon : kernel32: Synchronize the HLOCAL helpers with kernelbase.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 1ff9e76395e3acaa339f4ff4ecbff6ad20b0a82a URL: https://source.winehq.org/git/wine.git/?a=commit;h=1ff9e76395e3acaa339f4ff4… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 31 12:06:39 2022 +0200 kernel32: Synchronize the HLOCAL helpers with kernelbase. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/heap.c | 105 ++++++++++++++++++++++++++------------------------- 1 file changed, 53 insertions(+), 52 deletions(-) diff --git a/dlls/kernel32/heap.c b/dlls/kernel32/heap.c index fefb88d6857..4786131090d 100644 --- a/dlls/kernel32/heap.c +++ b/dlls/kernel32/heap.c @@ -140,42 +140,45 @@ BOOL WINAPI HeapDestroy( HANDLE heap /* [in] Handle of heap */ ) } -/* - * Win32 Global heap functions (GlobalXXX). - * These functions included in Win32 for compatibility with 16 bit Windows - * Especially the moveable blocks and handles are oldish. - * But the ability to directly allocate memory with GPTR and LPTR is widely - * used. - * - * The handle stuff looks horrible, but it's implemented almost like Win95 - * does it. - * - */ +/*********************************************************************** + * Global/local heap functions, keep in sync with kernelbase/memory.c + ***********************************************************************/ + +#include "pshpack1.h" + +struct mem_entry +{ + WORD magic; + void *ptr; + BYTE flags; + BYTE lock; +}; -#define MAGIC_GLOBAL_USED 0x5342 -#define HANDLE_TO_INTERN(h) ((PGLOBAL32_INTERN)(((char *)(h))-2)) -#define INTERN_TO_HANDLE(i) (&((i)->Pointer)) -#define POINTER_TO_HANDLE(p) (*(((const HGLOBAL *)(p))-2)) -#define ISHANDLE(h) (((ULONG_PTR)(h)&2)!=0) -#define ISPOINTER(h) (((ULONG_PTR)(h)&2)==0) -/* align the storage needed for the HGLOBAL on an 8byte boundary thus - * GlobalAlloc/GlobalReAlloc'ing with GMEM_MOVEABLE of memory with - * size = 8*k, where k=1,2,3,... alloc's exactly the given size. +#include "poppack.h" + +#define MAGIC_LOCAL_USED 0x5342 +#define POINTER_TO_HANDLE( p ) (*(((const HGLOBAL *)( p )) - 2)) +/* align the storage needed for the HLOCAL on an 8-byte boundary thus + * LocalAlloc/LocalReAlloc'ing with LMEM_MOVEABLE of memory with + * size = 8*k, where k=1,2,3,... allocs exactly the given size. * The Minolta DiMAGE Image Viewer heavily relies on this, corrupting * the output jpeg's > 1 MB if not */ -#define HGLOBAL_STORAGE (sizeof(HGLOBAL)*2) +#define HLOCAL_STORAGE (sizeof(HLOCAL) * 2) -#include "pshpack1.h" +static inline struct mem_entry *unsafe_mem_from_HLOCAL( HLOCAL handle ) +{ + struct mem_entry *mem = CONTAINING_RECORD( handle, struct mem_entry, ptr ); + if (!((ULONG_PTR)handle & 2)) return NULL; + if (mem->magic != MAGIC_LOCAL_USED) return NULL; + return mem; +} -typedef struct __GLOBAL32_INTERN +static inline void *unsafe_ptr_from_HLOCAL( HLOCAL handle ) { - WORD Magic; - LPVOID Pointer; - BYTE Flags; - BYTE LockCount; -} GLOBAL32_INTERN, *PGLOBAL32_INTERN; + if ((ULONG_PTR)handle & 2) return NULL; + return handle; +} -#include "poppack.h" /*********************************************************************** * GlobalLock (KERNEL32.@) @@ -217,7 +220,7 @@ void *WINAPI GlobalLock( HGLOBAL handle ) */ BOOL WINAPI GlobalUnlock( HGLOBAL handle ) { - if (ISPOINTER( handle )) return TRUE; + if (unsafe_ptr_from_HLOCAL( handle )) return TRUE; return LocalUnlock( handle ); } @@ -233,7 +236,7 @@ BOOL WINAPI GlobalUnlock( HGLOBAL handle ) */ HGLOBAL WINAPI GlobalHandle( const void *ptr ) { - PGLOBAL32_INTERN mem; + struct mem_entry *mem; HGLOBAL handle; LPCVOID test; @@ -253,7 +256,7 @@ HGLOBAL WINAPI GlobalHandle( const void *ptr ) /* note that if ptr is a pointer to a block allocated by */ /* GlobalAlloc with GMEM_MOVEABLE then magic test in HeapValidate */ /* will fail. */ - if (ISPOINTER( ptr )) + if ((ptr = unsafe_ptr_from_HLOCAL( (HLOCAL)ptr ))) { if (HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, ptr )) { @@ -265,11 +268,10 @@ HGLOBAL WINAPI GlobalHandle( const void *ptr ) else handle = (HGLOBAL)ptr; /* Now test handle either passed in or retrieved from pointer */ - mem = HANDLE_TO_INTERN( handle ); - if (mem->Magic == MAGIC_GLOBAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { - test = mem->Pointer; - if (HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, (const char *)test - HGLOBAL_STORAGE ) && /* obj(-handle) valid arena? */ + test = mem->ptr; + if (HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, (const char *)test - HLOCAL_STORAGE ) && /* obj(-handle) valid arena? */ HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, mem )) /* intern valid arena? */ break; /* valid moveable block */ } @@ -320,8 +322,9 @@ HGLOBAL WINAPI GlobalReAlloc( HGLOBAL handle, SIZE_T size, UINT flags ) */ SIZE_T WINAPI GlobalSize( HGLOBAL handle ) { - PGLOBAL32_INTERN mem; + struct mem_entry *mem; SIZE_T retval; + void *ptr; TRACE_(globalmem)( "handle %p\n", handle ); @@ -331,27 +334,26 @@ SIZE_T WINAPI GlobalSize( HGLOBAL handle ) return 0; } - if (ISPOINTER( handle )) + if ((ptr = unsafe_ptr_from_HLOCAL( handle ))) { - retval = HeapSize( GetProcessHeap(), 0, handle ); + retval = HeapSize( GetProcessHeap(), 0, ptr ); if (retval == ~(SIZE_T)0) /* It might be a GMEM_MOVEABLE data pointer */ { - retval = HeapSize( GetProcessHeap(), 0, (char *)handle - HGLOBAL_STORAGE ); - if (retval != ~(SIZE_T)0) retval -= HGLOBAL_STORAGE; + retval = HeapSize( GetProcessHeap(), 0, (char *)ptr - HLOCAL_STORAGE ); + if (retval != ~(SIZE_T)0) retval -= HLOCAL_STORAGE; } } else { RtlLockHeap( GetProcessHeap() ); - mem = HANDLE_TO_INTERN( handle ); - if (mem->Magic == MAGIC_GLOBAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { - if (!mem->Pointer) /* handle case of GlobalAlloc( ??,0) */ + if (!mem->ptr) /* handle case of GlobalAlloc( ??,0) */ retval = 0; else { - retval = HeapSize( GetProcessHeap(), 0, (char *)mem->Pointer - HGLOBAL_STORAGE ); - if (retval != ~(SIZE_T)0) retval -= HGLOBAL_STORAGE; + retval = HeapSize( GetProcessHeap(), 0, (char *)mem->ptr - HLOCAL_STORAGE ); + if (retval != ~(SIZE_T)0) retval -= HLOCAL_STORAGE; } } else @@ -418,23 +420,22 @@ VOID WINAPI GlobalUnfix( HGLOBAL handle ) */ UINT WINAPI GlobalFlags( HGLOBAL handle ) { - PGLOBAL32_INTERN mem; + struct mem_entry *mem; DWORD retval; TRACE_(globalmem)( "handle %p\n", handle ); - if (ISPOINTER( handle )) + if (unsafe_ptr_from_HLOCAL( handle )) { retval = 0; } else { RtlLockHeap( GetProcessHeap() ); - mem = HANDLE_TO_INTERN( handle ); - if (mem->Magic == MAGIC_GLOBAL_USED) + if ((mem = unsafe_mem_from_HLOCAL( handle ))) { - retval = mem->LockCount + (mem->Flags << 8); - if (mem->Pointer == 0) retval |= GMEM_DISCARDED; + retval = mem->lock + (mem->flags << 8); + if (mem->ptr == 0) retval |= GMEM_DISCARDED; } else {

1 0

Rémi Bernon : kernelbase: Implement internal KernelBaseGetGlobalData.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 981283a50b5a60c9b3e9d2b4487ba4c38930b9d2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=981283a50b5a60c9b3e9d2b4… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 31 12:06:40 2022 +0200 kernelbase: Implement internal KernelBaseGetGlobalData. And call it from kernel32 to get access to kernelbase global data. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/heap.c | 5 +++++ dlls/kernel32/kernel_main.c | 1 + dlls/kernel32/kernel_private.h | 11 +++++++++++ dlls/kernelbase/kernelbase.spec | 2 +- dlls/kernelbase/memory.c | 20 ++++++++++++++++++++ 5 files changed, 38 insertions(+), 1 deletion(-) diff --git a/dlls/kernel32/heap.c b/dlls/kernel32/heap.c index 4786131090d..86c16ef9b49 100644 --- a/dlls/kernel32/heap.c +++ b/dlls/kernel32/heap.c @@ -33,6 +33,9 @@ #include "winerror.h" #include "winnt.h" #include "winternl.h" + +#include "kernel_private.h" + #include "wine/exception.h" #include "wine/debug.h" @@ -156,6 +159,8 @@ struct mem_entry #include "poppack.h" +struct kernelbase_global_data *kernelbase_global_data; + #define MAGIC_LOCAL_USED 0x5342 #define POINTER_TO_HANDLE( p ) (*(((const HGLOBAL *)( p )) - 2)) /* align the storage needed for the HLOCAL on an 8-byte boundary thus diff --git a/dlls/kernel32/kernel_main.c b/dlls/kernel32/kernel_main.c index 352a873933f..7b45db73257 100644 --- a/dlls/kernel32/kernel_main.c +++ b/dlls/kernel32/kernel_main.c @@ -123,6 +123,7 @@ static BOOL process_attach( HMODULE module ) RtlSetUnhandledExceptionFilter( UnhandledExceptionFilter ); NtQuerySystemInformation( SystemBasicInformation, &system_info, sizeof(system_info), NULL ); + kernelbase_global_data = KernelBaseGetGlobalData(); copy_startup_info(); diff --git a/dlls/kernel32/kernel_private.h b/dlls/kernel32/kernel_private.h index 633511d6140..980a12b25a6 100644 --- a/dlls/kernel32/kernel_private.h +++ b/dlls/kernel32/kernel_private.h @@ -21,6 +21,17 @@ #ifndef __WINE_KERNEL_PRIVATE_H #define __WINE_KERNEL_PRIVATE_H +/* not compatible with windows */ +struct kernelbase_global_data +{ + struct mem_entry *mem_entries; + struct mem_entry *mem_entries_end; +}; + +void *WINAPI KernelBaseGetGlobalData(void); + +extern struct kernelbase_global_data *kernelbase_global_data; + NTSTATUS WINAPI BaseGetNamedObjectDirectory( HANDLE *dir ); static inline BOOL set_ntstatus( NTSTATUS status ) diff --git a/dlls/kernelbase/kernelbase.spec b/dlls/kernelbase/kernelbase.spec index a59d8581ff0..b5116ae9b5a 100644 --- a/dlls/kernelbase/kernelbase.spec +++ b/dlls/kernelbase/kernelbase.spec @@ -943,7 +943,7 @@ @ stdcall K32InitializeProcessForWsWatch(long) InitializeProcessForWsWatch @ stdcall K32QueryWorkingSet(long ptr long) QueryWorkingSet @ stdcall K32QueryWorkingSetEx(long ptr long) QueryWorkingSetEx -@ stub KernelBaseGetGlobalData +@ stdcall KernelBaseGetGlobalData() @ stdcall LCIDToLocaleName(long ptr long long) @ stdcall LCMapStringA(long long str long ptr long) @ stdcall LCMapStringEx(wstr long wstr long ptr long ptr ptr long) diff --git a/dlls/kernelbase/memory.c b/dlls/kernelbase/memory.c index e81d38564e7..6d42316760b 100644 --- a/dlls/kernelbase/memory.c +++ b/dlls/kernelbase/memory.c @@ -578,6 +578,13 @@ BOOL WINAPI DECLSPEC_HOTPATCH HeapWalk( HANDLE heap, PROCESS_HEAP_ENTRY *entry ) * Global/local heap functions ***********************************************************************/ +/* not compatible with windows */ +struct kernelbase_global_data +{ + struct mem_entry *mem_entries; + struct mem_entry *mem_entries_end; +}; + #include "pshpack1.h" struct mem_entry @@ -590,6 +597,8 @@ struct mem_entry #include "poppack.h" +static struct kernelbase_global_data kernelbase_global_data = {0}; + #define MAGIC_LOCAL_USED 0x5342 /* align the storage needed for the HLOCAL on an 8-byte boundary thus * LocalAlloc/LocalReAlloc'ing with LMEM_MOVEABLE of memory with @@ -617,6 +626,17 @@ static inline void *unsafe_ptr_from_HLOCAL( HLOCAL handle ) return handle; } + +/*********************************************************************** + * KernelBaseGetGlobalData (kernelbase.@) + */ +void *WINAPI KernelBaseGetGlobalData(void) +{ + WARN_(globalmem)( "semi-stub!\n" ); + return &kernelbase_global_data; +} + + /*********************************************************************** * GlobalAlloc (kernelbase.@) */

1 0

Rémi Bernon : kernelbase: Allocate HLOCAL / HGLOBAL from a static handle table.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 3c9d2cbaea2039d8ce5d8180ed231b0be6cfa072 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3c9d2cbaea2039d8ce5d8180… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Mar 31 12:06:41 2022 +0200 kernelbase: Allocate HLOCAL / HGLOBAL from a static handle table. Sharing the table pointers through KernelBaseGetGlobalData to check for handle validity in kernel32, and as native does it. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/heap.c | 20 +++++--- dlls/kernel32/tests/heap.c | 9 ---- dlls/kernelbase/memory.c | 115 ++++++++++++++++++++++++--------------------- 3 files changed, 75 insertions(+), 69 deletions(-) diff --git a/dlls/kernel32/heap.c b/dlls/kernel32/heap.c index 86c16ef9b49..dbb240d89fc 100644 --- a/dlls/kernel32/heap.c +++ b/dlls/kernel32/heap.c @@ -151,10 +151,17 @@ BOOL WINAPI HeapDestroy( HANDLE heap /* [in] Handle of heap */ ) struct mem_entry { - WORD magic; - void *ptr; - BYTE flags; - BYTE lock; + union + { + struct + { + WORD magic; + void *ptr; + BYTE flags; + BYTE lock; + }; + void *next_free; + }; }; #include "poppack.h" @@ -173,7 +180,9 @@ struct kernelbase_global_data *kernelbase_global_data; static inline struct mem_entry *unsafe_mem_from_HLOCAL( HLOCAL handle ) { struct mem_entry *mem = CONTAINING_RECORD( handle, struct mem_entry, ptr ); + struct kernelbase_global_data *data = kernelbase_global_data; if (!((ULONG_PTR)handle & 2)) return NULL; + if (mem < data->mem_entries || mem >= data->mem_entries_end) return NULL; if (mem->magic != MAGIC_LOCAL_USED) return NULL; return mem; } @@ -276,8 +285,7 @@ HGLOBAL WINAPI GlobalHandle( const void *ptr ) if ((mem = unsafe_mem_from_HLOCAL( handle ))) { test = mem->ptr; - if (HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, (const char *)test - HLOCAL_STORAGE ) && /* obj(-handle) valid arena? */ - HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, mem )) /* intern valid arena? */ + if (HeapValidate( GetProcessHeap(), HEAP_NO_SERIALIZE, (const char *)test - HLOCAL_STORAGE )) /* obj(-handle) valid arena? */ break; /* valid moveable block */ } handle = 0; diff --git a/dlls/kernel32/tests/heap.c b/dlls/kernel32/tests/heap.c index f6fb95cdddd..476be96cea8 100644 --- a/dlls/kernel32/tests/heap.c +++ b/dlls/kernel32/tests/heap.c @@ -290,15 +290,11 @@ static void test_GlobalAlloc(void) SetLastError( 0xdeadbeef ); mem = GlobalAlloc( GMEM_MOVEABLE | GMEM_DISCARDABLE, 0 ); - todo_wine ok( !mem, "GlobalAlloc succeeded\n" ); - todo_wine ok( GetLastError() == ERROR_NOT_ENOUGH_MEMORY, "got error %lu\n", GetLastError() ); SetLastError( 0xdeadbeef ); mem = LocalAlloc( LMEM_MOVEABLE | LMEM_DISCARDABLE, 0 ); - todo_wine ok( !mem, "LocalAlloc succeeded\n" ); - todo_wine ok( GetLastError() == ERROR_NOT_ENOUGH_MEMORY, "got error %lu\n", GetLastError() ); mem = GlobalAlloc( GMEM_DISCARDABLE, 0 ); @@ -417,7 +413,6 @@ static void test_GlobalAlloc(void) tmp_mem = GlobalFree( mem ); ok( !tmp_mem, "GlobalFree failed, error %lu\n", GetLastError() ); ok( !!entry->flags, "got unexpected flags %#Ix\n", entry->flags ); - todo_wine_if(sizeof(void *) == 4) ok( !((UINT_PTR)entry->flags & sizeof(void *)), "got unexpected ptr align\n" ); todo_wine_if(sizeof(void *) == 4) ok( !((UINT_PTR)entry->flags & (sizeof(void *) - 1)), "got unexpected ptr align\n" ); @@ -787,15 +782,11 @@ static void test_LocalAlloc(void) SetLastError( 0xdeadbeef ); mem = LocalAlloc( LMEM_MOVEABLE | LMEM_DISCARDABLE, 0 ); - todo_wine ok( !mem, "LocalAlloc succeeded\n" ); - todo_wine ok( GetLastError() == ERROR_NOT_ENOUGH_MEMORY, "got error %lu\n", GetLastError() ); SetLastError( 0xdeadbeef ); mem = GlobalAlloc( GMEM_MOVEABLE | GMEM_DISCARDABLE, 0 ); - todo_wine ok( !mem, "GlobalAlloc succeeded\n" ); - todo_wine ok( GetLastError() == ERROR_NOT_ENOUGH_MEMORY, "got error %lu\n", GetLastError() ); mem = LocalAlloc( LMEM_DISCARDABLE, 0 ); diff --git a/dlls/kernelbase/memory.c b/dlls/kernelbase/memory.c index 6d42316760b..5bdab31589d 100644 --- a/dlls/kernelbase/memory.c +++ b/dlls/kernelbase/memory.c @@ -589,15 +589,30 @@ struct kernelbase_global_data struct mem_entry { - WORD magic; - void *ptr; - BYTE flags; - BYTE lock; + union + { + struct + { + WORD magic; + void *ptr; + BYTE flags; + BYTE lock; + }; + void *next_free; + }; }; #include "poppack.h" -static struct kernelbase_global_data kernelbase_global_data = {0}; +#define MAX_MEM_HANDLES 0x10000 +static struct mem_entry mem_entries[MAX_MEM_HANDLES]; +static struct mem_entry *next_free_mem = mem_entries; + +static struct kernelbase_global_data kernelbase_global_data = +{ + .mem_entries = mem_entries, + .mem_entries_end = mem_entries + MAX_MEM_HANDLES, +}; #define MAGIC_LOCAL_USED 0x5342 /* align the storage needed for the HLOCAL on an 8-byte boundary thus @@ -610,7 +625,9 @@ static struct kernelbase_global_data kernelbase_global_data = {0}; static inline struct mem_entry *unsafe_mem_from_HLOCAL( HLOCAL handle ) { struct mem_entry *mem = CONTAINING_RECORD( handle, struct mem_entry, ptr ); + struct kernelbase_global_data *data = &kernelbase_global_data; if (!((ULONG_PTR)handle & 2)) return NULL; + if (mem < data->mem_entries || mem >= data->mem_entries_end) return NULL; if (mem->magic != MAGIC_LOCAL_USED) return NULL; return mem; } @@ -666,8 +683,10 @@ HGLOBAL WINAPI DECLSPEC_HOTPATCH GlobalFree( HLOCAL handle ) */ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalAlloc( UINT flags, SIZE_T size ) { + HANDLE heap = GetProcessHeap(); struct mem_entry *mem; DWORD heap_flags = 0; + HLOCAL handle; void *ptr; TRACE_(globalmem)( "flags %#x, size %#Ix\n", flags, size ); @@ -676,36 +695,44 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalAlloc( UINT flags, SIZE_T size ) if (!(flags & LMEM_MOVEABLE)) /* pointer */ { - ptr = HeapAlloc( GetProcessHeap(), heap_flags, size ); + ptr = HeapAlloc( heap, heap_flags, size ); TRACE_(globalmem)( "return %p\n", ptr ); return ptr; } - if (size > INT_MAX - HLOCAL_STORAGE) + RtlLockHeap( heap ); + if ((mem = next_free_mem) < mem_entries || mem >= mem_entries + MAX_MEM_HANDLES) + mem = NULL; + else { - SetLastError( ERROR_OUTOFMEMORY ); - return 0; + if (!mem->next_free) next_free_mem++; + else next_free_mem = mem->next_free; + mem->next_free = NULL; } - if (!(mem = HeapAlloc( GetProcessHeap(), 0, sizeof(*mem) ))) return 0; + RtlUnlockHeap( heap ); + + if (!mem) goto failed; + handle = HLOCAL_from_mem( mem ); mem->magic = MAGIC_LOCAL_USED; mem->flags = flags >> 8; mem->lock = 0; + mem->ptr = NULL; if (size) { - if (!(ptr = HeapAlloc(GetProcessHeap(), heap_flags, size + HLOCAL_STORAGE ))) - { - HeapFree( GetProcessHeap(), 0, mem ); - return 0; - } - *(HLOCAL *)ptr = HLOCAL_from_mem( mem ); + if (!(ptr = HeapAlloc( heap, heap_flags, size + HLOCAL_STORAGE ))) goto failed; + *(HLOCAL *)ptr = handle; mem->ptr = (char *)ptr + HLOCAL_STORAGE; } - else mem->ptr = NULL; - TRACE_(globalmem)( "return handle %p, ptr %p\n", HLOCAL_from_mem( mem ), mem->ptr ); - return HLOCAL_from_mem( mem ); + TRACE_(globalmem)( "return handle %p, ptr %p\n", handle, mem->ptr ); + return handle; + +failed: + if (mem) LocalFree( handle ); + SetLastError( ERROR_NOT_ENOUGH_MEMORY ); + return 0; } @@ -714,52 +741,32 @@ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalAlloc( UINT flags, SIZE_T size ) */ HLOCAL WINAPI DECLSPEC_HOTPATCH LocalFree( HLOCAL handle ) { + HANDLE heap = GetProcessHeap(); struct mem_entry *mem; - HLOCAL ret; + HLOCAL ret = handle; void *ptr; TRACE_(globalmem)( "handle %p\n", handle ); - RtlLockHeap( GetProcessHeap() ); - __TRY + RtlLockHeap( heap ); + if ((ptr = unsafe_ptr_from_HLOCAL( handle ))) { - ret = 0; - if ((ptr = unsafe_ptr_from_HLOCAL( handle ))) - { - if (!HeapFree( GetProcessHeap(), HEAP_NO_SERIALIZE, ptr )) - { - SetLastError( ERROR_INVALID_HANDLE ); - ret = handle; - } - } - else /* HANDLE */ - { - if ((mem = unsafe_mem_from_HLOCAL( handle ))) - { - mem->magic = 0xdead; - if (mem->ptr) - { - if (!HeapFree( GetProcessHeap(), HEAP_NO_SERIALIZE, (char *)mem->ptr - HLOCAL_STORAGE )) - ret = handle; - } - if (!HeapFree( GetProcessHeap(), HEAP_NO_SERIALIZE, mem )) ret = handle; - } - else - { - WARN_(globalmem)( "invalid handle %p\n", handle ); - SetLastError( ERROR_INVALID_HANDLE ); - ret = handle; - } - } + if (HeapFree( heap, HEAP_NO_SERIALIZE, ptr )) ret = 0; } - __EXCEPT_PAGE_FAULT + else if ((mem = unsafe_mem_from_HLOCAL( handle ))) + { + if (!mem->ptr || HeapFree( heap, HEAP_NO_SERIALIZE, (char *)mem->ptr - HLOCAL_STORAGE )) ret = 0; + mem->ptr = NULL; + mem->next_free = next_free_mem; + next_free_mem = mem; + } + RtlUnlockHeap( heap ); + + if (ret) { WARN_(globalmem)( "invalid handle %p\n", handle ); SetLastError( ERROR_INVALID_HANDLE ); - ret = handle; } - __ENDTRY - RtlUnlockHeap( GetProcessHeap() ); return ret; }

1 0

Alexandre Julliard : kernel32: Move the error message table to kernelbase.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 689e63485cb0a46d21c0e44b56d8fbac6a3e9df2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=689e63485cb0a46d21c0e44b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 31 07:55:42 2022 +0200 kernel32: Move the error message table to kernelbase. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/Makefile.in | 3 - dlls/kernelbase/Makefile.in | 1 + dlls/kernelbase/locale.c | 5 +- dlls/{kernel32 => kernelbase}/winerror.mc | 0 po/ar.po | 1590 ++++++++++++++--------------- po/ast.po | 1584 ++++++++++++++-------------- po/bg.po | 1578 ++++++++++++++-------------- po/ca.po | 1590 ++++++++++++++--------------- po/cs.po | 1578 ++++++++++++++-------------- po/da.po | 1590 ++++++++++++++--------------- po/de.po | 1578 ++++++++++++++-------------- po/el.po | 1580 ++++++++++++++-------------- po/en.po | 1578 ++++++++++++++-------------- po/en_US.po | 1578 ++++++++++++++-------------- po/eo.po | 1586 ++++++++++++++-------------- po/es.po | 1590 ++++++++++++++--------------- po/fa.po | 1580 ++++++++++++++-------------- po/fi.po | 1590 ++++++++++++++--------------- po/fr.po | 1578 ++++++++++++++-------------- po/he.po | 1578 ++++++++++++++-------------- po/hi.po | 1578 ++++++++++++++-------------- po/hr.po | 1590 ++++++++++++++--------------- po/hu.po | 1590 ++++++++++++++--------------- po/it.po | 1578 ++++++++++++++-------------- po/ja.po | 1590 ++++++++++++++--------------- po/ko.po | 1590 ++++++++++++++--------------- po/lt.po | 1578 ++++++++++++++-------------- po/ml.po | 1578 ++++++++++++++-------------- po/nb_NO.po | 1578 ++++++++++++++-------------- po/nl.po | 1578 ++++++++++++++-------------- po/or.po | 1578 ++++++++++++++-------------- po/pa.po | 1578 ++++++++++++++-------------- po/pl.po | 1578 ++++++++++++++-------------- po/pt_BR.po | 1578 ++++++++++++++-------------- po/pt_PT.po | 1578 ++++++++++++++-------------- po/rm.po | 1578 ++++++++++++++-------------- po/ro.po | 1578 ++++++++++++++-------------- po/ru.po | 1578 ++++++++++++++-------------- po/si.po | 1586 ++++++++++++++-------------- po/sk.po | 1578 ++++++++++++++-------------- po/sl.po | 1590 ++++++++++++++--------------- po/sr_RS(a)cyrillic.po | 1582 ++++++++++++++-------------- po/sr_RS(a)latin.po | 1582 ++++++++++++++-------------- po/sv.po | 1590 ++++++++++++++--------------- po/ta.po | 1578 ++++++++++++++-------------- po/te.po | 1578 ++++++++++++++-------------- po/th.po | 1580 ++++++++++++++-------------- po/tr.po | 1590 ++++++++++++++--------------- po/uk.po | 1578 ++++++++++++++-------------- po/wa.po | 1580 ++++++++++++++-------------- po/wine.pot | 1578 ++++++++++++++-------------- po/zh_CN.po | 1590 ++++++++++++++--------------- po/zh_TW.po | 1590 ++++++++++++++--------------- 53 files changed, 38766 insertions(+), 38771 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=689e63485cb0a46d21c0…

1 0

Alexandre Julliard : msvcrt: Use EnumSystemLocalesEx instead of directly accessing kernel32 resources.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: f14adf592838a52d1e47b2ff123ba83f5787265b URL: https://source.winehq.org/git/wine.git/?a=commit;h=f14adf592838a52d1e47b2ff… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 31 10:56:18 2022 +0200 msvcrt: Use EnumSystemLocalesEx instead of directly accessing kernel32 resources. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/locale.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/dlls/msvcrt/locale.c b/dlls/msvcrt/locale.c index 037660d0a91..2421830e7b5 100644 --- a/dlls/msvcrt/locale.c +++ b/dlls/msvcrt/locale.c @@ -255,22 +255,19 @@ static int compare_info(LCID lcid, DWORD flags, char* buff, const char* cmp, BOO } static BOOL CALLBACK -find_best_locale_proc(HMODULE hModule, LPCSTR type, LPCSTR name, WORD LangID, LONG_PTR lParam) +find_best_locale_proc( WCHAR *name, DWORD locale_flags, LPARAM lParam ) { locale_search_t *res = (locale_search_t *)lParam; - const LCID lcid = MAKELCID(LangID, SORT_DEFAULT); + const LCID lcid = LocaleNameToLCID( name, 0 ); char buff[MAX_ELEM_LEN]; unsigned int flags = 0; - if(PRIMARYLANGID(LangID) == LANG_NEUTRAL) - return CONTINUE_LOOKING; - #if _MSVCR_VER >= 110 if (res->allow_sname && compare_info(lcid,LOCALE_SNAME,buff,res->search_language, TRUE)) { TRACE(":Found locale: %s->%s\n", res->search_language, buff); res->match_flags = FOUND_SNAME; - res->found_lang_id = LangID; + res->found_lang_id = LANGIDFROMLCID(lcid); return STOP_LOOKING; } #endif @@ -305,7 +302,7 @@ find_best_locale_proc(HMODULE hModule, LPCSTR type, LPCSTR name, WORD LangID, LO { /* Found a better match than previously */ res->match_flags = flags; - res->found_lang_id = LangID; + res->found_lang_id = LANGIDFROMLCID(lcid); } if ((flags & (FOUND_LANGUAGE | FOUND_COUNTRY)) == (FOUND_LANGUAGE | FOUND_COUNTRY)) @@ -363,9 +360,7 @@ LCID locale_to_LCID(const char *locale, unsigned short *codepage, BOOL *sname) search.allow_sname = TRUE; } - EnumResourceLanguagesA(GetModuleHandleA("KERNEL32"), (LPSTR)RT_STRING, - (LPCSTR)LOCALE_ILANGUAGE,find_best_locale_proc, - (LONG_PTR)&search); + EnumSystemLocalesEx( find_best_locale_proc, 0, (LPARAM)&search, NULL); if (!search.match_flags) return -1;

1 0

Alexandre Julliard : kernel32: Remove the old locale information stored in resources.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 2366d19b458f534388b0f1f58afbd58f5aa021ac URL: https://source.winehq.org/git/wine.git/?a=commit;h=2366d19b458f534388b0f1f5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 31 11:01:16 2022 +0200 kernel32: Remove the old locale information stored in resources. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/Makefile.in | 1 - dlls/kernel32/locale_rc.rc | 260 --------------------------------------------- dlls/kernel32/nls/afk.nls | 159 --------------------------- dlls/kernel32/nls/ara.nls | 159 --------------------------- dlls/kernel32/nls/arb.nls | 159 --------------------------- dlls/kernel32/nls/are.nls | 159 --------------------------- dlls/kernel32/nls/arg.nls | 159 --------------------------- dlls/kernel32/nls/arh.nls | 159 --------------------------- dlls/kernel32/nls/ari.nls | 159 --------------------------- dlls/kernel32/nls/arj.nls | 159 --------------------------- dlls/kernel32/nls/ark.nls | 159 --------------------------- dlls/kernel32/nls/arl.nls | 159 --------------------------- dlls/kernel32/nls/arm.nls | 159 --------------------------- dlls/kernel32/nls/aro.nls | 159 --------------------------- dlls/kernel32/nls/arq.nls | 159 --------------------------- dlls/kernel32/nls/ars.nls | 159 --------------------------- dlls/kernel32/nls/art.nls | 159 --------------------------- dlls/kernel32/nls/aru.nls | 159 --------------------------- dlls/kernel32/nls/ary.nls | 159 --------------------------- dlls/kernel32/nls/ast.nls | 159 --------------------------- dlls/kernel32/nls/aze.nls | 159 --------------------------- dlls/kernel32/nls/azl.nls | 159 --------------------------- dlls/kernel32/nls/bel.nls | 159 --------------------------- dlls/kernel32/nls/ben.nls | 153 -------------------------- dlls/kernel32/nls/bgr.nls | 159 --------------------------- dlls/kernel32/nls/brf.nls | 160 ---------------------------- dlls/kernel32/nls/cat.nls | 159 --------------------------- dlls/kernel32/nls/chs.nls | 159 --------------------------- dlls/kernel32/nls/cht.nls | 159 --------------------------- dlls/kernel32/nls/cor.nls | 160 ---------------------------- dlls/kernel32/nls/csy.nls | 159 --------------------------- dlls/kernel32/nls/cym.nls | 160 ---------------------------- dlls/kernel32/nls/dan.nls | 159 --------------------------- dlls/kernel32/nls/dea.nls | 159 --------------------------- dlls/kernel32/nls/dec.nls | 159 --------------------------- dlls/kernel32/nls/del.nls | 159 --------------------------- dlls/kernel32/nls/des.nls | 159 --------------------------- dlls/kernel32/nls/deu.nls | 159 --------------------------- dlls/kernel32/nls/div.nls | 159 --------------------------- dlls/kernel32/nls/ell.nls | 159 --------------------------- dlls/kernel32/nls/ena.nls | 159 --------------------------- dlls/kernel32/nls/enb.nls | 159 --------------------------- dlls/kernel32/nls/enc.nls | 159 --------------------------- dlls/kernel32/nls/eng.nls | 159 --------------------------- dlls/kernel32/nls/eni.nls | 159 --------------------------- dlls/kernel32/nls/enj.nls | 159 --------------------------- dlls/kernel32/nls/enl.nls | 159 --------------------------- dlls/kernel32/nls/enp.nls | 159 --------------------------- dlls/kernel32/nls/ens.nls | 159 --------------------------- dlls/kernel32/nls/ent.nls | 159 --------------------------- dlls/kernel32/nls/enu.nls | 159 --------------------------- dlls/kernel32/nls/enw.nls | 159 --------------------------- dlls/kernel32/nls/enz.nls | 159 --------------------------- dlls/kernel32/nls/eox.nls | 175 ------------------------------ dlls/kernel32/nls/esa.nls | 159 --------------------------- dlls/kernel32/nls/esb.nls | 159 --------------------------- dlls/kernel32/nls/esc.nls | 159 --------------------------- dlls/kernel32/nls/esd.nls | 159 --------------------------- dlls/kernel32/nls/ese.nls | 159 --------------------------- dlls/kernel32/nls/esf.nls | 159 --------------------------- dlls/kernel32/nls/esg.nls | 159 --------------------------- dlls/kernel32/nls/esh.nls | 159 --------------------------- dlls/kernel32/nls/esi.nls | 159 --------------------------- dlls/kernel32/nls/esl.nls | 159 --------------------------- dlls/kernel32/nls/esm.nls | 159 --------------------------- dlls/kernel32/nls/esn.nls | 159 --------------------------- dlls/kernel32/nls/eso.nls | 159 --------------------------- dlls/kernel32/nls/esp.nls | 159 --------------------------- dlls/kernel32/nls/esr.nls | 159 --------------------------- dlls/kernel32/nls/ess.nls | 159 --------------------------- dlls/kernel32/nls/esu.nls | 159 --------------------------- dlls/kernel32/nls/esv.nls | 159 --------------------------- dlls/kernel32/nls/esy.nls | 159 --------------------------- dlls/kernel32/nls/esz.nls | 159 --------------------------- dlls/kernel32/nls/eti.nls | 159 --------------------------- dlls/kernel32/nls/euq.nls | 159 --------------------------- dlls/kernel32/nls/far.nls | 159 --------------------------- dlls/kernel32/nls/fin.nls | 159 --------------------------- dlls/kernel32/nls/fos.nls | 159 --------------------------- dlls/kernel32/nls/fra.nls | 159 --------------------------- dlls/kernel32/nls/frb.nls | 159 --------------------------- dlls/kernel32/nls/frc.nls | 159 --------------------------- dlls/kernel32/nls/frl.nls | 159 --------------------------- dlls/kernel32/nls/frm.nls | 159 --------------------------- dlls/kernel32/nls/frs.nls | 159 --------------------------- dlls/kernel32/nls/gae.nls | 160 ---------------------------- dlls/kernel32/nls/gdh.nls | 160 ---------------------------- dlls/kernel32/nls/gdv.nls | 160 ---------------------------- dlls/kernel32/nls/glc.nls | 159 --------------------------- dlls/kernel32/nls/guj.nls | 159 --------------------------- dlls/kernel32/nls/heb.nls | 160 ---------------------------- dlls/kernel32/nls/hin.nls | 159 --------------------------- dlls/kernel32/nls/hrv.nls | 159 --------------------------- dlls/kernel32/nls/hun.nls | 159 --------------------------- dlls/kernel32/nls/hye.nls | 159 --------------------------- dlls/kernel32/nls/ind.nls | 159 --------------------------- dlls/kernel32/nls/isl.nls | 159 --------------------------- dlls/kernel32/nls/ita.nls | 159 --------------------------- dlls/kernel32/nls/its.nls | 159 --------------------------- dlls/kernel32/nls/ivl.nls | 159 --------------------------- dlls/kernel32/nls/jpn.nls | 159 --------------------------- dlls/kernel32/nls/kan.nls | 159 --------------------------- dlls/kernel32/nls/kat.nls | 159 --------------------------- dlls/kernel32/nls/kkz.nls | 159 --------------------------- dlls/kernel32/nls/knk.nls | 159 --------------------------- dlls/kernel32/nls/kor.nls | 159 --------------------------- dlls/kernel32/nls/kyr.nls | 159 --------------------------- dlls/kernel32/nls/lth.nls | 159 --------------------------- dlls/kernel32/nls/lvi.nls | 159 --------------------------- dlls/kernel32/nls/mar.nls | 159 --------------------------- dlls/kernel32/nls/mki.nls | 159 --------------------------- dlls/kernel32/nls/mon.nls | 159 --------------------------- dlls/kernel32/nls/msb.nls | 159 --------------------------- dlls/kernel32/nls/msl.nls | 159 --------------------------- dlls/kernel32/nls/nep.nls | 153 -------------------------- dlls/kernel32/nls/nlb.nls | 159 --------------------------- dlls/kernel32/nls/nld.nls | 159 --------------------------- dlls/kernel32/nls/nls.nls | 165 ---------------------------- dlls/kernel32/nls/non.nls | 159 --------------------------- dlls/kernel32/nls/nor.nls | 160 ---------------------------- dlls/kernel32/nls/pan.nls | 159 --------------------------- dlls/kernel32/nls/plk.nls | 159 --------------------------- dlls/kernel32/nls/ptb.nls | 159 --------------------------- dlls/kernel32/nls/ptg.nls | 159 --------------------------- dlls/kernel32/nls/rmc.nls | 170 ----------------------------- dlls/kernel32/nls/rom.nls | 159 --------------------------- dlls/kernel32/nls/rus.nls | 159 --------------------------- dlls/kernel32/nls/san.nls | 159 --------------------------- dlls/kernel32/nls/sin.nls | 159 --------------------------- dlls/kernel32/nls/sky.nls | 159 --------------------------- dlls/kernel32/nls/slv.nls | 159 --------------------------- dlls/kernel32/nls/sqi.nls | 159 --------------------------- dlls/kernel32/nls/srb.nls | 159 --------------------------- dlls/kernel32/nls/srl.nls | 159 --------------------------- dlls/kernel32/nls/srm.nls | 156 --------------------------- dlls/kernel32/nls/sve.nls | 159 --------------------------- dlls/kernel32/nls/svf.nls | 159 --------------------------- dlls/kernel32/nls/swk.nls | 159 --------------------------- dlls/kernel32/nls/syr.nls | 159 --------------------------- dlls/kernel32/nls/tam.nls | 159 --------------------------- dlls/kernel32/nls/tel.nls | 159 --------------------------- dlls/kernel32/nls/tha.nls | 159 --------------------------- dlls/kernel32/nls/trk.nls | 159 --------------------------- dlls/kernel32/nls/ttt.nls | 159 --------------------------- dlls/kernel32/nls/ukr.nls | 159 --------------------------- dlls/kernel32/nls/urd.nls | 159 --------------------------- dlls/kernel32/nls/uzb.nls | 159 --------------------------- dlls/kernel32/nls/uzl.nls | 159 --------------------------- dlls/kernel32/nls/vit.nls | 159 --------------------------- dlls/kernel32/nls/wal.nls | 159 --------------------------- dlls/kernel32/nls/zhh.nls | 159 --------------------------- dlls/kernel32/nls/zhi.nls | 159 --------------------------- dlls/kernel32/nls/zhm.nls | 159 --------------------------- 153 files changed, 24296 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=2366d19b458f534388b0…

1 0

Hans Leidekker : crypt32/tests: Use wide character string literals.
by Alexandre Julliard 31 Mar '22

31 Mar '22

Module: wine Branch: master Commit: 33ff4a6671db8d515ef6f1a205c5fb46569ee978 URL: https://source.winehq.org/git/wine.git/?a=commit;h=33ff4a6671db8d515ef6f1a2… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Mar 31 11:06:03 2022 +0200 crypt32/tests: Use wide character string literals. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/crypt32/tests/cert.c | 52 ++++---- dlls/crypt32/tests/chain.c | 59 +++------- dlls/crypt32/tests/encode.c | 88 ++++++-------- dlls/crypt32/tests/main.c | 16 +-- dlls/crypt32/tests/object.c | 54 +++------ dlls/crypt32/tests/oid.c | 65 +++++----- dlls/crypt32/tests/protectdata.c | 13 +- dlls/crypt32/tests/sip.c | 35 +++--- dlls/crypt32/tests/store.c | 222 ++++++++++++---------------------- dlls/crypt32/tests/str.c | 249 ++++++++++++--------------------------- 10 files changed, 294 insertions(+), 559 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=33ff4a6671db8d515ef6…

1 0