December 2010 - Wine-commits

Louis Lenders : rpcrt4: Add stub for NdrMesTypeEncode2.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 1cbc5852bea3176cda18976ba7510cf92864f4d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1cbc5852bea3176cda18976ba… Author: Louis Lenders <xerox_xerox2000(a)yahoo.co.uk> Date: Tue Oct 26 22:56:11 2010 +0200 rpcrt4: Add stub for NdrMesTypeEncode2. (cherry picked from commit 92f41c56f7397f1763d90bee41e6ec1c04804595) --- dlls/rpcrt4/ndr_es.c | 6 ++++++ dlls/rpcrt4/rpcrt4.spec | 2 +- 2 files changed, 7 insertions(+), 1 deletions(-) diff --git a/dlls/rpcrt4/ndr_es.c b/dlls/rpcrt4/ndr_es.c index 52200dc..c3456ca 100644 --- a/dlls/rpcrt4/ndr_es.c +++ b/dlls/rpcrt4/ndr_es.c @@ -446,6 +446,12 @@ void RPC_ENTRY NdrMesTypeDecode2(handle_t Handle, const MIDL_TYPE_PICKLING_INFO FIXME("(%p, %p, %p, %p, %p)\n", Handle, pPicklingInfo, pStubDesc, pFormatString, pObject); } +void RPC_ENTRY NdrMesTypeEncode2(handle_t Handle, const MIDL_TYPE_PICKLING_INFO *pPicklingInfo, + const MIDL_STUB_DESC *pStubDesc, PFORMAT_STRING pFormatString, const void *pObject) +{ + FIXME("(%p, %p, %p, %p, %p)\n", Handle, pPicklingInfo, pStubDesc, pFormatString, pObject); +} + void RPC_ENTRY NdrMesTypeFree2(handle_t Handle, const MIDL_TYPE_PICKLING_INFO *pPicklingInfo, const MIDL_STUB_DESC *pStubDesc, PFORMAT_STRING pFormatString, void *pObject) { diff --git a/dlls/rpcrt4/rpcrt4.spec b/dlls/rpcrt4/rpcrt4.spec index 477b3f0..e0c6c11 100644 --- a/dlls/rpcrt4/rpcrt4.spec +++ b/dlls/rpcrt4/rpcrt4.spec @@ -227,7 +227,7 @@ @ stub NdrMesTypeAlignSize @ stdcall NdrMesTypeDecode2(ptr ptr ptr ptr ptr) @ stub NdrMesTypeDecode -@ stub NdrMesTypeEncode2 +@ stdcall NdrMesTypeEncode2(ptr ptr ptr ptr ptr) @ stub NdrMesTypeEncode @ stdcall NdrMesTypeFree2(ptr ptr ptr ptr ptr) @ stdcall NdrNonConformantStringBufferSize(ptr ptr ptr)

1 0

Marcus Meissner : winspool.drv: Check return value of EnumPrintersA.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 61a19597ff3c506014d36a16295b41841275dc53 URL: http://source.winehq.org/git/wine.git/?a=commit;h=61a19597ff3c506014d36a162… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Sun Oct 24 11:09:42 2010 +0100 winspool.drv: Check return value of EnumPrintersA. (cherry picked from commit c7f158824f41f37168222cb09a12c385d98315c3) --- dlls/winspool.drv/info.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/winspool.drv/info.c b/dlls/winspool.drv/info.c index 8c6486b..407bb2e 100644 --- a/dlls/winspool.drv/info.c +++ b/dlls/winspool.drv/info.c @@ -1010,8 +1010,7 @@ void WINSPOOL_LoadSystemPrinters(void) printers AddPrinter takes a while. So we'll tag all printers that were automatically added last time around, if they still exist we'll leave them be otherwise we'll delete them. */ - EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &needed, &num); - if(needed) { + if (EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, NULL, 0, &needed, &num) && needed) { PRINTER_INFO_5A* pi = HeapAlloc(GetProcessHeap(), 0, needed); if(EnumPrintersA(PRINTER_ENUM_LOCAL, NULL, 5, (LPBYTE)pi, needed, &needed, &num)) { for(i = 0; i < num; i++) {

1 0

Vladimir Panteleev : user32: Update last active popup for all parents.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 3f9aed869d04d9773bea54155ef6560e04dc7552 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3f9aed869d04d9773bea54155… Author: Vladimir Panteleev <vladimir(a)thecybershadow.net> Date: Sat Oct 16 20:22:58 2010 +0300 user32: Update last active popup for all parents. (cherry picked from commit 824c5089d2f0e7257111a65d65e8db94b113c5fa) --- server/window.c | 10 +++++++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/server/window.c b/server/window.c index 5fdfd13..4975bfe 100644 --- a/server/window.c +++ b/server/window.c @@ -569,9 +569,13 @@ int make_window_active( user_handle_t window ) if (!win) return 0; - /* set last active for window and its owner */ - win->last_active = win->handle; - if ((owner = get_user_object( win->owner, USER_WINDOW ))) owner->last_active = win->handle; + /* set last active for window and its owners */ + owner = win; + while (owner) + { + owner->last_active = win->handle; + owner = get_user_object( owner->owner, USER_WINDOW ); + } return 1; }

1 0

Alexandre Julliard : ntdll: Don' t call vectored exception handlers under the critical section.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: ac8680b2ad320f9904b6f72736165697f312fe09 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac8680b2ad320f9904b6f7273… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Nov 10 20:55:07 2010 +0100 ntdll: Don't call vectored exception handlers under the critical section. (cherry picked from commit 11e71ede147491381bba4762163803f540c7eb5b) --- dlls/ntdll/exception.c | 25 ++++++++++++++++++++++--- 1 files changed, 22 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/exception.c b/dlls/ntdll/exception.c index 63fd281..33868c7 100644 --- a/dlls/ntdll/exception.c +++ b/dlls/ntdll/exception.c @@ -44,6 +44,7 @@ typedef struct { struct list entry; PVECTORED_EXCEPTION_HANDLER func; + ULONG count; } VECTORED_HANDLER; static struct list vectored_handlers = LIST_INIT(vectored_handlers); @@ -158,21 +159,37 @@ LONG call_vectored_handlers( EXCEPTION_RECORD *rec, CONTEXT *context ) struct list *ptr; LONG ret = EXCEPTION_CONTINUE_SEARCH; EXCEPTION_POINTERS except_ptrs; + VECTORED_HANDLER *handler, *to_free = NULL; except_ptrs.ExceptionRecord = rec; except_ptrs.ContextRecord = context; RtlEnterCriticalSection( &vectored_handlers_section ); - LIST_FOR_EACH( ptr, &vectored_handlers ) + ptr = list_head( &vectored_handlers ); + while (ptr) { - VECTORED_HANDLER *handler = LIST_ENTRY( ptr, VECTORED_HANDLER, entry ); + handler = LIST_ENTRY( ptr, VECTORED_HANDLER, entry ); + handler->count++; + RtlLeaveCriticalSection( &vectored_handlers_section ); + RtlFreeHeap( GetProcessHeap(), 0, to_free ); + to_free = NULL; + TRACE( "calling handler at %p code=%x flags=%x\n", handler->func, rec->ExceptionCode, rec->ExceptionFlags ); ret = handler->func( &except_ptrs ); TRACE( "handler at %p returned %x\n", handler->func, ret ); + + RtlEnterCriticalSection( &vectored_handlers_section ); + ptr = list_next( &vectored_handlers, ptr ); + if (!--handler->count) /* removed during execution */ + { + list_remove( &handler->entry ); + to_free = handler; + } if (ret == EXCEPTION_CONTINUE_EXECUTION) break; } RtlLeaveCriticalSection( &vectored_handlers_section ); + RtlFreeHeap( GetProcessHeap(), 0, to_free ); return ret; } @@ -214,6 +231,7 @@ PVOID WINAPI RtlAddVectoredExceptionHandler( ULONG first, PVECTORED_EXCEPTION_HA if (handler) { handler->func = func; + handler->count = 1; RtlEnterCriticalSection( &vectored_handlers_section ); if (first) list_add_head( &vectored_handlers, &handler->entry ); else list_add_tail( &vectored_handlers, &handler->entry ); @@ -237,7 +255,8 @@ ULONG WINAPI RtlRemoveVectoredExceptionHandler( PVOID handler ) VECTORED_HANDLER *curr_handler = LIST_ENTRY( ptr, VECTORED_HANDLER, entry ); if (curr_handler == handler) { - list_remove( ptr ); + if (!--curr_handler->count) list_remove( ptr ); + else handler = NULL; /* don't free it yet */ ret = TRUE; break; }

1 0

André Hentschel : krnl386: Mention DOSBox.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 72e5e058d5c6cfbd48039b45288070f7e4267931 URL: http://source.winehq.org/git/wine.git/?a=commit;h=72e5e058d5c6cfbd48039b452… Author: André Hentschel <nerv(a)dawncrow.de> Date: Wed Nov 10 20:02:21 2010 +0100 krnl386: Mention DOSBox. (cherry picked from commit ac816c2a82e44dff205e850743973bf97059bebd) --- dlls/krnl386.exe16/dosexe.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/krnl386.exe16/dosexe.c b/dlls/krnl386.exe16/dosexe.c index c16d0df..876a015 100644 --- a/dlls/krnl386.exe16/dosexe.c +++ b/dlls/krnl386.exe16/dosexe.c @@ -643,7 +643,8 @@ static DWORD WINAPI MZ_DOSVM( LPVOID lpExtra ) if (GetLastError() == ERROR_NOT_SUPPORTED) MESSAGE( "wine: Cannot start DOS application %s\n" - " because vm86 mode is not supported on this platform.\n", + " because vm86 mode is not supported on this platform.\n" + " Try running the application with DOSBox.\n", debugstr_a(env) ); else FIXME( "vm86 mode failed error %u\n", GetLastError() );

1 0

André Hentschel : user32: Don' t paint new button styles instead of crashing.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: a15c44910ecbff3a30f6038793acc1ddf730d0fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=a15c44910ecbff3a30f603879… Author: André Hentschel <nerv(a)dawncrow.de> Date: Wed Nov 3 21:15:42 2010 +0100 user32: Don't paint new button styles instead of crashing. (cherry picked from commit e054fc23100f347ae3ed026d24f183aa85b8c802) --- dlls/user32/button.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/user32/button.c b/dlls/user32/button.c index 54d54dc..463c37f 100644 --- a/dlls/user32/button.c +++ b/dlls/user32/button.c @@ -114,7 +114,7 @@ static void UB_Paint( HWND hwnd, HDC hDC, UINT action ); static void OB_Paint( HWND hwnd, HDC hDC, UINT action ); static void BUTTON_CheckAutoRadioButton( HWND hwnd ); -#define MAX_BTN_TYPE 12 +#define MAX_BTN_TYPE 16 static const WORD maxCheckState[MAX_BTN_TYPE] = { @@ -128,7 +128,7 @@ static const WORD maxCheckState[MAX_BTN_TYPE] = BUTTON_UNCHECKED, /* BS_GROUPBOX */ BUTTON_UNCHECKED, /* BS_USERBUTTON */ BUTTON_CHECKED, /* BS_AUTORADIOBUTTON */ - BUTTON_UNCHECKED, /* Not defined */ + BUTTON_UNCHECKED, /* BS_PUSHBOX */ BUTTON_UNCHECKED /* BS_OWNERDRAW */ }; @@ -146,7 +146,7 @@ static const pfPaint btnPaintFunc[MAX_BTN_TYPE] = GB_Paint, /* BS_GROUPBOX */ UB_Paint, /* BS_USERBUTTON */ CB_Paint, /* BS_AUTORADIOBUTTON */ - NULL, /* Not defined */ + NULL, /* BS_PUSHBOX */ OB_Paint /* BS_OWNERDRAW */ };

1 0

Eryk Wieliczko : comdlg32: GetSaveFileName: Don't append .* extension.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 276b83559f87dbc834c6e0c0ac57a995d48e5c20 URL: http://source.winehq.org/git/wine.git/?a=commit;h=276b83559f87dbc834c6e0c0a… Author: Eryk Wieliczko <ewdevel(a)gmail.com> Date: Mon Nov 1 12:35:43 2010 +0100 comdlg32: GetSaveFileName: Don't append .* extension. (cherry picked from commit bd0fe7d26bafebc2e51ec54d354bf1e41bc3892e) --- dlls/comdlg32/filedlg.c | 18 ++++++++++++------ 1 files changed, 12 insertions(+), 6 deletions(-) diff --git a/dlls/comdlg32/filedlg.c b/dlls/comdlg32/filedlg.c index c689584..6778f17 100644 --- a/dlls/comdlg32/filedlg.c +++ b/dlls/comdlg32/filedlg.c @@ -2256,14 +2256,11 @@ BOOL FILEDLG95_OnOpen(HWND hwnd) /* if no extension is specified with file name, then */ /* attach the extension from file filter or default one */ - WCHAR *filterExt = NULL; + const WCHAR *filterExt = NULL; LPWSTR lpstrFilter = NULL; static const WCHAR szwDot[] = {'.',0}; int PathLength = lstrlenW(lpstrPathAndFile); - /* Attach the dot*/ - lstrcatW(lpstrPathAndFile, szwDot); - /*Get the file extension from file type filter*/ lpstrFilter = (LPWSTR) CBGetItemDataPtr(fodInfos->DlgInfos.hwndFileTypeCB, fodInfos->ofnInfos->nFilterIndex-1); @@ -2272,9 +2269,18 @@ BOOL FILEDLG95_OnOpen(HWND hwnd) filterExt = PathFindExtensionW(lpstrFilter); if ( filterExt && *filterExt ) /* attach the file extension from file type filter*/ - lstrcatW(lpstrPathAndFile, filterExt + 1); + filterExt = filterExt + 1; else if ( fodInfos->defext ) /* attach the default file extension*/ - lstrcatW(lpstrPathAndFile, fodInfos->defext); + filterExt = fodInfos->defext; + + /* If extension is .*, ignore it */ + if (filterExt[0] != '*') + { + /* Attach the dot*/ + lstrcatW(lpstrPathAndFile, szwDot); + /* Attach the extension */ + lstrcatW(lpstrPathAndFile, filterExt ); + } /* In Open dialog: if file does not exist try without extension */ if (!(fodInfos->DlgInfos.dwDlgProp & FODPROP_SAVEDLG) && !PathFileExistsW(lpstrPathAndFile))

1 0

Henri Verbeet : krnl386.exe: Return the default value if the section wasn' t found in GetPrivateProfileString16().
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: 59129ad1cd740b3575c593b6bf0ca5f97f38c0fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=59129ad1cd740b3575c593b6b… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Nov 4 10:37:33 2010 +0100 krnl386.exe: Return the default value if the section wasn't found in GetPrivateProfileString16(). Using GetPrivateProfileStringA() for removing trailing spaces etc. This restores behaviour from before 86c6021c71a563af88c1a8cc23ea489e92c8ca70. (cherry picked from commit c4c091d4b7c67487fbd332e7e606ec272d232f4f) --- dlls/krnl386.exe16/file.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/krnl386.exe16/file.c b/dlls/krnl386.exe16/file.c index 507b28d..0a1098f 100644 --- a/dlls/krnl386.exe16/file.c +++ b/dlls/krnl386.exe16/file.c @@ -555,9 +555,8 @@ INT16 WINAPI GetPrivateProfileString16( LPCSTR section, LPCSTR entry, ret = GetPrivateProfileSectionA( section, data, size, filename ); if (!ret) { - if (len) *buffer = 0; HeapFree( GetProcessHeap(), 0, data ); - return 0; + return GetPrivateProfileStringA( section, entry, def_val, buffer, len, filename ); } if (ret != size - 2) break; /* overflow, try again */

1 0

Marcus Meissner : ntdll: Check and fix PE header protections.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: ab3d4bc27b2ef39f432ff9d73b76820d3bc8c659 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ab3d4bc27b2ef39f432ff9d73… Author: Marcus Meissner <meissner(a)suse.de> Date: Mon Nov 8 03:34:27 2010 +0100 ntdll: Check and fix PE header protections. (cherry picked from commit 2e7dc3578663f052b8ee1d39df4a2ad41ed02300) --- dlls/ntdll/virtual.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/virtual.c b/dlls/ntdll/virtual.c index 1428be0..7e2fb8b 100644 --- a/dlls/ntdll/virtual.c +++ b/dlls/ntdll/virtual.c @@ -1416,10 +1416,13 @@ NTSTATUS virtual_create_builtin_view( void *module ) if (status) return status; + /* The PE header is always read-only, no write, no execute. */ + view->prot[0] = VPROT_COMMITTED | VPROT_READ; + sec = (IMAGE_SECTION_HEADER *)((char *)&nt->OptionalHeader + nt->FileHeader.SizeOfOptionalHeader); for (i = 0; i < nt->FileHeader.NumberOfSections; i++) { - DWORD flags = VPROT_SYSTEM | VPROT_IMAGE | VPROT_COMMITTED; + BYTE flags = VPROT_COMMITTED; if (sec[i].Characteristics & IMAGE_SCN_MEM_EXECUTE) flags |= VPROT_EXEC; if (sec[i].Characteristics & IMAGE_SCN_MEM_READ) flags |= VPROT_READ;

1 0

Jörg Höhle : kernel32/tests: GetVolumeInformationA X: succeeds iff drive's default dir is root.
by Alexandre Julliard 02 Dec '10

02 Dec '10

Module: wine Branch: stable Commit: fd5c8b3e7309a2dc6101a30c008690b31f702be3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd5c8b3e7309a2dc6101a30c0… Author: Jörg Höhle <hoehle(a)users.sourceforge.net> Date: Wed Jul 28 21:23:47 2010 +0200 kernel32/tests: GetVolumeInformationA X: succeeds iff drive's default dir is root. (cherry picked from commit fc872628d9640c1441f8d6e332cccc62b6638cea) --- dlls/kernel32/tests/volume.c | 128 +++++++++++++++++++++++++++++++----------- 1 files changed, 94 insertions(+), 34 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=fd5c8b3e7309a2dc6101a…

1 0