June 2011 - Wine-commits - list.winehq.org

Andrew Nguyen : ws2_32: Free the asynchronous query structure if thread creation fails.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: a5e0f09b52e045c70965947f8f98fa8b84de6edd URL: http://source.winehq.org/git/wine.git/?a=commit;h=a5e0f09b52e045c70965947f8… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Wed Jun 1 07:14:08 2011 -0500 ws2_32: Free the asynchronous query structure if thread creation fails. --- dlls/ws2_32/async.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/ws2_32/async.c b/dlls/ws2_32/async.c index 2f79968..ce28142 100644 --- a/dlls/ws2_32/async.c +++ b/dlls/ws2_32/async.c @@ -284,6 +284,7 @@ static HANDLE run_query( HWND hWnd, UINT uMsg, LPTHREAD_START_ROUTINE func, if (!thread) { SetLastError( WSAEWOULDBLOCK ); + HeapFree( GetProcessHeap(), 0, query ); return 0; } CloseHandle( thread );

1 0

Andrew Nguyen : ws2_32/tests: Properly destroy the async test window.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 2c74796456a74d00ddb1ce172fa8904620fea9c6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c74796456a74d00ddb1ce172… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Wed Jun 1 07:14:13 2011 -0500 ws2_32/tests: Properly destroy the async test window. --- dlls/ws2_32/tests/sock.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index 07803ff..5db9013 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -3796,7 +3796,7 @@ end: if (hThread != NULL) CloseHandle(hThread); if (hWnd != NULL) - CloseHandle(hWnd); + DestroyWindow(hWnd); if (hEvent != NULL) CloseHandle(hEvent); if (hEvent2 != NULL)

1 0

Andrew Nguyen : ws2_32: Avoid a crash with an optional pointer in WSAAsyncGetServByPort.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 9edafeb742d8ce0436e7e055998a94957cfacdd9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9edafeb742d8ce0436e7e0559… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Wed Jun 1 07:14:18 2011 -0500 ws2_32: Avoid a crash with an optional pointer in WSAAsyncGetServByPort. --- dlls/ws2_32/async.c | 16 ++++++++--- dlls/ws2_32/tests/sock.c | 63 ++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 75 insertions(+), 4 deletions(-) diff --git a/dlls/ws2_32/async.c b/dlls/ws2_32/async.c index ce28142..01d5d48 100644 --- a/dlls/ws2_32/async.c +++ b/dlls/ws2_32/async.c @@ -407,7 +407,7 @@ HANDLE WINAPI WSAAsyncGetServByPort(HWND hWnd, UINT uMsg, INT port, LPCSTR proto, LPSTR sbuf, INT buflen) { struct async_query_getservbyport *aq; - unsigned int len = strlen(proto) + 1; + unsigned int len = proto ? strlen(proto) + 1 : 0; TRACE("hwnd %p, msg %04x, port %i, proto %s\n", hWnd, uMsg, port, debugstr_a(proto)); @@ -416,9 +416,17 @@ HANDLE WINAPI WSAAsyncGetServByPort(HWND hWnd, UINT uMsg, INT port, SetLastError( WSAEWOULDBLOCK ); return 0; } - aq->serv_proto = (char *)(aq + 1); - aq->serv_port = port; - strcpy( aq->serv_proto, proto ); + + if (proto) + { + aq->serv_proto = (char *)(aq + 1); + strcpy( aq->serv_proto, proto ); + } + else + aq->serv_proto = NULL; + + aq->serv_port = port; + return run_query( hWnd, uMsg, async_getservbyport, &aq->query, sbuf, buflen ); } diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index 5db9013..f6a3e57 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -4810,6 +4810,67 @@ static void test_synchronous_WSAIoctl(void) CloseHandle( previous_port ); } +#define WM_ASYNCCOMPLETE (WM_USER + 100) +static HWND create_async_message_window(void) +{ + static const char class_name[] = "ws2_32 async message window class"; + + WNDCLASSEXA wndclass; + HWND hWnd; + + wndclass.cbSize = sizeof(wndclass); + wndclass.style = CS_HREDRAW | CS_VREDRAW; + wndclass.lpfnWndProc = DefWindowProcA; + wndclass.cbClsExtra = 0; + wndclass.cbWndExtra = 0; + wndclass.hInstance = GetModuleHandleA(NULL); + wndclass.hIcon = LoadIconA(NULL, IDI_APPLICATION); + wndclass.hIconSm = LoadIconA(NULL, IDI_APPLICATION); + wndclass.hCursor = LoadCursorA(NULL, IDC_ARROW); + wndclass.hbrBackground = (HBRUSH)(COLOR_WINDOW + 1); + wndclass.lpszClassName = class_name; + wndclass.lpszMenuName = NULL; + + RegisterClassExA(&wndclass); + + hWnd = CreateWindow(class_name, "ws2_32 async message window", WS_OVERLAPPEDWINDOW, + 0, 0, 500, 500, NULL, NULL, GetModuleHandleA(NULL), NULL); + if (!hWnd) + { + ok(0, "failed to create window: %u\n", GetLastError()); + return NULL; + } + + return hWnd; +} + +static void test_WSAAsyncGetServByPort(void) +{ + HWND hwnd = create_async_message_window(); + HANDLE ret; + char buffer[MAXGETHOSTSTRUCT]; + + if (!hwnd) + return; + + /* FIXME: The asynchronous window messages should be tested. */ + + /* Parameters are not checked when initiating the asynchronous operation. */ + ret = WSAAsyncGetServByPort(NULL, 0, 0, NULL, NULL, 0); + ok(ret != NULL, "WSAAsyncGetServByPort returned NULL\n"); + + ret = WSAAsyncGetServByPort(hwnd, WM_ASYNCCOMPLETE, 0, NULL, NULL, 0); + ok(ret != NULL, "WSAAsyncGetServByPort returned NULL\n"); + + ret = WSAAsyncGetServByPort(hwnd, WM_ASYNCCOMPLETE, htons(80), NULL, NULL, 0); + ok(ret != NULL, "WSAAsyncGetServByPort returned NULL\n"); + + ret = WSAAsyncGetServByPort(hwnd, WM_ASYNCCOMPLETE, htons(80), NULL, buffer, MAXGETHOSTSTRUCT); + ok(ret != NULL, "WSAAsyncGetServByPort returned NULL\n"); + + DestroyWindow(hwnd); +} + /**************** Main program ***************/ START_TEST( sock ) @@ -4870,6 +4931,8 @@ START_TEST( sock ) test_sioRoutingInterfaceQuery(); + test_WSAAsyncGetServByPort(); + /* this is a io heavy test, do it at the end so the kernel doesn't start dropping packets */ test_send(); test_synchronous_WSAIoctl();

1 0

Andrew Nguyen : dbghelp: Implement SymEnumSourceFilesW.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 6ee16099a3b541c656b2146821fc6787aca6112b URL: http://source.winehq.org/git/wine.git/?a=commit;h=6ee16099a3b541c656b214682… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Tue May 31 06:12:44 2011 -0500 dbghelp: Implement SymEnumSourceFilesW. --- dlls/dbghelp/dbghelp.spec | 2 +- dlls/dbghelp/dbghelp_private.h | 3 + dlls/dbghelp/module.c | 6 +- dlls/dbghelp/source.c | 122 +++++++++++++++++++++++++++++++++++++--- 4 files changed, 121 insertions(+), 12 deletions(-) diff --git a/dlls/dbghelp/dbghelp.spec b/dlls/dbghelp/dbghelp.spec index c50ab22..ddb9d33 100644 --- a/dlls/dbghelp/dbghelp.spec +++ b/dlls/dbghelp/dbghelp.spec @@ -45,7 +45,7 @@ @ stub SymEnumProcesses @ stub SymEnumSourceFileTokens @ stdcall SymEnumSourceFiles(ptr int64 str ptr ptr) -@ stub SymEnumSourceFilesW +@ stdcall SymEnumSourceFilesW(ptr int64 wstr ptr ptr) @ stub SymEnumSourceLines @ stub SymEnumSourceLinesW @ stub SymEnumSym diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index 4a64253..43aca49 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -538,6 +538,9 @@ extern struct module* module_find_by_addr(const struct process* pcs, unsigned long addr, enum module_type type) DECLSPEC_HIDDEN; extern struct module* + module_find_by_nameW(const struct process* pcs, + const WCHAR* name) DECLSPEC_HIDDEN; +extern struct module* module_find_by_nameA(const struct process* pcs, const char* name) DECLSPEC_HIDDEN; extern struct module* diff --git a/dlls/dbghelp/module.c b/dlls/dbghelp/module.c index 7f75cc2..2a1471f 100644 --- a/dlls/dbghelp/module.c +++ b/dlls/dbghelp/module.c @@ -234,10 +234,10 @@ struct module* module_new(struct process* pcs, const WCHAR* name, } /*********************************************************************** - * module_find_by_name + * module_find_by_nameW * */ -static struct module* module_find_by_name(const struct process* pcs, const WCHAR* name) +struct module* module_find_by_nameW(const struct process* pcs, const WCHAR* name) { struct module* module; @@ -254,7 +254,7 @@ struct module* module_find_by_nameA(const struct process* pcs, const char* name) WCHAR wname[MAX_PATH]; MultiByteToWideChar(CP_ACP, 0, name, -1, wname, sizeof(wname) / sizeof(WCHAR)); - return module_find_by_name(pcs, wname); + return module_find_by_nameW(pcs, wname); } /*********************************************************************** diff --git a/dlls/dbghelp/source.c b/dlls/dbghelp/source.c index 5f12b9f..3bed023 100644 --- a/dlls/dbghelp/source.c +++ b/dlls/dbghelp/source.c @@ -157,17 +157,19 @@ const char* source_get(const struct module* module, unsigned idx) } /****************************************************************** - * SymEnumSourceFiles (DBGHELP.@) + * SymEnumSourceFilesW (DBGHELP.@) * */ -BOOL WINAPI SymEnumSourceFiles(HANDLE hProcess, ULONG64 ModBase, PCSTR Mask, - PSYM_ENUMSOURCEFILES_CALLBACK cbSrcFiles, - PVOID UserContext) +BOOL WINAPI SymEnumSourceFilesW(HANDLE hProcess, ULONG64 ModBase, PCWSTR Mask, + PSYM_ENUMSOURCEFILES_CALLBACKW cbSrcFiles, + PVOID UserContext) { struct module_pair pair; - SOURCEFILE sf; + SOURCEFILEW sf; char* ptr; - + WCHAR* conversion_buffer = NULL; + DWORD conversion_buffer_len = 0; + if (!cbSrcFiles) return FALSE; pair.pcs = process_find_by_handle(hProcess); if (!pair.pcs) return FALSE; @@ -181,7 +183,7 @@ BOOL WINAPI SymEnumSourceFiles(HANDLE hProcess, ULONG64 ModBase, PCSTR Mask, { if (Mask[0] == '!') { - pair.requested = module_find_by_nameA(pair.pcs, Mask + 1); + pair.requested = module_find_by_nameW(pair.pcs, Mask + 1); if (!module_get_debug(&pair)) return FALSE; } else @@ -193,15 +195,119 @@ BOOL WINAPI SymEnumSourceFiles(HANDLE hProcess, ULONG64 ModBase, PCSTR Mask, if (!pair.effective->sources) return FALSE; for (ptr = pair.effective->sources; *ptr; ptr += strlen(ptr) + 1) { + DWORD len = MultiByteToWideChar(CP_ACP, 0, ptr, -1, NULL, 0); + + if (len > conversion_buffer_len) + { + HeapFree(GetProcessHeap(), 0, conversion_buffer); + conversion_buffer = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!conversion_buffer) return FALSE; + conversion_buffer_len = len; + } + + MultiByteToWideChar(CP_ACP, 0, ptr, -1, conversion_buffer, len); + /* FIXME: not using Mask */ sf.ModBase = ModBase; - sf.FileName = ptr; + sf.FileName = conversion_buffer; if (!cbSrcFiles(&sf, UserContext)) break; } + HeapFree(GetProcessHeap(), 0, conversion_buffer); return TRUE; } +struct enum_sources_files_context +{ + PSYM_ENUMSOURCEFILES_CALLBACK callbackA; + PVOID caller_context; + char *conversion_buffer; + DWORD conversion_buffer_len; + DWORD callback_error; +}; + +static BOOL CALLBACK enum_source_files_W_to_A(PSOURCEFILEW source_file, PVOID context) +{ + struct enum_sources_files_context *ctx = context; + SOURCEFILE source_fileA; + DWORD len; + + len = WideCharToMultiByte(CP_ACP, 0, source_file->FileName, -1, NULL, 0, NULL, NULL); + if (len > ctx->conversion_buffer_len) + { + char *ptr = ctx->conversion_buffer ? HeapReAlloc(GetProcessHeap(), 0, ctx->conversion_buffer, len) : + HeapAlloc(GetProcessHeap(), 0, len); + + if (!ptr) + { + ctx->callback_error = ERROR_OUTOFMEMORY; + return FALSE; + } + + ctx->conversion_buffer = ptr; + ctx->conversion_buffer_len = len; + } + + WideCharToMultiByte(CP_ACP, 0, source_file->FileName, -1, ctx->conversion_buffer, len, NULL, NULL); + + source_fileA.ModBase = source_file->ModBase; + source_fileA.FileName = ctx->conversion_buffer; + return ctx->callbackA(&source_fileA, ctx->caller_context); +} + +/****************************************************************** + * SymEnumSourceFiles (DBGHELP.@) + * + */ +BOOL WINAPI SymEnumSourceFiles(HANDLE hProcess, ULONG64 ModBase, PCSTR Mask, + PSYM_ENUMSOURCEFILES_CALLBACK cbSrcFiles, + PVOID UserContext) +{ + WCHAR *maskW = NULL; + PSYM_ENUMSOURCEFILES_CALLBACKW callbackW; + PVOID context; + struct enum_sources_files_context callback_context = {cbSrcFiles, UserContext}; + BOOL ret; + + if (Mask) + { + DWORD len = MultiByteToWideChar(CP_ACP, 0, Mask, -1, NULL, 0); + + maskW = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (!maskW) + { + SetLastError(ERROR_OUTOFMEMORY); + return FALSE; + } + + MultiByteToWideChar(CP_ACP, 0, Mask, -1, maskW, len); + } + + if (cbSrcFiles) + { + callbackW = enum_source_files_W_to_A; + context = &callback_context; + } + else + { + callbackW = NULL; + context = UserContext; + } + + ret = SymEnumSourceFilesW(hProcess, ModBase, maskW, callbackW, context); + + if (callback_context.callback_error) + { + SetLastError(callback_context.callback_error); + ret = FALSE; + } + + HeapFree(GetProcessHeap(), 0, callback_context.conversion_buffer); + HeapFree(GetProcessHeap(), 0, maskW); + + return ret; +} + /****************************************************************** * SymGetSourceFileToken (DBGHELP.@) *

1 0

Alasdair Sinclair : comctl32/tab: Fully initialize itemData to fix x64 test failure.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: b6484ccbebb67d107934f9b6d4a142eb70ac946d URL: http://source.winehq.org/git/wine.git/?a=commit;h=b6484ccbebb67d107934f9b6d… Author: Alasdair Sinclair <alasdairsinc(a)gmail.com> Date: Wed Jun 1 03:24:30 2011 +0100 comctl32/tab: Fully initialize itemData to fix x64 test failure. --- dlls/comctl32/tab.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/comctl32/tab.c b/dlls/comctl32/tab.c index 0424f7f..1ad8c53 100644 --- a/dlls/comctl32/tab.c +++ b/dlls/comctl32/tab.c @@ -1770,6 +1770,7 @@ TAB_DrawItemInterior(const TAB_INFO *infoPtr, HDC hdc, INT iItem, RECT *drawRect { /* this could be considered broken on 64 bit, but that's how it works - only first 4 bytes are copied */ + dis.itemData = 0; memcpy(&dis.itemData, (ULONG_PTR*)TAB_GetItem(infoPtr, iItem)->extra, 4); }

1 0

Huw Davies : gdi32/tests: Add tests for 24 bpp brushes.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: cc4c2529620a3eaf78440a8ba2e71e55ff44bb58 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cc4c2529620a3eaf78440a8ba… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Jun 1 09:41:42 2011 +0100 gdi32/tests: Add tests for 24 bpp brushes. --- dlls/gdi32/tests/dib.c | 50 ++++++++++++++++++++++++++++++++++++++++++----- 1 files changed, 44 insertions(+), 6 deletions(-) diff --git a/dlls/gdi32/tests/dib.c b/dlls/gdi32/tests/dib.c index e1ef345..c750a46 100644 --- a/dlls/gdi32/tests/dib.c +++ b/dlls/gdi32/tests/dib.c @@ -93,6 +93,7 @@ static const char *sha1_graphics_a8r8g8b8[] = "b2261353decda2712b83538ab434a49ce21f3172", "ef654fedcb494dae79559f4db8b691ae2d522a3f", "a694872f38e66a7ff471440c3e6a9310ef78328a", + "c311dd74325e8cebfc8529a6d24a6fa4ecb7137e", "d7398de15b2837a58a62a701ca1b3384625afec4", "a78b28472bb7ff480ddedd06b9cf2daa775fa7ae", "5246ef357e7317b9d141a3294d300c195da76cb7", @@ -121,6 +122,7 @@ static const char *sha1_graphics_a8b8g8r8[] = "1f1fc165a4dae7ba118ddccb58a279bfe3876b0a", "66da15b6780a4ca3b3d2eb1d1dba4e30f7b74964", "42fefefe4435570fa8038ec759140c66c76570e9", + "ca7e859647b9498b53fdd92543ad8aea98ff46f3", "3369889a67d6c79a24ee15f7d14374f9995215e4", "473a1fd07df800c87a5d3286b642ace10c61c6af", "10cd25a0ed5cd8f978d7d68236f81d949b938e84", @@ -149,6 +151,7 @@ static const char *sha1_graphics_24[] = "1036b91d93e31cd1d4740d0c8642e115e5a38188", "3469776cc7d8f6fd3bce17a39eec0b4092294e49", "1cb7f3fcf42c9ca47a0689195ced239a09d3c308", + "413a7989969c229dee4ab1798362f32f96cf0a10", "0bb222e540b82720d4971e4a2fc626899af03e03", "adc20832d8c43f1cf372d8392535492013cd2306", "45649794dcbcabda487f66f7a80fc1bec79047a1", @@ -177,6 +180,7 @@ static const char *sha1_graphics_r5g5b5[] = "449092689226a1172b6086ba1181d6b6d6499f26", "5c636ffadec10fbe440b552fe6436f3dbc607dcf", "4aac89fc18c128eddb69eea658272af53138a1cb", + "897d16f4d6a6ddad685d23ed7828d4f676539b75", "9d21bcfdeaf1ca5d47eb823bdefc24d7a95f4f56", "6daaf945a955928c5c124c880522ca4634fb2343", "12a288390d16e1efa99d4185301de48a4d433b14", @@ -205,6 +209,7 @@ static const char *sha1_graphics_r4g4b4[] = "25fcb75aa687aac35b8f72640889fe92413e00c5", "dbc8d66b419880108793db91c05766c0c6efd86d", "6092ccbab6b6e073c6ac8244d122f2cfc453aa38", + "e61a4f2657a444d8c49f84fb944f9f847667bf2b", "32b6e0aa79b7e96cd0ab2da167f6463c011023a8", "1d283aa4d2b2114f7809fe59357d590c7c779aa7", "29640e2ddd2d3016da14507c3ce9b2ce32f39bb4", @@ -233,6 +238,7 @@ static const char *sha1_graphics_8[] = "73e2859ce849f756f954718ce3c90f02e31712b6", "196d832d283bf642d2e481e5452ca175f7902761", "9552f48b88982633a44001227abb847dae4d06b0", + "6e375e1485a1e45ac6ab10af49645d5fb2e76dff", "cfc67c325c7cdf96d90af9b3cceb8d0504cbb3b0", "7262364067e03c7fa498af1d59d228d6c63b460e", "5241241a355a667ef0834049adf4218e8b3f16b8", @@ -261,6 +267,7 @@ static const char *sha1_graphics_4[] = "3b53d24178cfacba53103a44dfd5d072b15a6781", "fa0d5f2d9419d8310fe4af7a290d13c5026c77de", "5070ecae5085209508f7450d5f21c69737a178e9", + "d17f4358ae529f920960ed89e535902ee13b0033", "0f44e12ecd1ea7e39433890443626d4fc35204a4", "eb38683e812fd13dca971ba8f4cfd2b6820d3524", "73bbc83f88f1aaa6df0158b63e70bb3165163163", @@ -435,7 +442,8 @@ static const RECT rectangles[] = {200, 40, 210, 43} /* height == 3 */ }; -static const BITMAPINFOHEADER dib_brush_header_32 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 32, BI_RGB, 0, 0, 0, 0, 0}; +static const BITMAPINFOHEADER dib_brush_header_8888 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 32, BI_RGB, 0, 0, 0, 0, 0}; +static const BITMAPINFOHEADER dib_brush_header_24 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 24, BI_RGB, 0, 0, 0, 0, 0}; static const BITMAPINFOHEADER dib_brush_header_555 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 16, BI_RGB, 0, 0, 0, 0, 0}; static const BITMAPINFOHEADER dib_brush_header_8 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 8, BI_RGB, 0, 0, 0, 0, 0}; static const BITMAPINFOHEADER dib_brush_header_4 = {sizeof(BITMAPINFOHEADER), 16, -16, 1, 4, BI_RGB, 0, 0, 0, 0, 0}; @@ -618,9 +626,9 @@ static void draw_graphics(HDC hdc, BITMAPINFO *bmi, BYTE *bits, const char ***sh ExtSelectClipRgn(hdc, NULL, RGN_COPY); - /* DIB pattern brush */ + /* 8888 DIB pattern brush */ - brush_bi->bmiHeader = dib_brush_header_32; + brush_bi->bmiHeader = dib_brush_header_8888; brush_bits = (BYTE*)brush_bi + sizeof(BITMAPINFOHEADER); memset(brush_bits, 0, 16 * 16 * sizeof(DWORD)); brush_bits[2] = 0xff; @@ -645,13 +653,13 @@ static void draw_graphics(HDC hdc, BITMAPINFO *bmi, BYTE *bits, const char ***sh y += 25; } } - compare_hash(bmi, bits, sha1, "top-down dib brush patblt"); + compare_hash(bmi, bits, sha1, "top-down 8888 dib brush patblt"); memset(bits, 0xcc, dib_size); SelectObject(hdc, orig_brush); DeleteObject(dib_brush); - /* Bottom-up DIB pattern brush */ + /* 8888 bottom-up DIB pattern brush */ brush_bi->bmiHeader.biHeight = -brush_bi->bmiHeader.biHeight; @@ -675,9 +683,39 @@ static void draw_graphics(HDC hdc, BITMAPINFO *bmi, BYTE *bits, const char ***sh y += 25; } } - compare_hash(bmi, bits, sha1, "bottom-up dib brush patblt"); + compare_hash(bmi, bits, sha1, "bottom-up 8888 dib brush patblt"); memset(bits, 0xcc, dib_size); + /* 24 bpp dib pattern brush */ + + brush_bi->bmiHeader = dib_brush_header_24; + brush_bits = (BYTE*)brush_bi + sizeof(BITMAPINFOHEADER); + memset(brush_bits, 0, 16 * 16 * 3); + brush_bits[0] = brush_bits[3] = brush_bits[6] = brush_bits[8] = 0xff; + brush_bits[49] = brush_bits[52] = 0xff; + + dib_brush = CreateDIBPatternBrushPt(brush_bi, DIB_RGB_COLORS); + + SelectObject(hdc, dib_brush); + SetBrushOrgEx(hdc, 1, 1, NULL); + + for(i = 0, y = 10; i < 256; i++) + { + BOOL ret; + + if(!rop_uses_src(rop3[i])) + { + ret = PatBlt(hdc, 10 + i, y, 100, 20, rop3[i]); + ok(ret, "got FALSE for %x\n", rop3[i]); + y += 25; + } + } + compare_hash(bmi, bits, sha1, "top-down 24 bpp brush patblt"); + memset(bits, 0xcc, dib_size); + + SelectObject(hdc, orig_brush); + DeleteObject(dib_brush); + /* 555 dib pattern brush */ brush_bi->bmiHeader = dib_brush_header_555;

1 0

Huw Davies : gdi32: Fix an error in the blue channel of the 32 -> 16 bpp conversion and update the tests to catch this.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: ea361367ed87a18704e8d36646e73692f3a1c69b URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea361367ed87a18704e8d3664… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Jun 1 09:41:43 2011 +0100 gdi32: Fix an error in the blue channel of the 32 -> 16 bpp conversion and update the tests to catch this. --- dlls/gdi32/dibdrv/primitives.c | 2 +- dlls/gdi32/tests/dib.c | 29 +++++++++++++++-------------- 2 files changed, 16 insertions(+), 15 deletions(-) diff --git a/dlls/gdi32/dibdrv/primitives.c b/dlls/gdi32/dibdrv/primitives.c index 4711a65..411cddd 100644 --- a/dlls/gdi32/dibdrv/primitives.c +++ b/dlls/gdi32/dibdrv/primitives.c @@ -1038,7 +1038,7 @@ static BOOL convert_to_555(dib_info *dst, const dib_info *src, const RECT *src_r src_val = *src_pixel++; *dst_pixel++ = ((src_val >> 9) & 0x7c00) | ((src_val >> 6) & 0x03e0) | - ((src_val >> 3) & 0x001e); + ((src_val >> 3) & 0x001f); } dst_start += dst->stride / 2; src_start += src->stride / 4; diff --git a/dlls/gdi32/tests/dib.c b/dlls/gdi32/tests/dib.c index c750a46..60791ca 100644 --- a/dlls/gdi32/tests/dib.c +++ b/dlls/gdi32/tests/dib.c @@ -91,8 +91,8 @@ static const char *sha1_graphics_a8r8g8b8[] = "6c530622a025d872a642e8f950867884d7b136cb", "7c07d91b8f68fb31821701b3dcb96de018bf0c66", "b2261353decda2712b83538ab434a49ce21f3172", - "ef654fedcb494dae79559f4db8b691ae2d522a3f", - "a694872f38e66a7ff471440c3e6a9310ef78328a", + "35f731c0f6356b8f30651bb3cbe0d922c49deba5", + "9b9874c1c1d92afa554137e191d34ea33acc322f", "c311dd74325e8cebfc8529a6d24a6fa4ecb7137e", "d7398de15b2837a58a62a701ca1b3384625afec4", "a78b28472bb7ff480ddedd06b9cf2daa775fa7ae", @@ -120,8 +120,8 @@ static const char *sha1_graphics_a8b8g8r8[] = "bf57a6a37fb107d29ed3d45695919887abcb7902", "3db0f8bcca3d94920aa57be6321202b8c3c08822", "1f1fc165a4dae7ba118ddccb58a279bfe3876b0a", - "66da15b6780a4ca3b3d2eb1d1dba4e30f7b74964", - "42fefefe4435570fa8038ec759140c66c76570e9", + "8e09abb108e137c99527ab4c9bd07d95b9254bbb", + "b0178632775d29bec2b16de7b9b8287115c40d0f", "ca7e859647b9498b53fdd92543ad8aea98ff46f3", "3369889a67d6c79a24ee15f7d14374f9995215e4", "473a1fd07df800c87a5d3286b642ace10c61c6af", @@ -149,8 +149,8 @@ static const char *sha1_graphics_24[] = "0fa8cf332a56bb6d7e14e85861fdd60f51d70501", "593d694cdcc8349b3bfc8257041dbcb27e61da45", "1036b91d93e31cd1d4740d0c8642e115e5a38188", - "3469776cc7d8f6fd3bce17a39eec0b4092294e49", - "1cb7f3fcf42c9ca47a0689195ced239a09d3c308", + "1898073cdb35ca4d2b21bba933ac16a0b4297317", + "5068bff794553cf5a3145ae407c9a2984357844c", "413a7989969c229dee4ab1798362f32f96cf0a10", "0bb222e540b82720d4971e4a2fc626899af03e03", "adc20832d8c43f1cf372d8392535492013cd2306", @@ -178,8 +178,8 @@ static const char *sha1_graphics_r5g5b5[] = "420e39ff3bdd04c4b6cc2c98e99cb7993c7a0de5", "1fabf0fdd046857b1974e31c1c1764fa9d1a762f", "449092689226a1172b6086ba1181d6b6d6499f26", - "5c636ffadec10fbe440b552fe6436f3dbc607dcf", - "4aac89fc18c128eddb69eea658272af53138a1cb", + "1a92a60f190d33ef06d9decb56fd3fdd33f3af03", + "e61f5978c2e28c0c6d8f5eefe0f840c975586efc", "897d16f4d6a6ddad685d23ed7828d4f676539b75", "9d21bcfdeaf1ca5d47eb823bdefc24d7a95f4f56", "6daaf945a955928c5c124c880522ca4634fb2343", @@ -207,8 +207,8 @@ static const char *sha1_graphics_r4g4b4[] = "115a6bd382410a4a1d3c1fa71d8bf02536863e38", "65c6d1228e3b6d63d42647f14217bc1658b70d9a", "25fcb75aa687aac35b8f72640889fe92413e00c5", - "dbc8d66b419880108793db91c05766c0c6efd86d", - "6092ccbab6b6e073c6ac8244d122f2cfc453aa38", + "3bddf9d53e89560b083302b146cd33791b13d941", + "a81504498c7a7bb46340ce74476a42f70f2730b1", "e61a4f2657a444d8c49f84fb944f9f847667bf2b", "32b6e0aa79b7e96cd0ab2da167f6463c011023a8", "1d283aa4d2b2114f7809fe59357d590c7c779aa7", @@ -236,8 +236,8 @@ static const char *sha1_graphics_8[] = "5ba8f99ca034666effa556748c49a0f5a015125f", "b67ba2f55659c75ac72c1112494461bb3086e1a4", "73e2859ce849f756f954718ce3c90f02e31712b6", - "196d832d283bf642d2e481e5452ca175f7902761", - "9552f48b88982633a44001227abb847dae4d06b0", + "b1dff0f5dd233b44ee568878c5d3f8ae1d80c6d9", + "1f27dc1a1316fb7a4a78fe40fcd4bdae3aaad218", "6e375e1485a1e45ac6ab10af49645d5fb2e76dff", "cfc67c325c7cdf96d90af9b3cceb8d0504cbb3b0", "7262364067e03c7fa498af1d59d228d6c63b460e", @@ -265,8 +265,8 @@ static const char *sha1_graphics_4[] = "665bbbc749a5ffeedc0d62aef0661a5ce845b017", "1f26a01730f67d40ea711a50d9d801bac15a642e", "3b53d24178cfacba53103a44dfd5d072b15a6781", - "fa0d5f2d9419d8310fe4af7a290d13c5026c77de", - "5070ecae5085209508f7450d5f21c69737a178e9", + "c52cfd57f26037723d37192722fc3a217f280c9e", + "e34da6500cf2e424d980714d92737cf6c31a7bda", "d17f4358ae529f920960ed89e535902ee13b0033", "0f44e12ecd1ea7e39433890443626d4fc35204a4", "eb38683e812fd13dca971ba8f4cfd2b6820d3524", @@ -636,6 +636,7 @@ static void draw_graphics(HDC hdc, BITMAPINFO *bmi, BYTE *bits, const char ***sh brush_bits[14] = 0xff; brush_bits[65] = 0xff; brush_bits[69] = 0xff; + brush_bits[72] = 0xff; dib_brush = CreateDIBPatternBrushPt(brush_bi, DIB_RGB_COLORS);

1 0

André Hentschel : winedbg: Increase buffer for function names.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 3ea13c9c49fa738b97b9614884d8cfaea357932b URL: http://source.winehq.org/git/wine.git/?a=commit;h=3ea13c9c49fa738b97b961488… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue May 31 20:17:34 2011 +0200 winedbg: Increase buffer for function names. --- programs/winedbg/info.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/info.c b/programs/winedbg/info.c index 51cf207..bc59eed 100644 --- a/programs/winedbg/info.c +++ b/programs/winedbg/info.c @@ -833,7 +833,7 @@ void info_win32_exception(void) break; case EXCEPTION_WINE_STUB: { - char dll[32], name[64]; + char dll[32], name[256]; memory_get_string(dbg_curr_process, (void*)rec->ExceptionInformation[0], TRUE, FALSE, dll, sizeof(dll));

1 0

Marko Nikolic : comctl32/tests: Use unsigned constants to remove sign comparison warning.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 8c13dcc2e9f20165a27244d212ea4a8933449d98 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8c13dcc2e9f20165a27244d21… Author: Marko Nikolic <grkoma(a)gmail.com> Date: Tue May 31 23:38:46 2011 +0200 comctl32/tests: Use unsigned constants to remove sign comparison warning. --- dlls/comctl32/tests/treeview.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/comctl32/tests/treeview.c b/dlls/comctl32/tests/treeview.c index 5fd26a0..324473d 100644 --- a/dlls/comctl32/tests/treeview.c +++ b/dlls/comctl32/tests/treeview.c @@ -634,7 +634,7 @@ static void test_get_set_bkcolor(void) /* If the value is -1, the control is using the system color for the background color. */ crColor = (COLORREF)SendMessage( hTree, TVM_GETBKCOLOR, 0, 0 ); - ok(crColor == -1, "Default background color reported as 0x%.8x\n", crColor); + ok(crColor == ~0u, "Default background color reported as 0x%.8x\n", crColor); /* Test for black background */ SendMessage( hTree, TVM_SETBKCOLOR, 0, RGB(0,0,0) ); @@ -873,7 +873,7 @@ static void test_get_set_textcolor(void) flush_sequences(sequences, NUM_MSG_SEQUENCES); crColor = (COLORREF)SendMessage( hTree, TVM_GETTEXTCOLOR, 0, 0 ); - ok(crColor == -1, "Default text color reported as 0x%.8x\n", crColor); + ok(crColor == ~0u, "Default text color reported as 0x%.8x\n", crColor); /* Test for black text */ SendMessage( hTree, TVM_SETTEXTCOLOR, 0, RGB(0,0,0) ); @@ -1453,7 +1453,7 @@ static void test_WM_PAINT(void) hTree = create_treeview_control(0); clr = SendMessageA(hTree, TVM_SETBKCOLOR, 0, RGB(255, 0, 0)); - ok(clr == -1, "got %d, expected -1\n", clr); + ok(clr == ~0u, "got %d, expected -1\n", clr); hdc = GetDC(hMainWnd);

1 0

Marko Nikolic : comctl32/tests: Removed sign comparison warning in tooltip tests.
by Alexandre Julliard 01 Jun '11

01 Jun '11

Module: wine Branch: master Commit: 2dd0d8ab2a200e74fca6a32ecec93a1811770e7b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2dd0d8ab2a200e74fca6a32ec… Author: Marko Nikolic <grkoma(a)gmail.com> Date: Tue May 31 23:38:47 2011 +0200 comctl32/tests: Removed sign comparison warning in tooltip tests. --- dlls/comctl32/tests/tooltips.c | 24 ++++++++++++------------ 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/dlls/comctl32/tests/tooltips.c b/dlls/comctl32/tests/tooltips.c index 8ca3f7b..596598f 100644 --- a/dlls/comctl32/tests/tooltips.c +++ b/dlls/comctl32/tests/tooltips.c @@ -148,7 +148,7 @@ static void test_customdraw(void) { {CDRF_NEWFONT, TEST_CDDS_PREPAINT} }; - int iterationNumber; + DWORD iterationNumber; WNDCLASSA wc; LRESULT lResult; @@ -435,7 +435,7 @@ static void test_ttm_gettoolinfo(void) ti.uFlags = 0; ti.uId = 0x1234ABCD; ti.lpszText = NULL; - ti.lParam = 0xdeadbeef; + ti.lParam = 0x1abe11ed; GetClientRect(hwnd, &ti.rect); r = SendMessageA(hwnd, TTM_ADDTOOLA, 0, (LPARAM)&ti); ok(r, "Adding the tool to the tooltip failed\n"); @@ -444,9 +444,9 @@ static void test_ttm_gettoolinfo(void) ti.lParam = 0xaaaaaaaa; r = SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&ti); ok(r, "Getting tooltip info failed\n"); - ok(0xdeadbeef == ti.lParam || - broken(0xdeadbeef != ti.lParam), /* comctl32 < 5.81 */ - "Expected 0xdeadbeef, got %lx\n", ti.lParam); + ok(0x1abe11ed == ti.lParam || + broken(0x1abe11ed != ti.lParam), /* comctl32 < 5.81 */ + "Expected 0x1abe11ed, got %lx\n", ti.lParam); tiW.cbSize = TTTOOLINFOW_V2_SIZE; tiW.hwnd = NULL; @@ -454,22 +454,22 @@ static void test_ttm_gettoolinfo(void) tiW.lParam = 0xaaaaaaaa; r = SendMessageA(hwnd, TTM_GETTOOLINFOW, 0, (LPARAM)&tiW); ok(r, "Getting tooltip info failed\n"); - ok(0xdeadbeef == tiW.lParam || - broken(0xdeadbeef != tiW.lParam), /* comctl32 < 5.81 */ - "Expected 0xdeadbeef, got %lx\n", tiW.lParam); + ok(0x1abe11ed == tiW.lParam || + broken(0x1abe11ed != tiW.lParam), /* comctl32 < 5.81 */ + "Expected 0x1abe11ed, got %lx\n", tiW.lParam); ti.cbSize = TTTOOLINFOA_V2_SIZE; ti.uId = 0x1234ABCD; - ti.lParam = 0xaaaaaaaa; + ti.lParam = 0x55555555; SendMessageA(hwnd, TTM_SETTOOLINFOA, 0, (LPARAM)&ti); ti.cbSize = TTTOOLINFOA_V2_SIZE; ti.lParam = 0xdeadbeef; r = SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&ti); ok(r, "Getting tooltip info failed\n"); - ok(0xaaaaaaaa == ti.lParam || - broken(0xaaaaaaaa != ti.lParam), /* comctl32 < 5.81 */ - "Expected 0xaaaaaaaa, got %lx\n", ti.lParam); + ok(0x55555555 == ti.lParam || + broken(0x55555555 != ti.lParam), /* comctl32 < 5.81 */ + "Expected 0x55555555, got %lx\n", ti.lParam); DestroyWindow(hwnd);

1 0