December 2021 - Wine-commits

Alexandre Julliard : ntdll/tests: Mark a failing test as todo.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: b75caec103ae5dd827674872afdbcf1537f95144 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b75caec103ae5dd827674872… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 16:15:13 2021 +0100 ntdll/tests: Mark a failing test as todo. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/tests/wow64.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/tests/wow64.c b/dlls/ntdll/tests/wow64.c index 9e09c60c32c..efe51dd507b 100644 --- a/dlls/ntdll/tests/wow64.c +++ b/dlls/ntdll/tests/wow64.c @@ -39,6 +39,7 @@ static NTSTATUS (WINAPI *pNtWow64ReadVirtualMemory64)(HANDLE,ULONG64,void*,ULONG static NTSTATUS (WINAPI *pNtWow64WriteVirtualMemory64)(HANDLE,ULONG64,const void *,ULONG64,ULONG64*); #endif +static BOOL is_win64 = (sizeof(void *) > sizeof(int)); static BOOL is_wow64; static void *code_mem; @@ -234,6 +235,7 @@ static void test_peb_teb(void) NTSTATUS status; void *redir; SIZE_T res; + BOOL ret; TEB teb; PEB peb; TEB32 teb32; @@ -326,15 +328,17 @@ static void test_peb_teb(void) params32.EnvironmentSize, params.EnvironmentSize ); } - ok( DebugActiveProcess( pi.dwProcessId ), "debugging failed\n" ); + ret = DebugActiveProcess( pi.dwProcessId ); + todo_wine_if( is_win64 ) + ok( ret, "debugging failed\n" ); if (!ReadProcessMemory( pi.hProcess, proc_info.PebBaseAddress, &peb, sizeof(peb), &res )) res = 0; ok( res == sizeof(peb), "wrong len %lx\n", res ); - ok( peb.BeingDebugged == 1, "BeingDebugged is %u\n", peb.BeingDebugged ); + ok( peb.BeingDebugged == !!ret, "BeingDebugged is %u\n", peb.BeingDebugged ); if (!is_wow64) { if (!ReadProcessMemory( pi.hProcess, ULongToPtr(teb32.Peb), &peb32, sizeof(peb32), &res )) res = 0; ok( res == sizeof(peb32), "wrong len %lx\n", res ); - ok( peb32.BeingDebugged == 1, "BeingDebugged is %u\n", peb32.BeingDebugged ); + ok( peb32.BeingDebugged == !!ret, "BeingDebugged is %u\n", peb32.BeingDebugged ); } TerminateProcess( pi.hProcess, 0 );

1 0

Alexandre Julliard : kernel32/tests: Fix some test failures on Windows.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 1431702f5a0085fabdb8e22651dce18defd07148 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1431702f5a0085fabdb8e226… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 17:04:46 2021 +0100 kernel32/tests: Fix some test failures on Windows. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/file.c | 10 +++++++--- dlls/kernel32/tests/loader.c | 3 ++- dlls/kernel32/tests/locale.c | 4 ++-- 3 files changed, 11 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c index b4c7adacf7e..60fe532adc2 100644 --- a/dlls/kernel32/tests/file.c +++ b/dlls/kernel32/tests/file.c @@ -2709,7 +2709,7 @@ static void test_FindFirstFileA(void) /* try FindFirstFileA on "C:\foo\" */ SetLastError( 0xdeadbeaf ); - if (!GetTempFileNameA( buffer, "foo", 0, nonexistent ) && GetLastError() == ERROR_ACCESS_DENIED) + if (!GetTempFileNameA( buffer, "foo", 0, nonexistent )) { char tmp[MAX_PATH]; GetTempPathA( sizeof(tmp), tmp ); @@ -2825,8 +2825,12 @@ static void test_FindFirstFileA(void) strcat(buffer2, "foo\\bar\\nul"); handle = FindFirstFileA(buffer2, &data); err = GetLastError(); - ok ( handle == INVALID_HANDLE_VALUE, "FindFirstFile on %s should fail\n", buffer2 ); - ok ( err == ERROR_PATH_NOT_FOUND, "Bad Error number %d\n", err ); + ok( handle == INVALID_HANDLE_VALUE || broken(1), /* win8 */ + "FindFirstFile on %s should fail\n", buffer2 ); + if (handle == INVALID_HANDLE_VALUE) + ok( err == ERROR_PATH_NOT_FOUND, "Bad Error number %d\n", err ); + else + CloseHandle( handle ); /* try FindFirstFileA on "c:\foo\nul\bar" */ SetLastError( 0xdeadbeaf ); diff --git a/dlls/kernel32/tests/loader.c b/dlls/kernel32/tests/loader.c index 308cf1a44a0..c5857174403 100644 --- a/dlls/kernel32/tests/loader.c +++ b/dlls/kernel32/tests/loader.c @@ -718,7 +718,8 @@ static NTSTATUS map_image_section( const IMAGE_NT_HEADERS *nt_header, const IMAG else { ok( ldr_status == expect_status || - broken(il_only && !expect_status && ldr_status == STATUS_INVALID_IMAGE_FORMAT), + broken(il_only && !expect_status && ldr_status == STATUS_INVALID_IMAGE_FORMAT) || + broken(nt_header->Signature == IMAGE_OS2_SIGNATURE && ldr_status == STATUS_INVALID_IMAGE_NE_FORMAT), "%u: wrong status %x/%x\n", line, ldr_status, expect_status ); ok( !expect_fallback || broken(il_only) || broken(wrong_machine), "%u: failed with %x expected fallback\n", line, ldr_status ); diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c index f529b11a1a7..1f30547df5b 100644 --- a/dlls/kernel32/tests/locale.c +++ b/dlls/kernel32/tests/locale.c @@ -6774,13 +6774,13 @@ static void test_NLSVersion(void) ok( !ret, "IsValidNLSVersion succeeded\n" ); ok( GetLastError() == 0, "wrong error %u\n", GetLastError() ); - info.dwNLSVersion -= 0x200; + info.dwNLSVersion -= 0x800; SetLastError( 0xdeadbeef ); ret = pIsValidNLSVersion( COMPARE_STRING, L"en-US", &info ); ok( !ret, "IsValidNLSVersion succeeded\n" ); ok( GetLastError() == 0, "wrong error %u\n", GetLastError() ); - info.dwNLSVersion += 0x100; + info.dwNLSVersion += 0x700; info.dwDefinedVersion += 0x100; SetLastError( 0xdeadbeef ); ret = pIsValidNLSVersion( COMPARE_STRING, L"en-US", &info );

1 0

Alexandre Julliard : kernel32/tests: Remove an unreliable test.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 3f80ea69fbbb875250b91329ba12f5a842f9bfe8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3f80ea69fbbb875250b91329… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 18:15:42 2021 +0100 kernel32/tests: Remove an unreliable test. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/process.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index 0d1a2f1a31d..6251ee05f94 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -1644,8 +1644,6 @@ static void test_Console(void) ok(modeInC == (modeIn ^ 1), "Wrong console mode\n"); ok(modeOutC == (modeOut ^ 1), "Wrong console-SB mode\n"); - trace("cursor position(X): %d/%d\n",sbi.dwCursorPosition.X, sbiC.dwCursorPosition.X); - ok(sbiC.dwCursorPosition.Y == (sbi.dwCursorPosition.Y ^ 1), "Wrong cursor position\n"); release_memory(); DeleteFileA(resfile);

1 0

Alexandre Julliard : kernelbase/tests: Fix some test failures on Windows.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: f08eb97c48f16c25024eb2f7b669fb3d82ffec34 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f08eb97c48f16c25024eb2f7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 18:03:55 2021 +0100 kernelbase/tests: Fix some test failures on Windows. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/tests/process.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/kernelbase/tests/process.c b/dlls/kernelbase/tests/process.c index 7aa07df6e4e..a28411f4e5b 100644 --- a/dlls/kernelbase/tests/process.c +++ b/dlls/kernelbase/tests/process.c @@ -63,7 +63,7 @@ static void test_CompareObjectHandles(void) CloseHandle( h1 ); - h1 = CreateFileA( "\\\\.\\NUL", GENERIC_ALL, 0, NULL, OPEN_EXISTING, 0, 0 ); + h1 = CreateFileA( "\\\\.\\NUL", GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, 0 ); ok( h1 != INVALID_HANDLE_VALUE, "CreateFile failed (%d)\n", GetLastError() ); h2 = NULL; @@ -76,7 +76,7 @@ static void test_CompareObjectHandles(void) CloseHandle( h2 ); - h2 = CreateFileA( "\\\\.\\NUL", GENERIC_ALL, 0, NULL, OPEN_EXISTING, 0, 0 ); + h2 = CreateFileA( "\\\\.\\NUL", GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0, 0 ); ok( h2 != INVALID_HANDLE_VALUE, "CreateFile failed (%d)\n", GetLastError() ); SetLastError(0);

1 0

Alexandre Julliard : schedsvc/tests: Fix a test failure on recent Windows versions.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 2c1a68e35c35d0d2c771d97412850d388abfc47b URL: https://source.winehq.org/git/wine.git/?a=commit;h=2c1a68e35c35d0d2c771d974… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 18:28:33 2021 +0100 schedsvc/tests: Fix a test failure on recent Windows versions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/schedsvc/tests/rpcapi.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/dlls/schedsvc/tests/rpcapi.c b/dlls/schedsvc/tests/rpcapi.c index b806b97539d..a8f75ded7a5 100644 --- a/dlls/schedsvc/tests/rpcapi.c +++ b/dlls/schedsvc/tests/rpcapi.c @@ -520,7 +520,10 @@ todo_wine if (hr != E_ACCESSDENIED) { ok(!info, "expected NULL, info %p\n", info); - hr = IIDFromString(path, &iid); + if (path[0] == '\\') + hr = IIDFromString(path + 1, &iid); + else + hr = IIDFromString(path, &iid); ok(hr == S_OK, "IIDFromString(%s) error %#x\n", wine_dbgstr_w(path), hr); hr = SchRpcDelete(path, 0); ok(hr == S_OK, "expected S_OK, got %#x\n", hr);

1 0

Alexandre Julliard : advapi32/tests: Skip event log tests on Windows when access is not allowed.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: ea7eb95682be3894a8aed81ac497036562aa6f15 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ea7eb95682be3894a8aed81a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 16 20:29:30 2021 +0100 advapi32/tests: Skip event log tests on Windows when access is not allowed. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/advapi32/tests/eventlog.c | 34 +++++++++++++++++++++++++++++++--- 1 file changed, 31 insertions(+), 3 deletions(-) diff --git a/dlls/advapi32/tests/eventlog.c b/dlls/advapi32/tests/eventlog.c index 0c5d1e2523f..7e66e9ad8d7 100644 --- a/dlls/advapi32/tests/eventlog.c +++ b/dlls/advapi32/tests/eventlog.c @@ -65,6 +65,11 @@ static BOOL create_backup(const char *filename) DWORD rc, attribs; handle = OpenEventLogA(NULL, "Application"); + if (!handle && (GetLastError() == ERROR_ACCESS_DENIED || GetLastError() == RPC_S_SERVER_UNAVAILABLE)) + { + win_skip( "Can't open event log\n" ); + return FALSE; + } ok(handle != NULL, "OpenEventLogA(Application) failed : %d\n", GetLastError()); DeleteFileA(filename); @@ -274,6 +279,11 @@ static void test_oldest(void) ok(oldest == 0xdeadbeef, "Expected oldest to stay unchanged\n"); handle = OpenEventLogA(NULL, "Application"); + if (!handle && (GetLastError() == ERROR_ACCESS_DENIED || GetLastError() == RPC_S_SERVER_UNAVAILABLE)) + { + win_skip( "Can't open event log\n" ); + return; + } ok(handle != NULL, "OpenEventLogA(Application) failed : %d\n", GetLastError()); SetLastError(0xdeadbeef); @@ -327,6 +337,11 @@ static void test_backup(void) ok(GetFileAttributesA(backup) == INVALID_FILE_ATTRIBUTES, "Expected no backup file\n"); handle = OpenEventLogA(NULL, "Application"); + if (!handle && (GetLastError() == ERROR_ACCESS_DENIED || GetLastError() == RPC_S_SERVER_UNAVAILABLE)) + { + win_skip( "Can't open event log\n" ); + return; + } ok(handle != NULL, "OpenEventLogA(Application) failed : %d\n", GetLastError()); SetLastError(0xdeadbeef); @@ -439,6 +454,11 @@ static void test_read(void) HeapFree(GetProcessHeap(), 0, buf); handle = OpenEventLogA(NULL, "Application"); + if (!handle && (GetLastError() == ERROR_ACCESS_DENIED || GetLastError() == RPC_S_SERVER_UNAVAILABLE)) + { + win_skip( "Can't open event log\n" ); + return; + } ok(handle != NULL, "OpenEventLogA(Application) failed : %d\n", GetLastError()); /* Show that we need the proper dwFlags with a (for the rest) proper call */ @@ -537,7 +557,10 @@ static void test_openbackup(void) SetLastError(0xdeadbeef); handle = OpenBackupEventLogA(NULL, "idontexist.evt"); ok(handle == NULL, "Didn't expect a handle\n"); - ok(GetLastError() == ERROR_FILE_NOT_FOUND, "Expected ERROR_FILE_NOT_FOUND, got %d\n", GetLastError()); + ok(GetLastError() == ERROR_FILE_NOT_FOUND || + GetLastError() == ERROR_ACCESS_DENIED || + GetLastError() == RPC_S_SERVER_UNAVAILABLE, + "got %d\n", GetLastError()); SetLastError(0xdeadbeef); handle = OpenBackupEventLogA("IDontExist", NULL); @@ -593,8 +616,10 @@ static void test_openbackup(void) handle = OpenBackupEventLogA(NULL, backup); ok(handle == NULL, "Didn't expect a handle\n"); ok(GetLastError() == ERROR_NOT_ENOUGH_MEMORY || + GetLastError() == ERROR_ACCESS_DENIED || + GetLastError() == RPC_S_SERVER_UNAVAILABLE || GetLastError() == ERROR_EVENTLOG_FILE_CORRUPT, /* Vista and Win7 */ - "Expected ERROR_NOT_ENOUGH_MEMORY, got %d\n", GetLastError()); + "got %d\n", GetLastError()); CloseEventLog(handle); DeleteFileA(backup); @@ -604,7 +629,10 @@ static void test_openbackup(void) SetLastError(0xdeadbeef); handle = OpenBackupEventLogA(NULL, backup); ok(handle == NULL, "Didn't expect a handle\n"); - ok(GetLastError() == ERROR_EVENTLOG_FILE_CORRUPT, "Expected ERROR_EVENTLOG_FILE_CORRUPT, got %d\n", GetLastError()); + ok(GetLastError() == ERROR_EVENTLOG_FILE_CORRUPT || + GetLastError() == ERROR_ACCESS_DENIED || + GetLastError() == RPC_S_SERVER_UNAVAILABLE, + "got %d\n", GetLastError()); CloseEventLog(handle); DeleteFileA(backup); }

1 0

Jacek Caban : winebuild: Allow using clang for as command.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: f4ddcc36e97d391e841f82a07d4c06bd7c0dc701 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f4ddcc36e97d391e841f82a0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Dec 16 17:16:51 2021 +0100 winebuild: Allow using clang for as command. Since a9b5bb326a1514e2c4185633ab34b22c9bbc9863, Wine needs get_as_command to work without cc_command available. While as is usually a GCC dependency, Clang toolchains may not contain it. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winebuild/utils.c | 35 ++++++++++++++++++++++++++--------- 1 file changed, 26 insertions(+), 9 deletions(-) diff --git a/tools/winebuild/utils.c b/tools/winebuild/utils.c index a4393850823..42c5dca8516 100644 --- a/tools/winebuild/utils.c +++ b/tools/winebuild/utils.c @@ -250,7 +250,7 @@ struct strarray find_tool( const char *name, const char * const *names ) names++; } - if (!file && strcmp( name, "as" )) /* llvm-as is not a gas replacement */ + if (!file) { if (cc_command.count) file = find_clang_tool( cc_command, name ); if (!file && !(file = find_binary( "llvm", name ))) @@ -289,11 +289,36 @@ struct strarray find_link_tool(void) struct strarray get_as_command(void) { struct strarray args = empty_strarray; + const char *file; unsigned int i; + int using_cc = 0; if (cc_command.count) { strarray_addall( &args, cc_command ); + using_cc = 1; + } + else if (as_command.count) + { + strarray_addall( &args, as_command ); + } + else if ((file = find_binary( target_alias, "as" )) || (file = find_binary( target_alias, "gas "))) + { + strarray_add( &args, file ); + } + else if ((file = find_binary( NULL, "clang" ))) + { + strarray_add( &args, file ); + if (target_alias) + { + strarray_add( &args, "-target" ); + strarray_add( &args, target_alias ); + } + using_cc = 1; + } + + if (using_cc) + { strarray_add( &args, "-xassembler" ); strarray_add( &args, "-c" ); if (force_pointer_size) @@ -306,14 +331,6 @@ struct strarray get_as_command(void) return args; } - if (!as_command.count) - { - static const char * const commands[] = { "gas", "as", NULL }; - as_command = find_tool( "as", commands ); - } - - strarray_addall( &args, as_command ); - if (force_pointer_size) { switch (target.platform)

1 0

Eric Pouech : dbghelp: Silence some FIXMEs.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 6c64e6dee4dc467d240ef44f177d95068a59e7d3 URL: https://source.winehq.org/git/wine.git/?a=commit;h=6c64e6dee4dc467d240ef44f… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Wed Dec 15 13:32:00 2021 +0100 dbghelp: Silence some FIXMEs. Reading gecko's PDB generate a lot of those. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dbghelp/msc.c | 2 +- dlls/dbghelp/symbol.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c index 894d01a4c00..317f7bb84b5 100644 --- a/dlls/dbghelp/msc.c +++ b/dlls/dbghelp/msc.c @@ -1926,7 +1926,7 @@ static struct symt_inlinesite* codeview_create_inline_site(const struct msc_debu if (!found) { - FIXME("Couldn't find start address of inlined\n"); + WARN("Couldn't find start address of inlined\n"); return NULL; } diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c index 2443dbf7e73..d3ba4521fa0 100644 --- a/dlls/dbghelp/symbol.c +++ b/dlls/dbghelp/symbol.c @@ -612,7 +612,7 @@ BOOL symt_add_inlinesite_range(struct module* module, { struct addr_range* ar = (struct addr_range*)vector_at(&inlined->vranges, i); if (!addr_range_disjoint(ar, p) && !addr_range_inside(ar, p)) - FIXME("Added addr_range not compatible with parent\n"); + WARN("Added addr_range not compatible with parent\n"); } } }

1 0

Eric Pouech : msvcrt: Fix double-free and memory leak in type_info destructor.
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 01111b941d469cb13ed9d3f63c2935930679bcb8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=01111b941d469cb13ed9d3f6… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Thu Dec 16 19:06:48 2021 +0100 msvcrt: Fix double-free and memory leak in type_info destructor. Spotted by toying with the gcc's static analyzer. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcp90/cxx.h | 2 +- dlls/msvcrt/cxx.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msvcp90/cxx.h b/dlls/msvcp90/cxx.h index 29854d16bab..701ab02fc05 100644 --- a/dlls/msvcp90/cxx.h +++ b/dlls/msvcp90/cxx.h @@ -457,7 +457,7 @@ void * __thiscall type_info_vector_dtor(type_info * _this, unsigned int flags) \ /* we have an array, with the number of elements stored before the first object */ \ INT_PTR i, *ptr = (INT_PTR *)_this - 1; \ \ - for (i = *ptr - 1; i >= 0; i--) free(_this->name); \ + for (i = *ptr - 1; i >= 0; i--) free(_this[i].name); \ free(ptr); \ } \ else \ diff --git a/dlls/msvcrt/cxx.h b/dlls/msvcrt/cxx.h index 1a5442725ef..cacbb1524b6 100644 --- a/dlls/msvcrt/cxx.h +++ b/dlls/msvcrt/cxx.h @@ -291,7 +291,7 @@ void * __thiscall type_info_vector_dtor(type_info * _this, unsigned int flags) \ /* we have an array, with the number of elements stored before the first object */ \ INT_PTR i, *ptr = (INT_PTR *)_this - 1; \ \ - for (i = *ptr - 1; i >= 0; i--) free(_this->name); \ + for (i = *ptr - 1; i >= 0; i--) free(_this[i].name); \ free(ptr); \ } \ else \

1 0

John Sullivan : user32: Call DestroyIcon() to delete icon handles in CopyImage().
by Alexandre Julliard 16 Dec '21

16 Dec '21

Module: wine Branch: master Commit: 4a200821672dfe719dc5b20df853e719678496b7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4a200821672dfe719dc5b20d… Author: John Sullivan <jswinebz(a)kanargh.org.uk> Date: Thu Dec 16 11:58:37 2021 -0600 user32: Call DestroyIcon() to delete icon handles in CopyImage(). Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52207 Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/cursoricon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c index 828c88de92f..582b2612688 100644 --- a/dlls/user32/cursoricon.c +++ b/dlls/user32/cursoricon.c @@ -3183,7 +3183,7 @@ HANDLE WINAPI CopyImage( HANDLE hnd, UINT type, INT desiredx, release_icon_frame( icon, frame ); release_user_handle_ptr( icon ); - if (res && (flags & LR_COPYDELETEORG)) DeleteObject( hnd ); + if (res && (flags & LR_COPYDELETEORG)) DestroyIcon( hnd ); return res; } }

1 0