December 2017 - Wine-devel - list.winehq.org

[PATCH 2/2] po: Update Simplified Chinese translation.
by Kim Malmo 21 Dec '17

21 Dec '17

Signed-off-by: Kim Malmo <berencamlost(a)msn.com> --- po/zh_CN.po | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/po/zh_CN.po b/po/zh_CN.po index e1e92d261d..d3baf01035 100644 --- a/po/zh_CN.po +++ b/po/zh_CN.po @@ -10402,7 +10402,7 @@ msgstr "" "/d 创建目录符号链接。\n" "/h 创建硬链接。\n" "/j 创建目录结。\n" -"link_nam 为符号链接的名称。\n" +"link_name 为符号链接的名称。\n" "target 为 link_name 指向的路径。\n" #: cmd.rc:312 -- 2.14.1

2 1

[PATCH 1/3] testbot: Add a history Record when the Engine starts or stops.
by Francois Gouget 21 Dec '17

21 Dec '17

A missing stop record will show that the TestBot Engine crashed (or was stopped abruptly). The gap between the stop and start records will also allow identifying the downtime. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/bin/Engine.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/testbot/bin/Engine.pl b/testbot/bin/Engine.pl index 36a18ade..827879cf 100755 --- a/testbot/bin/Engine.pl +++ b/testbot/bin/Engine.pl @@ -749,6 +749,7 @@ sub main() LogMsg "Capping MaxRevertsWhileRunningVMs to MaxRevertingVMs ($MaxRevertsWhileRunningVMs)\n"; } $MaxVMsWhenIdle = $MaxActiveVMs if (!defined $MaxVMsWhenIdle); + SaveRecord('engine', 'start'); Cleanup(1); # Check for patches that arrived while the server was off. @@ -871,6 +872,7 @@ sub main() } } } + SaveRecord('engine', 'stop'); LogMsg "Normal WineTestBot Engine shutdown\n"; return 0; -- 2.15.1

1 2

[PATCH 1/2] testbot/web: Make the title class a bit more generic.
by Francois Gouget 21 Dec '17

21 Dec '17

Instead of hardcoding the text color it now inherits it from the normal non-link text. This makes it usable on white backgrounds too. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> --- testbot/web/WineTestBot.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/web/WineTestBot.css b/testbot/web/WineTestBot.css index 8c03a914..e58a4d80 100644 --- a/testbot/web/WineTestBot.css +++ b/testbot/web/WineTestBot.css @@ -340,7 +340,7 @@ pre .boterror { color: #e55600; } .canceled { color: black; } -a.title { color:white; text-decoration: none; } +a.title { color:inherit; text-decoration: none; } th.Record { text-align: center; } td.Record { text-align: center; } -- 2.15.1

1 1

wine-2.0.4-rc1 Available
by Michael Stefaniuc 21 Dec '17

21 Dec '17

Hello guys, I have pushed wine-2.0.4-rc1 (with tag) to https://github.com/mstefani/wine-stable http://static-winehq.193b.starter-ca-central-1.openshiftapps.com/stable/ Commit wise, the documentation / man page patches for http ==> https dominate this release. Other than "what's new" I have pulled some small fixes to the Wine programs. The interesting parts from the ANNOUNCE file are: What's new in this release (see below for details): - Various bug fixes - Updates to the National Language Support files - Documentation updates ---------------------------------------------------------------- Bugs fixed in 2.0.4 (total 31): 7369 eISLP -help doesn't display, 'text controller' error message 14530 EM_PASTESPECIAL unimplemented in richedit 26046 comctl32/monthcal tests show some invalid reads under valgrind 28741 Odd coordinate in hit test causes uninitialized value references in MONTHCAL_CalculateDayOfWeek()? 37160 If D3D_DISABLE_9EX is defined, d3d9.h can't be compiled 37321 foobar2000 'Scheduler' plugin crashes on unimplemented function shell32.dll.ShellExec_RunDLLW 40025 Winrar installer fails to start 42256 PhotoFiltre 6 and 7 don't erase the mouse selection when dragging it 42715 Cxbx (Xbox emulator) doesn`t start (wine: Invalid address) 43044 League of Legends needs function msvcp140.dll.?__ExceptionPtrAssign@@YAXPAXPBX(a)Z 43046 Multiple apps need api-ms-win-crt-stdio-l1-1-0.dll.__stdio_common_vfprintf_s (Civilization VI, Mafia 3, SketchBook) 43163 Mixcraft 8 crashes when adding a video track ('IAMTimelineGroup', '{9eed4f00-b8a6-11d2-8023-00c0df10d434}' not implemented, 'qedit.dll') 43290 MyLifeOrganized crashes when mousing over tabs 43293 strange behavior- Eudora scrambles email addresses 43324 Beamng.drive requires msvcr120.dll.vsscanf 43453 Magic Online (Magic The Gathering: Online) Crashes due to race condition in secur32 43475 XM6 TypeG crashes on startup. 43503 FUJITSU Software ATLAS: unimplemented function msvcr80.dll._ismbcl0 called in 32-bit code 43527 Magic Online (Magic The Gathering: Online) Crashes due to LOCALE_SNAN returning empty string 43536 WIC can't encode to GUID_WICPixelFormat32bppPBGRA PNG 43603 clipboard PIXMAP regression after upgrading from 1.8.x to 2.x 43658 Text in Deletion Confirmation MsgBox has wrong background colour 43668 Terris Universal Remote UFB 234 - update database does not work 43707 [GdiPlus] HatchStyle is incorrectly named in Wine. Should be GpHatchStyle 43714 ImageMagick crashes with msvcr120.dll._vfprintf_l 43770 Fidibo installer crashes very early in win7 mode 43774 Chromium-based browser engine (CEFv3) used by several games randomly crashes on shutdown (World of Warships 0.6.x) 43779 In ole32/storage32.c , miss check for the result of HeapAlloc 43877 "Cannot allocate DOS memory" error with 16-bit Windows applications 43912 wine-2.0.3/programs/wineboot/wineboot.c:684]: (style) Suspicious condition 44085 winecfg: Use more secure HTTPS URL in about ---------------------------------------------------------------- Changes since 2.0.3: Akihiro Sagawa (5): msvcrt/tests: Add tests for mbstowcs and wcstombs with empty strings. msvcrt: Fix mbstowcs with empty strings. readme: Update Japanese translation. explorer: Use neutral language for the version resource. regsvr32: Use neutral language for the version resource. Alex Henrie (5): msvcrt: Note that __stdio_common functions are for ucrtbase. ucrtbase: Add __stdio_common_vfprintf_s. shell32: Implement ShellExec_RunDLL. include: Add D3D_DISABLE_9EX checks to d3d9.h. user32: Implement IDI_SHIELD. Alexandre Julliard (10): winex11: If importing an X11 clipboard format fails, try other candidate formats for the target. ntdll: Support loading binaries that start inside the DOS area. ntdll: Ignore the preload start address if it is zero. README: Remove references to removed Wiki pages. readme: Patches should now be sent to wine-devel. configure: Replace AC_HEADER_MAJOR macro to handle glibc 2.25 sys/types.h breakage. wineconsole: Avoid some fatal errors. wineconsole: Remove duplicate line feeds. wordpad: New high resolution cursor generated from SVG. explorer: Don't forward the system tray to the display driver in desktop mode. Alistair Leslie-Hughes (3): ole32: Add pointer check. qedit/tests: Add test for the IAMTimelineGroup interface. qedit: Support IAMTimelineGroup interface in IAMTimelineObj. Andrew Eikum (2): kernel32: Use different default sublangs for Spanish and Chinese. explorer: Don't leak PIDL. Andrey Semakin (4): winemine: Mine random placing fix. winemine: Added Shift+LClick hotkey to uncover multiple cells. winemine: Add WM_RBUTTONUP to cases of TestMines(). winemine: Save board without need to correctly finish the app. André Hentschel (1): ntdll: Bump Win 10 version to 15063. Anton Romanov (3): windowscodecs: PNGEncoder: Return 32bppRGBA pixel format by default. secur32: Fix race between schan_(Encrypt|Decrypt)Message. secur32: Protect SSLRead/Write with cs on OSX. Austin English (30): readme: Document that wine can be run from the build directory. readme: Clarify the wording a bit. announce: Move WineHQ URLs to https. readme: Move WineHQ URLs to https. configure: Move WineHQ URLs to https. loader: Move WineHQ URLs to https. server: Move WineHQ URLs to https. widl: Move WineHQ URLs to https. winegcc: Move WineHQ URLs to https. wrc: Move WineHQ URLs to https. winemaker: Move WineHQ URLs to https. wmc: Move WineHQ URLs to https. winebuild: Move WineHQ URLs to https. winedump: Move WineHQ URLs to https. c2man.pl: Move WineHQ URLs to https. msiexec: Move WineHQ URLs to https. notepad: Move WineHQ URLs to https. regedit: Move WineHQ URLs to https. winemine: Move WineHQ URLs to https. winepath: Move WineHQ URLs to https. wineconsole: Move WineHQ URLs to https. regsvr32: Move WineHQ URLs to https. wineboot: Move WineHQ URLs to https. winecfg: Move WineHQ URLs to https. winefile: Move WineHQ URLs to https. winedbg: Move WineHQ URLs to https. iexplore: Move WineHQ URLs to https. appwiz.cpl: Move WineHQ URLs to https. maintainers: Move WineHQ URLs to https. loader/wine.pl.UTF-8.man.in: Move WineHQ URL to https. Bas Weelinck (1): start: Ignore multiple quoted arguments as title. Daniel Lehman (2): msvcrt: Add __ExceptionPtrAssign. server: Fix copy & paste for number of subkeys. Dmitry Timoshkov (3): user32/tests: Test DialogBoxParam using a dialog template with invalid control class. user32: DialogBoxParam should return -1 when dialog control creation fails. cmd: '$E' in the prompt should be interpreted as an escape code (0x1b). Fabian Maurer (5): shell32: Draw text on message boxes without background. gdiplus: Avoid division by zero in SOFTWARE_GdipDrawThinPath. wineboot: In ProcessRunKeys use correct parentheses in condition. ole32: Don't redefine htole32/htole16. winecfg: Remove processorArchitecture from manifest. Gijs Vermeulen (1): msvcrt: Add _vfprintf_l. Henri Verbeet (1): wordpad: Fall back to the default position if the saved position is off-screen. Hugh Bellamy (1): gdiplus: Define GpHatchStyle. Huw D. M. Davies (6): gdi32: Add a helper to fill rectangles with a given pixel. gdi32: Respect the rop mode in SetPixel. ole32: In the non-aggregated case a request for IUnknown actually returns IOleCache2. riched20: Add support for EM_PASTESPECIAL. winefile: Initialize length before calling GetWindowPlacement(). winedbg: Return after raising an exception. Jacek Caban (1): services: Call RpcMgmtWaitServerListen in RPC_Stop. Lauri Kenttä (5): readme: Fix word order. readme: Update Finnish translation. readme: Update Finnish translation. readme: Use UTF-8 consistently in all translations. readme: Move WineHQ URLs to https in translations. Li Keqing (1): wined3d: Add Intel HD Graphics 5000 to the GPU list. Nikolay Sivov (15): msvcrt: Added _ismbcl0(). kernel32/nls: Added genitive month names for Polish. kernel32/nls: Added genitive month names for Belarusian. kernel32/nls: Added genitive month names for Armenian. kernel32/nls: Added genitive month names for Croatian. kernel32/nls: Added genitive month names for Czech. kernel32/nls: Added Nepali locale data. kernel32/nls: Added Bangla (India) locale data. kernel32/nls: Added LOCALE_SNAN entries. comctl32/monthcal: Make sure set today date is valid before using it (Valgrind). comctl32/monthcal: Initialize day field before setting final hit test result (Valgrind). msxml3/httpreq: Support HEAD request. msvcr120: Added vsscanf() and vswscanf(). readme: Update Russian translation. msi: Avoid uninitialized pointer access on error path (Coverity). Piotr Caban (4): msvcrt: Fix strcpy implementation so it works on overlapping buffers. msvcr120: Add wctype implementation. wine.inf: Add Windows Messaging Subsystem MAPI registry key. vbscript: Don't leak IEnumVARIANT in interp_enumnext. Stefan Dösinger (2): ntdll: Do not queue a completion status if pipe ops fail synchronously. d3dx9/tests: Use float bit patterns in test_D3DXFloat_Array. Vincent Povirk (2): loader/wine.inf: Put 32-bit .ds fakedlls in windows/twain_32. wine.inf: Put 64-bit .ds fakedlls in windows/twain_64. Zebediah Figura (1): d3dcompiler: Fix a buffer size computation. -- Michael Stefaniuc mstefani(a)winehq.org

1 0

[PATCH] quartz: Wait for the PullPin thread to terminate.
by Zebediah Figura 21 Dec '17

21 Dec '17

Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- Fixes an intermittent failure in the filtergraph tests where the source file was not being freed before IFilterGraph2_Release() returned. dlls/quartz/pin.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/dlls/quartz/pin.c b/dlls/quartz/pin.c index 355d87d..a17d256 100644 --- a/dlls/quartz/pin.c +++ b/dlls/quartz/pin.c @@ -540,13 +540,7 @@ static void PullPin_Thread_Stop(PullPin *This) { TRACE("(%p)->()\n", This); - EnterCriticalSection(This->pin.pCritSec); - { - CloseHandle(This->hThread); - This->hThread = NULL; - SetEvent(This->hEventStateChanged); - } - LeaveCriticalSection(This->pin.pCritSec); + SetEvent(This->hEventStateChanged); IBaseFilter_Release(This->pin.pinInfo.pFilter); @@ -706,6 +700,11 @@ static HRESULT PullPin_StopProcessing(PullPin * This) assert(WaitForSingleObject(This->thread_sleepy, 0) == WAIT_TIMEOUT); ResetEvent(This->hEventStateChanged); SetEvent(This->thread_sleepy); + + WaitForSingleObject(This->hThread, INFINITE); + CloseHandle(This->hThread); + This->hThread = NULL; + return S_OK; } -- 2.7.4

1 0

[PATCH 3/3] server: Add support for querying FileNameInformation on named pipes.
by Jacek Caban 21 Dec '17

21 Dec '17

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/ntdll/tests/pipe.c | 3 --- server/named_pipe.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 53 insertions(+), 5 deletions(-)

1 0

[PATCH 2/3] server: Introduce get_file_info request for server-side NtQueryInformationFile implementation.
by Jacek Caban 21 Dec '17

21 Dec '17

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- I made it more general than needed by the next patch. We will probably need it in such form for calls on files backed by drivers, but it will also be useful for other info classes (eg. FILE_PIPE_INFORMATION and FILE_PIPE_LOCAL_INFORMATION won't need a separated request) as well as other object types for which client-side fd-based implementation may not be right. This series keeps changes to minimum (due to code freeze). dlls/ntdll/file.c | 21 ++++++++++++++++++++- server/change.c | 1 + server/console.c | 1 + server/device.c | 1 + server/fd.c | 18 ++++++++++++++++++ server/file.c | 1 + server/file.h | 3 +++ server/mailslot.c | 3 +++ server/mapping.c | 1 + server/named_pipe.c | 3 +++ server/protocol.def | 8 ++++++++ server/serial.c | 1 + server/sock.c | 2 ++ 13 files changed, 63 insertions(+), 1 deletion(-)

1 0

[PATCH 1/3] ntdll/tests: Add NtQueryInformationFile(FileNameInformation) on named pipes tests.
by Jacek Caban 21 Dec '17

21 Dec '17

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/ntdll/tests/pipe.c | 71 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 71 insertions(+)

1 0

[PATCH] ddraw: Validate the surface descriptor size in GetDisplayMode.
by Alexandre Julliard 21 Dec '17

21 Dec '17

For bug 44180. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/ddraw.c | 27 +++++++++++++-------------- dlls/ddraw/tests/ddraw1.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw2.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw4.c | 17 +++++++++++++++++ dlls/ddraw/tests/ddraw7.c | 17 +++++++++++++++++ 5 files changed, 81 insertions(+), 14 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index d3e491c3e8bb..27be401a2e57 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -1627,17 +1627,14 @@ static HRESULT WINAPI ddraw7_GetDisplayMode(IDirectDraw7 *iface, DDSURFACEDESC2 struct ddraw *ddraw = impl_from_IDirectDraw7(iface); struct wined3d_display_mode mode; HRESULT hr; - DWORD Size; TRACE("iface %p, surface_desc %p.\n", iface, DDSD); - wined3d_mutex_lock(); /* This seems sane */ - if (!DDSD) - { - wined3d_mutex_unlock(); + if (!DDSD || (DDSD->dwSize != sizeof(DDSURFACEDESC) && DDSD->dwSize != sizeof(DDSURFACEDESC2))) return DDERR_INVALIDPARAMS; - } + + wined3d_mutex_lock(); if (FAILED(hr = wined3d_get_adapter_display_mode(ddraw->wined3d, WINED3DADAPTER_DEFAULT, &mode, NULL))) { @@ -1646,10 +1643,8 @@ static HRESULT WINAPI ddraw7_GetDisplayMode(IDirectDraw7 *iface, DDSURFACEDESC2 return hr; } - Size = DDSD->dwSize; - memset(DDSD, 0, Size); - - DDSD->dwSize = Size; + memset(DDSD, 0, DDSD->dwSize); + DDSD->dwSize = sizeof(*DDSD); DDSD->dwFlags |= DDSD_HEIGHT | DDSD_WIDTH | DDSD_PIXELFORMAT | DDSD_PITCH | DDSD_REFRESHRATE; DDSD->dwWidth = mode.width; DDSD->dwHeight = mode.height; @@ -1682,21 +1677,25 @@ static HRESULT WINAPI ddraw4_GetDisplayMode(IDirectDraw4 *iface, DDSURFACEDESC2 static HRESULT WINAPI ddraw2_GetDisplayMode(IDirectDraw2 *iface, DDSURFACEDESC *surface_desc) { struct ddraw *ddraw = impl_from_IDirectDraw2(iface); + HRESULT hr; TRACE("iface %p, surface_desc %p.\n", iface, surface_desc); - /* FIXME: Test sizes, properly convert surface_desc */ - return ddraw7_GetDisplayMode(&ddraw->IDirectDraw7_iface, (DDSURFACEDESC2 *)surface_desc); + hr = ddraw7_GetDisplayMode(&ddraw->IDirectDraw7_iface, (DDSURFACEDESC2 *)surface_desc); + if (SUCCEEDED(hr)) surface_desc->dwSize = sizeof(*surface_desc); + return hr; } static HRESULT WINAPI ddraw1_GetDisplayMode(IDirectDraw *iface, DDSURFACEDESC *surface_desc) { struct ddraw *ddraw = impl_from_IDirectDraw(iface); + HRESULT hr; TRACE("iface %p, surface_desc %p.\n", iface, surface_desc); - /* FIXME: Test sizes, properly convert surface_desc */ - return ddraw7_GetDisplayMode(&ddraw->IDirectDraw7_iface, (DDSURFACEDESC2 *)surface_desc); + hr = ddraw7_GetDisplayMode(&ddraw->IDirectDraw7_iface, (DDSURFACEDESC2 *)surface_desc); + if (SUCCEEDED(hr)) surface_desc->dwSize = sizeof(*surface_desc); + return hr; } /***************************************************************************** diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index c306c4db1a32..b8ac74919f89 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -10132,6 +10132,23 @@ static void test_surface_desc_size(void) IDirectDrawSurface_Release(surface); } + /* GetDisplayMode() */ + for (j = 0; j < sizeof(desc_sizes) / sizeof(*desc_sizes); ++j) + { + memset(&desc, 0xcc, sizeof(desc)); + desc.dwSize = desc_sizes[j]; + expected_hr = (desc.dwSize == sizeof(DDSURFACEDESC) || desc.dwSize == sizeof(DDSURFACEDESC2)) + ? DD_OK : DDERR_INVALIDPARAMS; + hr = IDirectDraw_GetDisplayMode(ddraw, &desc.desc1); + ok(hr == expected_hr, "Got hr %#x, expected %#x, size %u.\n", hr, expected_hr, desc_sizes[j]); + if (SUCCEEDED(hr)) + { + ok(desc.dwSize == sizeof(DDSURFACEDESC), "Wrong size %u for %u.\n", desc.dwSize, desc_sizes[j]); + ok(desc.blob[desc_sizes[j]] == 0xcc, "Overflow for size %u.\n", desc_sizes[j]); + ok(desc.blob[desc_sizes[j] - 1] != 0xcc, "Struct not cleared for size %u.\n", desc_sizes[j]); + } + } + refcount = IDirectDraw7_Release(ddraw); ok(!refcount, "DirectDraw has %u references left.\n", refcount); } diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index 8a7801ee3e57..716103335f1b 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -11643,6 +11643,23 @@ static void test_surface_desc_size(void) IDirectDrawSurface_Release(surface); } + /* GetDisplayMode() */ + for (j = 0; j < sizeof(desc_sizes) / sizeof(*desc_sizes); ++j) + { + memset(&desc, 0xcc, sizeof(desc)); + desc.dwSize = desc_sizes[j]; + expected_hr = (desc.dwSize == sizeof(DDSURFACEDESC) || desc.dwSize == sizeof(DDSURFACEDESC2)) + ? DD_OK : DDERR_INVALIDPARAMS; + hr = IDirectDraw2_GetDisplayMode(ddraw, &desc.desc1); + ok(hr == expected_hr, "Got hr %#x, expected %#x, size %u.\n", hr, expected_hr, desc_sizes[j]); + if (SUCCEEDED(hr)) + { + ok(desc.dwSize == sizeof(DDSURFACEDESC), "Wrong size %u for %u.\n", desc.dwSize, desc_sizes[j]); + ok(desc.blob[desc_sizes[j]] == 0xcc, "Overflow for size %u.\n", desc_sizes[j]); + ok(desc.blob[desc_sizes[j] - 1] != 0xcc, "Struct not cleared for size %u.\n", desc_sizes[j]); + } + } + refcount = IDirectDraw2_Release(ddraw); ok(!refcount, "DirectDraw has %u references left.\n", refcount); } diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 048331dfc39d..46362e5c1dc0 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -13247,6 +13247,23 @@ static void test_surface_desc_size(void) IDirectDrawSurface_Release(surface); } + /* GetDisplayMode() */ + for (j = 0; j < sizeof(desc_sizes) / sizeof(*desc_sizes); ++j) + { + memset(&desc, 0xcc, sizeof(desc)); + desc.dwSize = desc_sizes[j]; + expected_hr = (desc.dwSize == sizeof(DDSURFACEDESC) || desc.dwSize == sizeof(DDSURFACEDESC2)) + ? DD_OK : DDERR_INVALIDPARAMS; + hr = IDirectDraw4_GetDisplayMode(ddraw, &desc.desc2); + ok(hr == expected_hr, "Got hr %#x, expected %#x, size %u.\n", hr, expected_hr, desc_sizes[j]); + if (SUCCEEDED(hr)) + { + ok(desc.dwSize == sizeof(DDSURFACEDESC2), "Wrong size %u for %u.\n", desc.dwSize, desc_sizes[j]); + ok(desc.blob[desc_sizes[j]] == 0xcc, "Overflow for size %u.\n", desc_sizes[j]); + ok(desc.blob[desc_sizes[j] - 1] != 0xcc, "Struct not cleared for size %u.\n", desc_sizes[j]); + } + } + refcount = IDirectDraw4_Release(ddraw); ok(!refcount, "DirectDraw has %u references left.\n", refcount); } diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index d1ff85d5d54f..c7a2b138e97b 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -12547,6 +12547,23 @@ static void test_surface_desc_size(void) IDirectDrawSurface_Release(surface); } + /* GetDisplayMode() */ + for (j = 0; j < sizeof(desc_sizes) / sizeof(*desc_sizes); ++j) + { + memset(&desc, 0xcc, sizeof(desc)); + desc.dwSize = desc_sizes[j]; + expected_hr = (desc.dwSize == sizeof(DDSURFACEDESC) || desc.dwSize == sizeof(DDSURFACEDESC2)) + ? DD_OK : DDERR_INVALIDPARAMS; + hr = IDirectDraw7_GetDisplayMode(ddraw, &desc.desc2); + ok(hr == expected_hr, "Got hr %#x, expected %#x, size %u.\n", hr, expected_hr, desc_sizes[j]); + if (SUCCEEDED(hr)) + { + ok(desc.dwSize == sizeof(DDSURFACEDESC2), "Wrong size %u for %u.\n", desc.dwSize, desc_sizes[j]); + ok(desc.blob[desc_sizes[j]] == 0xcc, "Overflow for size %u.\n", desc_sizes[j]); + ok(desc.blob[desc_sizes[j] - 1] != 0xcc, "Struct not cleared for size %u.\n", desc_sizes[j]); + } + } + refcount = IDirectDraw7_Release(ddraw); ok(!refcount, "DirectDraw has %u references left.\n", refcount); } -- 2.15.0 -- Alexandre Julliard julliard(a)winehq.org

3 2

[PATCH] wined3d: Consider CPU blitter when clearing discarded textures.
by Józef Kucia 21 Dec '17

21 Dec '17

In order to avoid invalidating system memory for "converted" surfaces. Fixes a regression introduced by commit 56128d18d4df ("wined3d: Drop the special case for "converted" surfaces in wined3d_surface_blt()."). Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> --- dlls/ddraw/tests/ddraw4.c | 9 ++++----- dlls/ddraw/tests/ddraw7.c | 9 ++++----- dlls/wined3d/surface.c | 4 +++- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 048331dfc39d..fb42f85b0048 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -9960,11 +9960,10 @@ static void test_color_fill(void) hr = IDirectDrawSurface4_Lock(surface, NULL, &surface_desc, DDLOCK_READONLY, 0); ok(SUCCEEDED(hr), "Failed to lock surface, hr %#x, surface %s.\n", hr, tests[i].name); color = surface_desc.lpSurface; - todo_wine_if(tests[i].caps & DDSCAPS_VIDEOMEMORY && U3(z_fmt).dwZBitMask != 0xffff) - ok((*color & U3(z_fmt).dwZBitMask) == (tests[i].result & U3(z_fmt).dwZBitMask) - || broken((*color & U3(z_fmt).dwZBitMask) == (expected_broken & U3(z_fmt).dwZBitMask)), - "Got clear result 0x%08x, expected 0x%08x, surface %s.\n", - *color & U3(z_fmt).dwZBitMask, tests[i].result & U3(z_fmt).dwZBitMask, tests[i].name); + ok((*color & U3(z_fmt).dwZBitMask) == (tests[i].result & U3(z_fmt).dwZBitMask) + || broken((*color & U3(z_fmt).dwZBitMask) == (expected_broken & U3(z_fmt).dwZBitMask)), + "Got clear result 0x%08x, expected 0x%08x, surface %s.\n", + *color & U3(z_fmt).dwZBitMask, tests[i].result & U3(z_fmt).dwZBitMask, tests[i].name); hr = IDirectDrawSurface4_Unlock(surface, NULL); ok(SUCCEEDED(hr), "Failed to unlock surface, hr %#x, surface %s.\n", hr, tests[i].name); } diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index d1ff85d5d54f..2fb845378c42 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -9911,11 +9911,10 @@ static void test_color_fill(void) hr = IDirectDrawSurface7_Lock(surface, NULL, &surface_desc, DDLOCK_READONLY, 0); ok(SUCCEEDED(hr), "Failed to lock surface, hr %#x, surface %s.\n", hr, tests[i].name); color = surface_desc.lpSurface; - todo_wine_if(tests[i].caps & DDSCAPS_VIDEOMEMORY && U3(z_fmt).dwZBitMask != 0xffff) - ok((*color & U3(z_fmt).dwZBitMask) == (tests[i].result & U3(z_fmt).dwZBitMask) - || broken((*color & U3(z_fmt).dwZBitMask) == (expected_broken & U3(z_fmt).dwZBitMask)), - "Got clear result 0x%08x, expected 0x%08x, surface %s.\n", - *color & U3(z_fmt).dwZBitMask, tests[i].result & U3(z_fmt).dwZBitMask, tests[i].name); + ok((*color & U3(z_fmt).dwZBitMask) == (tests[i].result & U3(z_fmt).dwZBitMask) + || broken((*color & U3(z_fmt).dwZBitMask) == (expected_broken & U3(z_fmt).dwZBitMask)), + "Got clear result 0x%08x, expected 0x%08x, surface %s.\n", + *color & U3(z_fmt).dwZBitMask, tests[i].result & U3(z_fmt).dwZBitMask, tests[i].name); hr = IDirectDrawSurface7_Unlock(surface, NULL); ok(SUCCEEDED(hr), "Failed to unlock surface, hr %#x, surface %s.\n", hr, tests[i].name); } diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 02e0c70382d8..da22c7ad9aaf 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2698,13 +2698,15 @@ static BOOL ffp_blitter_use_cpu_clear(struct wined3d_rendertarget_view *view) { struct wined3d_resource *resource; struct wined3d_texture *texture; + DWORD locations; resource = view->resource; if (resource->type == WINED3D_RTYPE_BUFFER) return resource->pool == WINED3D_POOL_SYSTEM_MEM; texture = texture_from_resource(resource); - if (texture->sub_resources[view->sub_resource_idx].locations & resource->map_binding) + locations = texture->sub_resources[view->sub_resource_idx].locations; + if (locations & (resource->map_binding | WINED3D_LOCATION_DISCARDED)) return resource->pool == WINED3D_POOL_SYSTEM_MEM || (texture->flags & WINED3D_TEXTURE_PIN_SYSMEM); return resource->pool == WINED3D_POOL_SYSTEM_MEM && !(texture->flags & WINED3D_TEXTURE_CONVERTED); -- 2.13.6

2 1