January 2010 - Wine-commits

Paul Vriens : mshtml/tests: Fix a test failure on NT4.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 157b1f69027275e2b0f191463d093e65ba3cec93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=157b1f69027275e2b0f191463… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Tue Jan 5 11:12:39 2010 +0100 mshtml/tests: Fix a test failure on NT4. --- dlls/mshtml/tests/dom.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 8744129..5c7380e 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -2130,7 +2130,8 @@ static void _test_elem_filters(unsigned line, IUnknown *unk) IHTMLFiltersCollection *filters; hres = IHTMLElement_get_filters(elem, &filters); - ok_(__FILE__,line) (hres == S_OK, "get_filters failed: %08x\n", hres); + ok_(__FILE__,line) (hres == S_OK || broken(hres == REGDB_E_CLASSNOTREG) /* NT4 */, + "get_filters failed: %08x\n", hres); if(hres == S_OK) { LONG len; @@ -2141,7 +2142,8 @@ static void _test_elem_filters(unsigned line, IUnknown *unk) ok_(__FILE__,line) (len == 0, "expect 0 got %d\n", len); hres = IHTMLFiltersCollection_QueryInterface(filters, &IID_IDispatchEx, (void**)&dispex); - ok(hres == S_OK || broken(hres == E_NOINTERFACE), "Could not get IDispatchEx interface: %08x\n", hres); + ok_(__FILE__,line) (hres == S_OK || broken(hres == E_NOINTERFACE), + "Could not get IDispatchEx interface: %08x\n", hres); if(SUCCEEDED(hres)) { test_disp((IUnknown*)filters, &IID_IHTMLFiltersCollection, "[object]"); IDispatchEx_Release(dispex);

1 0

Alistair Leslie-Hughes : gdiplus: Implement GdipIsVisibleClipEmpty.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 03e31f9b86dc645574ca2ee21038d04997093f54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=03e31f9b86dc645574ca2ee21… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Dec 23 13:53:27 2009 +1100 gdiplus: Implement GdipIsVisibleClipEmpty. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphics.c | 23 +++++++++++++++++++++++ 2 files changed, 24 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index aa64a3c..7d6081d 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -424,7 +424,7 @@ @ stdcall GdipIsOutlineVisiblePathPoint(ptr long long ptr ptr ptr) @ stdcall GdipIsOutlineVisiblePathPointI(ptr long long ptr ptr ptr) @ stdcall GdipIsStyleAvailable(ptr long ptr) -@ stub GdipIsVisibleClipEmpty +@ stdcall GdipIsVisibleClipEmpty(ptr ptr) @ stdcall GdipIsVisiblePathPoint(ptr long long ptr ptr) @ stdcall GdipIsVisiblePathPointI(ptr long long ptr ptr) @ stdcall GdipIsVisiblePoint(ptr long long ptr) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 67b0ff9..39ab18c 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -4359,3 +4359,26 @@ GpStatus WINGDIPAPI GdipRecordMetafileI(HDC hdc, EmfType type, GDIPCONST GpRect FIXME("(%p %d %p %d %p %p): stub\n", hdc, type, frameRect, frameUnit, desc, metafile); return NotImplemented; } + +/***************************************************************************** + * GdipIsVisibleClipEmpty [GDIPLUS.@] + */ +GpStatus WINGDIPAPI GdipIsVisibleClipEmpty(GpGraphics *graphics, BOOL *res) +{ + GpStatus stat; + GpRegion* rgn; + + TRACE("(%p, %p)\n", graphics, res); + + if((stat = GdipCreateRegion(&rgn)) != Ok) + return stat; + + if((stat = get_visible_clip_region(graphics, rgn)) != Ok) + goto cleanup; + + stat = GdipIsEmptyRegion(rgn, graphics, res); + +cleanup: + GdipDeleteRegion(rgn); + return stat; +}

1 0

Alexandre Julliard : winedos: Merge all of winedos back into krnl386.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 26a42f845284bdf642b863597dcb1a90d17b3e46 URL: http://source.winehq.org/git/wine.git/?a=commit;h=26a42f845284bdf642b863597… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 5 14:26:05 2010 +0100 winedos: Merge all of winedos back into krnl386. --- configure | 10 - configure.ac | 2 - dlls/ifsmgr.vxd/Makefile.in | 2 +- dlls/ifsmgr.vxd/ifsmgr.c | 24 +- dlls/krnl386.exe16/Makefile.in | 25 + dlls/{winedos => krnl386.exe16}/dma.c | 0 dlls/{winedos => krnl386.exe16}/dosaspi.c | 0 dlls/{winedos => krnl386.exe16}/dosconf.c | 0 dlls/{winedos/devices.c => krnl386.exe16/dosdev.c} | 0 dlls/{winedos/module.c => krnl386.exe16/dosexe.c} | 18 +- dlls/{winedos => krnl386.exe16}/dosexe.h | 19 +- dlls/krnl386.exe16/dosmem.c | 501 ++++++++++++++++-- dlls/{winedos => krnl386.exe16}/dosvm.c | 23 +- dlls/{winedos => krnl386.exe16}/fpu.c | 0 dlls/krnl386.exe16/global.c | 18 - dlls/krnl386.exe16/instr.c | 33 +- dlls/{winedos => krnl386.exe16}/int09.c | 0 dlls/{winedos => krnl386.exe16}/int10.c | 2 +- dlls/{winedos => krnl386.exe16}/int13.c | 0 dlls/{winedos => krnl386.exe16}/int15.c | 0 dlls/{winedos => krnl386.exe16}/int16.c | 0 dlls/{winedos => krnl386.exe16}/int21.c | 1 + dlls/{winedos => krnl386.exe16}/int25.c | 0 dlls/{winedos => krnl386.exe16}/int26.c | 0 dlls/{winedos => krnl386.exe16}/int2f.c | 52 -- dlls/{winedos => krnl386.exe16}/int31.c | 13 +- dlls/{winedos => krnl386.exe16}/int33.c | 0 dlls/{winedos => krnl386.exe16}/int67.c | 0 dlls/{winedos => krnl386.exe16}/interrupts.c | 19 +- dlls/{winedos => krnl386.exe16}/ioports.c | 9 +- dlls/krnl386.exe16/kernel16_private.h | 59 +-- dlls/krnl386.exe16/krnl386.exe16.spec | 4 + dlls/krnl386.exe16/ne_module.c | 1 + dlls/krnl386.exe16/relay.c | 176 ++++++- dlls/{winedos => krnl386.exe16}/soundblaster.c | 0 dlls/{winedos => krnl386.exe16}/timer.c | 0 dlls/{winedos => krnl386.exe16}/vga.c | 0 dlls/{winedos => krnl386.exe16}/vga.h | 0 dlls/krnl386.exe16/vxd.c | 1 + dlls/vwin32.vxd/Makefile.in | 2 +- dlls/vwin32.vxd/vwin32.c | 35 +-- dlls/winedos/Makefile.in | 39 -- dlls/winedos/dosmem.c | 551 -------------------- dlls/winedos/relay.c | 204 -------- dlls/winedos/winedos.spec | 13 - dlls/winedos/wprocs.spec | 19 - programs/winevdm/winevdm.c | 14 +- 47 files changed, 736 insertions(+), 1153 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=26a42f845284bdf642b86…

1 0

Alexandre Julliard : kernel: Stop exporting some 16-bit functions that are no longer used outside of kernel .
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 5d8861b0844a2e569f23a853054c9c91706b3e15 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5d8861b0844a2e569f23a8530… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 5 16:28:43 2010 +0100 kernel: Stop exporting some 16-bit functions that are no longer used outside of kernel. --- dlls/krnl386.exe16/krnl386.exe16.spec | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/dlls/krnl386.exe16/krnl386.exe16.spec b/dlls/krnl386.exe16/krnl386.exe16.spec index 13e7091..f3dbbea 100644 --- a/dlls/krnl386.exe16/krnl386.exe16.spec +++ b/dlls/krnl386.exe16/krnl386.exe16.spec @@ -696,9 +696,7 @@ ################################################################ # 16-bit symbols not available from kernel32 but used by other 16-bit dlls # -@ stdcall -arch=win32 AllocCStoDSAlias16(long) @ stdcall -arch=win32 AllocSelectorArray16(long) -@ stdcall -arch=win32 ConvertDialog32To16(ptr long ptr) @ stdcall -arch=win32 FarGetOwner16(long) @ stdcall -arch=win32 FarSetOwner16(long long) @ stdcall -arch=win32 FindResource16(long str str) @@ -710,10 +708,6 @@ @ stdcall -arch=win32 GetExePtr(long) @ stdcall -arch=win32 GetExpWinVer16(long) @ stdcall -arch=win32 GetModuleHandle16(str) -@ stdcall -arch=win32 GetSelectorLimit16(long) -@ stdcall -arch=win32 GlobalDOSAlloc16(long) -@ stdcall -arch=win32 GlobalDOSFree16(long) -@ stdcall -arch=win32 GlobalFlags16(long) @ stdcall -arch=win32 GlobalReAlloc16(long long long) @ stdcall -arch=win32 InitTask16(ptr) @ stdcall -arch=win32 IsBadReadPtr16(long long) @@ -731,7 +725,6 @@ @ stdcall -arch=win32 LocalCountFree16() @ stdcall -arch=win32 LocalHeapSize16() @ stdcall -arch=win32 LockResource16(long) -@ stdcall -arch=win32 SelectorAccessRights16(long long long) @ stdcall -arch=win32 SetSelectorBase(long long) @ stdcall -arch=win32 SetSelectorLimit16(long long) @ stdcall -arch=win32 SizeofResource16(long long)

1 0

Hans Leidekker : wininet: Fix a memory leak.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: e5180c101bf760e4ab6e5775767572bfe34df5fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5180c101bf760e4ab6e57757… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Jan 5 14:47:37 2010 +0100 wininet: Fix a memory leak. Found by Valgrind. --- dlls/wininet/http.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 70f5f6a..5b1bb22 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -3592,6 +3592,7 @@ static DWORD HTTP_HttpSendRequestW(http_request_t *lpwhr, LPCWSTR lpszHeaders, lpwhr->lpHttpSession->lpszPassword, Host->lpszValue)) { + HeapFree(GetProcessHeap(), 0, requestString); loop_next = TRUE; break; }

1 0

Detlef Riekenberg : user32: Move a comment about HLOCAL16 to user.exe.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 2f529c39fda88a9c5803bd5ff3bf9c63ad0247ac URL: http://source.winehq.org/git/wine.git/?a=commit;h=2f529c39fda88a9c5803bd5ff… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Tue Jan 5 15:58:01 2010 +0100 user32: Move a comment about HLOCAL16 to user.exe. --- dlls/user.exe16/message.c | 11 +++++++++++ dlls/user32/edit.c | 14 +------------- 2 files changed, 12 insertions(+), 13 deletions(-) diff --git a/dlls/user.exe16/message.c b/dlls/user.exe16/message.c index 011e5c4..1ff58a7 100644 --- a/dlls/user.exe16/message.c +++ b/dlls/user.exe16/message.c @@ -2094,6 +2094,17 @@ static LRESULT combo_proc16( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, return wow_handlers32.combo_proc( hwnd, msg, wParam, lParam, FALSE ); } +/********************************************************************* + * edit_lock_buffer (internal) + * + * A 16 bit application might send an EM_GETHANDLE message and expect a HLOCAL16 + * (16 bit SEG:OFF handler). From that moment on we have to keep using this + * 16 bit memory handler, because it is supposed to be valid at all times after + * EM_GETHANDLE. + * We create a HLOCAL16 buffer in edit_get_handle and copy the text from the + * HLOCAL buffer, when needed + * + */ #define GWW_HANDLE16 sizeof(void*) diff --git a/dlls/user32/edit.c b/dlls/user32/edit.c index d7ab5b4..20a853e 100644 --- a/dlls/user32/edit.c +++ b/dlls/user32/edit.c @@ -1088,19 +1088,7 @@ static inline void text_buffer_changed(EDITSTATE *es) } /********************************************************************* - * - * EDIT_LockBuffer - * - * This acts as a LocalLock16(), but it locks only once. This way - * you can call it whenever you like, without unlocking. - * - * Initially the edit control allocates a HLOCAL32 buffer - * (32 bit linear memory handler). However, 16 bit application - * might send an EM_GETHANDLE message and expect a HLOCAL16 (16 bit SEG:OFF - * handler). From that moment on we have to keep using this 16 bit memory - * handler, because it is supposed to be valid at all times after EM_GETHANDLE. - * What we do is create a HLOCAL16 buffer, copy the text, and do pointer - * conversion. + * EDIT_LockBuffer * */ static void EDIT_LockBuffer(EDITSTATE *es)

1 0

Henri Verbeet : wined3d: Handle stateblock capture for default lights created while recording.
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: f5c8153eddecd709bbcb15fd1bae110a16bf0029 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5c8153eddecd709bbcb15fd1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 4 21:33:11 2010 +0100 wined3d: Handle stateblock capture for default lights created while recording. For non-recorded stateblocks a light always exists in the device stateblock as well, since that's where it's copied from. However, when SetLightEnable() is called for a light that didn't a exist yet while recording a stateblock, the light is only created on the recorded stateblock. --- dlls/wined3d/stateblock.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/stateblock.c b/dlls/wined3d/stateblock.c index 03c3e73..48f9eac 100644 --- a/dlls/wined3d/stateblock.c +++ b/dlls/wined3d/stateblock.c @@ -573,8 +573,17 @@ static void record_lights(IWineD3DStateBlockImpl *This, const IWineD3DStateBlock if (!updated) { - ERR("Light %u in stateblock %p does not exist in device stateblock %p.\n", + /* This can happen if the light was originally created as a + * default light for SetLightEnable() while recording. */ + WARN("Light %u in stateblock %p does not exist in device stateblock %p.\n", src->OriginalIndex, This, targetStateBlock); + + src->OriginalParms = WINED3D_default_light; + if (src->glIndex != -1) + { + This->activeLights[src->glIndex] = NULL; + src->glIndex = -1; + } } } }

1 0

Henri Verbeet : avifil32: Properly check the required buffer size in AVIFILE_ReadBlock().
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 4c36e12be6fdc1afc6431df3f930715f587ef757 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4c36e12be6fdc1afc6431df3f… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 4 21:33:12 2010 +0100 avifil32: Properly check the required buffer size in AVIFILE_ReadBlock(). --- dlls/avifil32/avifile.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index 49e141f..2486611 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -2027,7 +2027,7 @@ static HRESULT AVIFILE_ReadBlock(IAVIStreamImpl *This, DWORD pos, size += 2 * sizeof(DWORD); /* check that buffer is big enough -- don't trust dwSuggestedBufferSize */ - if (This->lpBuffer == NULL || size < This->cbBuffer) { + if (This->lpBuffer == NULL || This->cbBuffer < size) { DWORD maxSize = max(size, This->sInfo.dwSuggestedBufferSize); if (This->lpBuffer == NULL)

1 0

Henri Verbeet : avifil32: Just assign maxSize to This-> cbBuffer in AVIFILE_ReadBlock().
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: 1fb72a37eb350993a4335cfc790be75f5bce1b72 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1fb72a37eb350993a4335cfc7… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 4 21:33:13 2010 +0100 avifil32: Just assign maxSize to This->cbBuffer in AVIFILE_ReadBlock(). --- dlls/avifil32/avifile.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index 2486611..2d39659 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -2036,7 +2036,7 @@ static HRESULT AVIFILE_ReadBlock(IAVIStreamImpl *This, DWORD pos, This->lpBuffer = HeapReAlloc(GetProcessHeap(), 0, This->lpBuffer, maxSize); if (This->lpBuffer == NULL) return AVIERR_MEMORY; - This->cbBuffer = max(size, This->sInfo.dwSuggestedBufferSize); + This->cbBuffer = maxSize; } /* now read the complete chunk into our buffer */

1 0

Henri Verbeet : avifil32: Don't leak the buffer on HeapReAlloc() failure in AVIFILE_ReadBlock().
by Alexandre Julliard 05 Jan '10

05 Jan '10

Module: wine Branch: master Commit: c274d6f06e28784857903f2c4599db928f50c42d URL: http://source.winehq.org/git/wine.git/?a=commit;h=c274d6f06e28784857903f2c4… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jan 4 21:33:14 2010 +0100 avifil32: Don't leak the buffer on HeapReAlloc() failure in AVIFILE_ReadBlock(). --- dlls/avifil32/avifile.c | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index 2d39659..c4f191a 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -2030,12 +2030,14 @@ static HRESULT AVIFILE_ReadBlock(IAVIStreamImpl *This, DWORD pos, if (This->lpBuffer == NULL || This->cbBuffer < size) { DWORD maxSize = max(size, This->sInfo.dwSuggestedBufferSize); - if (This->lpBuffer == NULL) + if (This->lpBuffer == NULL) { This->lpBuffer = HeapAlloc(GetProcessHeap(), 0, maxSize); - else - This->lpBuffer = HeapReAlloc(GetProcessHeap(), 0, This->lpBuffer, maxSize); - if (This->lpBuffer == NULL) - return AVIERR_MEMORY; + if (!This->lpBuffer) return AVIERR_MEMORY; + } else { + void *new_buffer = HeapReAlloc(GetProcessHeap(), 0, This->lpBuffer, maxSize); + if (!new_buffer) return AVIERR_MEMORY; + This->lpBuffer = new_buffer; + } This->cbBuffer = maxSize; }

1 0