wine-commits September 2018

wine-commits@winehq.org

2 participants
670 discussions

Nikolay Sivov : crypt32/base64: Fix certificate request header and trailer in CryptBinaryToStringW() output.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: fb53f6e79d9c1948af206f58ba2f9a294d23c8d7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=fb53f6e79d9c1948af206f58… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 25 16:55:36 2018 +0300 crypt32/base64: Fix certificate request header and trailer in CryptBinaryToStringW() output. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/crypt32/base64.c | 4 +- dlls/crypt32/tests/base64.c | 226 ++++++++++++++++++++++++++++++-------------- 2 files changed, 159 insertions(+), 71 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=fb53f6e79d9c1948af20…

1 0

Alexandre Julliard : kernel32: Remove the DOS/Win16/ OS2 binary distinction.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 1060567cc592aaf7aa453d43a0a48bf7e9ac87f3 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1060567cc592aaf7aa453d43… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 25 19:23:00 2018 +0200 kernel32: Remove the DOS/Win16/OS2 binary distinction. They all need to be run through winevdm. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/kernel_private.h | 2 -- dlls/kernel32/module.c | 72 +----------------------------------------- dlls/kernel32/process.c | 8 ++--- 3 files changed, 3 insertions(+), 79 deletions(-) diff --git a/dlls/kernel32/kernel_private.h b/dlls/kernel32/kernel_private.h index 9073e18..b465d17 100644 --- a/dlls/kernel32/kernel_private.h +++ b/dlls/kernel32/kernel_private.h @@ -69,8 +69,6 @@ enum binary_type BINARY_UNKNOWN = 0, BINARY_PE, BINARY_WIN16, - BINARY_OS216, - BINARY_DOS, BINARY_UNIX_EXE, BINARY_UNIX_LIB }; diff --git a/dlls/kernel32/module.c b/dlls/kernel32/module.c index 31e3014..f03b498 100644 --- a/dlls/kernel32/module.c +++ b/dlls/kernel32/module.c @@ -259,54 +259,6 @@ BOOL WINAPI DisableThreadLibraryCalls( HMODULE hModule ) } -/* Check whether a file is an OS/2 or a very old Windows executable - * by testing on import of KERNEL. - * - * Reading the module imports is the only reasonable way of discerning - * old Windows binaries from OS/2 ones. - */ -static DWORD MODULE_Decide_OS2_OldWin(HANDLE hfile, const IMAGE_DOS_HEADER *mz, const IMAGE_OS2_HEADER *ne) -{ - DWORD currpos = SetFilePointer( hfile, 0, NULL, SEEK_CUR); - DWORD ret = BINARY_OS216; - LPWORD modtab = NULL; - LPSTR nametab = NULL; - DWORD len; - int i; - - /* read modref table */ - if ( (SetFilePointer( hfile, mz->e_lfanew + ne->ne_modtab, NULL, SEEK_SET ) == -1) - || (!(modtab = HeapAlloc( GetProcessHeap(), 0, ne->ne_cmod*sizeof(WORD)))) - || (!(ReadFile(hfile, modtab, ne->ne_cmod*sizeof(WORD), &len, NULL))) - || (len != ne->ne_cmod*sizeof(WORD)) ) - goto done; - - /* read imported names table */ - if ( (SetFilePointer( hfile, mz->e_lfanew + ne->ne_imptab, NULL, SEEK_SET ) == -1) - || (!(nametab = HeapAlloc( GetProcessHeap(), 0, ne->ne_enttab - ne->ne_imptab))) - || (!(ReadFile(hfile, nametab, ne->ne_enttab - ne->ne_imptab, &len, NULL))) - || (len != ne->ne_enttab - ne->ne_imptab) ) - goto done; - - for (i=0; i < ne->ne_cmod; i++) - { - LPSTR module = &nametab[modtab[i]]; - TRACE("modref: %.*s\n", module[0], &module[1]); - if (!(strncmp(&module[1], "KERNEL", module[0]))) - { /* very old Windows file */ - MESSAGE("This seems to be a very old (pre-3.0) Windows executable. Expect crashes, especially if this is a real-mode binary !\n"); - ret = BINARY_WIN16; - break; - } - } - -done: - HeapFree( GetProcessHeap(), 0, modtab); - HeapFree( GetProcessHeap(), 0, nametab); - SetFilePointer( hfile, currpos, NULL, SEEK_SET); /* restore filepos */ - return ret; -} - /*********************************************************************** * MODULE_GetBinaryType */ @@ -460,7 +412,7 @@ void MODULE_get_binary_info( HANDLE hfile, struct binary_info *info ) * This will tell us if there is more header information * to read or not. */ - info->type = BINARY_DOS; + info->type = BINARY_WIN16; info->arch = IMAGE_FILE_MACHINE_I386; if (SetFilePointer( hfile, header.mz.e_lfanew, NULL, SEEK_SET ) == -1) return; if (!ReadFile( hfile, &ext_header, sizeof(ext_header), &len, NULL ) || len < 4) return; @@ -504,28 +456,6 @@ void MODULE_get_binary_info( HANDLE hfile, struct binary_info *info ) } } } - else if (!memcmp( &ext_header.os2.ne_magic, "NE", 2 )) - { - /* This is a Windows executable (NE) header. This can - * mean either a 16-bit OS/2 or a 16-bit Windows or even a - * DOS program (running under a DOS extender). To decide - * which, we'll have to read the NE header. - */ - if (len >= sizeof(ext_header.os2)) - { - if (ext_header.os2.ne_flags & NE_FFLAGS_LIBMODULE) info->flags |= BINARY_FLAG_DLL; - switch ( ext_header.os2.ne_exetyp ) - { - case 1: info->type = BINARY_OS216; break; /* OS/2 */ - case 2: info->type = BINARY_WIN16; break; /* Windows */ - case 3: info->type = BINARY_DOS; break; /* European MS-DOS 4.x */ - case 4: info->type = BINARY_WIN16; break; /* Windows 386; FIXME: is this 32bit??? */ - case 5: info->type = BINARY_DOS; break; /* BOSS, Borland Operating System Services */ - /* other types, e.g. 0 is: "unknown" */ - default: info->type = MODULE_Decide_OS2_OldWin(hfile, &header.mz, &ext_header.os2); break; - } - } - } } } diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index b0b5ccf..e159047 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -2478,9 +2478,7 @@ static BOOL create_process_impl( LPCWSTR app_name, LPWSTR cmd_line, LPSECURITY_A retv = create_process( hFile, name, tidy_cmdline, envW, cur_dir, process_attr, thread_attr, inherit, flags, startup_info, info, unixdir, &binary_info, FALSE ); break; - case BINARY_OS216: case BINARY_WIN16: - case BINARY_DOS: TRACE( "starting %s as Win16/DOS binary\n", debugstr_w(name) ); retv = create_vdm_process( name, tidy_cmdline, envW, cur_dir, process_attr, thread_attr, inherit, flags, startup_info, info, unixdir, &binary_info, FALSE ); @@ -2498,7 +2496,7 @@ static BOOL create_process_impl( LPCWSTR app_name, LPWSTR cmd_line, LPSECURITY_A if (!strcmpiW( p, comW ) || !strcmpiW( p, pifW )) { TRACE( "starting %s as DOS binary\n", debugstr_w(name) ); - binary_info.type = BINARY_DOS; + binary_info.type = BINARY_WIN16; binary_info.arch = IMAGE_FILE_MACHINE_I386; retv = create_vdm_process( name, tidy_cmdline, envW, cur_dir, process_attr, thread_attr, inherit, flags, startup_info, info, unixdir, @@ -2640,12 +2638,10 @@ static void exec_process( LPCWSTR name ) /* check for .com or .pif extension */ if (!(p = strrchrW( name, '.' ))) break; if (strcmpiW( p, comW ) && strcmpiW( p, pifW )) break; - binary_info.type = BINARY_DOS; + binary_info.type = BINARY_WIN16; binary_info.arch = IMAGE_FILE_MACHINE_I386; /* fall through */ - case BINARY_OS216: case BINARY_WIN16: - case BINARY_DOS: TRACE( "starting %s as Win16/DOS binary\n", debugstr_w(name) ); create_vdm_process( name, GetCommandLineW(), NULL, NULL, NULL, NULL, FALSE, 0, &startup_info, &info, NULL, &binary_info, TRUE );

1 0

Alexandre Julliard : kernel32: Reimplement GetBinaryTypeW to rely on the server for header parsing.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 57512807b9da8c5852e0ff1466950bad9ece36ad URL: https://source.winehq.org/git/wine.git/?a=commit;h=57512807b9da8c5852e0ff14… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 25 17:42:19 2018 +0200 kernel32: Reimplement GetBinaryTypeW to rely on the server for header parsing. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/module.c | 122 ++++++++++++++++++++++++++----------------------- 1 file changed, 65 insertions(+), 57 deletions(-) diff --git a/dlls/kernel32/module.c b/dlls/kernel32/module.c index 1188298..31e3014 100644 --- a/dlls/kernel32/module.c +++ b/dlls/kernel32/module.c @@ -563,76 +563,84 @@ void MODULE_get_binary_info( HANDLE hfile, struct binary_info *info ) * ".com" and ".pif" files are only recognized by their file name extension, * as per native Windows. */ -BOOL WINAPI GetBinaryTypeW( LPCWSTR lpApplicationName, LPDWORD lpBinaryType ) +BOOL WINAPI GetBinaryTypeW( LPCWSTR name, LPDWORD type ) { - BOOL ret = FALSE; - HANDLE hfile; - struct binary_info binary_info; + static const WCHAR comW[] = { '.','c','o','m',0 }; + static const WCHAR pifW[] = { '.','p','i','f',0 }; + HANDLE hfile, mapping; + NTSTATUS status; + const WCHAR *ptr; - TRACE("%s\n", debugstr_w(lpApplicationName) ); + TRACE("%s\n", debugstr_w(name) ); - /* Sanity check. - */ - if ( lpApplicationName == NULL || lpBinaryType == NULL ) - return FALSE; + if (type == NULL) return FALSE; - /* Open the file indicated by lpApplicationName for reading. - */ - hfile = CreateFileW( lpApplicationName, GENERIC_READ, FILE_SHARE_READ, - NULL, OPEN_EXISTING, 0, 0 ); + hfile = CreateFileW( name, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, 0 ); if ( hfile == INVALID_HANDLE_VALUE ) return FALSE; - /* Check binary type - */ - MODULE_get_binary_info( hfile, &binary_info ); - switch (binary_info.type) - { - case BINARY_UNKNOWN: + status = NtCreateSection( &mapping, STANDARD_RIGHTS_REQUIRED | SECTION_QUERY, + NULL, NULL, PAGE_READONLY, SEC_IMAGE, hfile ); + CloseHandle( hfile ); + + switch (status) { - static const WCHAR comW[] = { '.','C','O','M',0 }; - static const WCHAR pifW[] = { '.','P','I','F',0 }; - const WCHAR *ptr; - - /* try to determine from file name */ - ptr = strrchrW( lpApplicationName, '.' ); - if (!ptr) break; - if (!strcmpiW( ptr, comW )) + case STATUS_SUCCESS: { - *lpBinaryType = SCS_DOS_BINARY; - ret = TRUE; + SECTION_IMAGE_INFORMATION info; + + status = NtQuerySection( mapping, SectionImageInformation, &info, sizeof(info), NULL ); + CloseHandle( mapping ); + if (status) return FALSE; + switch (info.Machine) + { + case IMAGE_FILE_MACHINE_I386: + case IMAGE_FILE_MACHINE_ARM: + case IMAGE_FILE_MACHINE_THUMB: + case IMAGE_FILE_MACHINE_ARMNT: + case IMAGE_FILE_MACHINE_POWERPC: + *type = SCS_32BIT_BINARY; + return TRUE; + case IMAGE_FILE_MACHINE_AMD64: + case IMAGE_FILE_MACHINE_ARM64: + *type = SCS_64BIT_BINARY; + return TRUE; + } + return FALSE; } - else if (!strcmpiW( ptr, pifW )) + case STATUS_INVALID_IMAGE_WIN_16: + *type = SCS_WOW_BINARY; + return TRUE; + case STATUS_INVALID_IMAGE_WIN_32: + *type = SCS_32BIT_BINARY; + return TRUE; + case STATUS_INVALID_IMAGE_WIN_64: + *type = SCS_64BIT_BINARY; + return TRUE; + case STATUS_INVALID_IMAGE_NE_FORMAT: + *type = SCS_OS216_BINARY; + return TRUE; + case STATUS_INVALID_IMAGE_PROTECT: + *type = SCS_DOS_BINARY; + return TRUE; + case STATUS_INVALID_IMAGE_NOT_MZ: + if ((ptr = strrchrW( name, '.' ))) { - *lpBinaryType = SCS_PIF_BINARY; - ret = TRUE; + if (!strcmpiW( ptr, comW )) + { + *type = SCS_DOS_BINARY; + return TRUE; + } + if (!strcmpiW( ptr, pifW )) + { + *type = SCS_PIF_BINARY; + return TRUE; + } } - break; - } - case BINARY_PE: - *lpBinaryType = (binary_info.flags & BINARY_FLAG_64BIT) ? SCS_64BIT_BINARY : SCS_32BIT_BINARY; - ret = TRUE; - break; - case BINARY_WIN16: - *lpBinaryType = SCS_WOW_BINARY; - ret = TRUE; - break; - case BINARY_OS216: - *lpBinaryType = SCS_OS216_BINARY; - ret = TRUE; - break; - case BINARY_DOS: - *lpBinaryType = SCS_DOS_BINARY; - ret = TRUE; - break; - case BINARY_UNIX_EXE: - case BINARY_UNIX_LIB: - ret = FALSE; - break; + return FALSE; + default: + return FALSE; } - - CloseHandle( hfile ); - return ret; } /***********************************************************************

1 0

Alexandre Julliard : server: Return more specific error status for NE binaries.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 622aeeba6a37fe85f516b9ceeb3f66c9fcc6865f URL: https://source.winehq.org/git/wine.git/?a=commit;h=622aeeba6a37fe85f516b9ce… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 25 16:52:00 2018 +0200 server: Return more specific error status for NE binaries. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/loader.c | 18 ++++++++++++++++++ server/mapping.c | 7 +++++-- server/trace.c | 1 + 3 files changed, 24 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/tests/loader.c b/dlls/kernel32/tests/loader.c index e7a1a43..bc88c5f 100644 --- a/dlls/kernel32/tests/loader.c +++ b/dlls/kernel32/tests/loader.c @@ -972,6 +972,24 @@ static void test_Loader(void) nt_header.Signature = IMAGE_OS2_SIGNATURE; status = map_image_section( &nt_header, &section, section_data, __LINE__ ); ok( status == STATUS_INVALID_IMAGE_NE_FORMAT, "NtCreateSection error %08x\n", status ); + for (i = 0; i < 16; i++) + { + ((IMAGE_OS2_HEADER *)&nt_header)->ne_exetyp = i; + status = map_image_section( &nt_header, &section, section_data, __LINE__ ); + switch (i) + { + case 2: + ok( status == STATUS_INVALID_IMAGE_WIN_16, "NtCreateSection %u error %08x\n", i, status ); + break; + case 5: + ok( status == STATUS_INVALID_IMAGE_PROTECT, "NtCreateSection %u error %08x\n", i, status ); + break; + default: + ok( status == STATUS_INVALID_IMAGE_NE_FORMAT, "NtCreateSection %u error %08x\n", i, status ); + break; + } + } + ((IMAGE_OS2_HEADER *)&nt_header)->ne_exetyp = ((IMAGE_OS2_HEADER *)&nt_header_template)->ne_exetyp; nt_header.Signature = 0xdeadbeef; status = map_image_section( &nt_header, &section, section_data, __LINE__ ); diff --git a/server/mapping.c b/server/mapping.c index 1ed8c9d..2955766 100644 --- a/server/mapping.c +++ b/server/mapping.c @@ -587,8 +587,11 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s if (size < sizeof(nt)) memset( (char *)&nt + size, 0, sizeof(nt) - size ); if (nt.Signature != IMAGE_NT_SIGNATURE) { - if (*(WORD *)&nt.Signature == IMAGE_OS2_SIGNATURE) return STATUS_INVALID_IMAGE_NE_FORMAT; - return STATUS_INVALID_IMAGE_PROTECT; + IMAGE_OS2_HEADER *os2 = (IMAGE_OS2_HEADER *)&nt; + if (os2->ne_magic != IMAGE_OS2_SIGNATURE) return STATUS_INVALID_IMAGE_PROTECT; + if (os2->ne_exetyp == 2) return STATUS_INVALID_IMAGE_WIN_16; + if (os2->ne_exetyp == 5) return STATUS_INVALID_IMAGE_PROTECT; + return STATUS_INVALID_IMAGE_NE_FORMAT; } switch (nt.opt.hdr32.Magic) diff --git a/server/trace.c b/server/trace.c index 7b54c4f..5140ab1 100644 --- a/server/trace.c +++ b/server/trace.c @@ -5480,6 +5480,7 @@ static const struct { "INVALID_IMAGE_NE_FORMAT", STATUS_INVALID_IMAGE_NE_FORMAT }, { "INVALID_IMAGE_NOT_MZ", STATUS_INVALID_IMAGE_NOT_MZ }, { "INVALID_IMAGE_PROTECT", STATUS_INVALID_IMAGE_PROTECT }, + { "INVALID_IMAGE_WIN_16", STATUS_INVALID_IMAGE_WIN_16 }, { "INVALID_IMAGE_WIN_64", STATUS_INVALID_IMAGE_WIN_64 }, { "INVALID_LOCK_SEQUENCE", STATUS_INVALID_LOCK_SEQUENCE }, { "INVALID_OWNER", STATUS_INVALID_OWNER },

1 0

Jason Edmeades : msvcrt: Do not put cmd.exe special environment variables into the environ.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 0dcfc97fcbea15037d15dd1021870de25b8382df URL: https://source.winehq.org/git/wine.git/?a=commit;h=0dcfc97fcbea15037d15dd10… Author: Jason Edmeades <us(a)edmeades.me.uk> Date: Tue Sep 25 08:39:59 2018 +0100 msvcrt: Do not put cmd.exe special environment variables into the environ. All the special environment variables from the command shell which track directory use are stripped out from the C runtime environ/wenviron. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=45320 Signed-off-by: Jason Edmeades <us(a)edmeades.me.uk> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/data.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/msvcrt/data.c b/dlls/msvcrt/data.c index 06e16a8..f08d9b1 100644 --- a/dlls/msvcrt/data.c +++ b/dlls/msvcrt/data.c @@ -73,7 +73,8 @@ char ** msvcrt_SnapshotOfEnvironmentA(char **blk) for (ptr = environ_strings; *ptr; ptr += strlen(ptr) + 1) { - count++; + /* Don't count environment variables starting with '=' which are command shell specific */ + if (*ptr != '=') count++; len += strlen(ptr) + 1; } if (blk) @@ -88,7 +89,8 @@ char ** msvcrt_SnapshotOfEnvironmentA(char **blk) memcpy(&blk[count],environ_strings,len); for (ptr = (char*) &blk[count]; *ptr; ptr += strlen(ptr) + 1) { - blk[i++] = ptr; + /* Skip special environment strings set by the command shell */ + if (*ptr != '=') blk[i++] = ptr; } } blk[i] = NULL; @@ -105,7 +107,8 @@ MSVCRT_wchar_t ** msvcrt_SnapshotOfEnvironmentW(MSVCRT_wchar_t **wblk) for (wptr = wenviron_strings; *wptr; wptr += strlenW(wptr) + 1) { - count++; + /* Don't count environment variables starting with '=' which are command shell specific */ + if (*wptr != '=') count++; len += strlenW(wptr) + 1; } if (wblk) @@ -119,7 +122,8 @@ MSVCRT_wchar_t ** msvcrt_SnapshotOfEnvironmentW(MSVCRT_wchar_t **wblk) memcpy(&wblk[count],wenviron_strings,len * sizeof(MSVCRT_wchar_t)); for (wptr = (MSVCRT_wchar_t*)&wblk[count]; *wptr; wptr += strlenW(wptr) + 1) { - wblk[i++] = wptr; + /* Skip special environment strings set by the command shell */ + if (*wptr != '=') wblk[i++] = wptr; } } wblk[i] = NULL;

1 0

Zebediah Figura : quartz/filtergraph: Improve HRESULT propagation from IFilterGraph2_Connect().
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 58148a31fcc4962eee1017242e03fc8680dccf67 URL: https://source.winehq.org/git/wine.git/?a=commit;h=58148a31fcc4962eee101724… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Mon Sep 24 23:55:21 2018 -0500 quartz/filtergraph: Improve HRESULT propagation from IFilterGraph2_Connect(). Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/filtergraph.c | 11 +++++-- dlls/quartz/tests/filtergraph.c | 67 +++++++++++++++++++++++++++++++++++++++-- 2 files changed, 73 insertions(+), 5 deletions(-) diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c index 4f3e0a7..8d99ee5 100644 --- a/dlls/quartz/filtergraph.c +++ b/dlls/quartz/filtergraph.c @@ -1147,7 +1147,11 @@ static HRESULT WINAPI FilterGraph2_Connect(IFilterGraph2 *iface, IPin *ppinOut, /* Try direct connection first */ hr = IPin_Connect(ppinOut, ppinIn, NULL); - if (SUCCEEDED(hr)) + + /* If direct connection succeeded, we should propagate that return value. + * If it returned VFW_E_NOT_CONNECTED or VFW_E_NO_AUDIO_HARDWARE, then don't + * even bother trying intermediate filters, since they won't succeed. */ + if (SUCCEEDED(hr) || hr == VFW_E_NOT_CONNECTED || hr == VFW_E_NO_AUDIO_HARDWARE) goto out; TRACE("Direct connection failed, trying to render using extra filters\n"); @@ -1345,6 +1349,9 @@ error: } } + if (FAILED(hr)) + hr = VFW_E_CANNOT_CONNECT; + IEnumMoniker_Release(pEnumMoniker); out: @@ -1357,7 +1364,7 @@ out: --This->recursioncount; LeaveCriticalSection(&This->cs); TRACE("--> %08x\n", hr); - return SUCCEEDED(hr) ? S_OK : hr; + return hr; } /* Render all output pins of the given filter. Helper for FilterGraph2_Render(). */ diff --git a/dlls/quartz/tests/filtergraph.c b/dlls/quartz/tests/filtergraph.c index 6c8b9fa..5693c86 100644 --- a/dlls/quartz/tests/filtergraph.c +++ b/dlls/quartz/tests/filtergraph.c @@ -609,7 +609,6 @@ static void test_render_run(const WCHAR *file) ok(!refs, "Graph has %u references\n", refs); hr = test_graph_builder_connect_file(filename); -todo_wine ok(hr == VFW_E_CANNOT_CONNECT, "got %#x\n", hr); } else @@ -776,6 +775,8 @@ struct testpin unsigned int type_count, enum_idx; AM_MEDIA_TYPE *request_mt, *accept_mt; + HRESULT Connect_hr; + HRESULT EnumMediaTypes_hr; HRESULT QueryInternalConnections_hr; }; @@ -964,6 +965,9 @@ static HRESULT WINAPI testpin_EnumMediaTypes(IPin *iface, IEnumMediaTypes **out) struct testpin *pin = impl_from_IPin(iface); if (winetest_debug > 1) trace("%p->EnumMediaTypes()\n", pin); + if (FAILED(pin->EnumMediaTypes_hr)) + return pin->EnumMediaTypes_hr; + *out = &pin->IEnumMediaTypes_iface; IEnumMediaTypes_AddRef(*out); pin->enum_idx = 0; @@ -1063,6 +1067,8 @@ static void testpin_init(struct testpin *pin, const IPinVtbl *vtbl, PIN_DIRECTIO pin->IEnumMediaTypes_iface.lpVtbl = &testenummt_vtbl; pin->ref = 1; pin->dir = dir; + pin->Connect_hr = S_OK; + pin->EnumMediaTypes_hr = S_OK; pin->QueryInternalConnections_hr = E_NOTIMPL; } @@ -1077,12 +1083,16 @@ static HRESULT WINAPI testsource_Connect(IPin *iface, IPin *peer, const AM_MEDIA HRESULT hr; if (winetest_debug > 1) trace("%p->Connect(%p)\n", pin, peer); + if (FAILED(pin->Connect_hr)) + return pin->Connect_hr; + ok(!mt, "Got media type %p.\n", mt); if (SUCCEEDED(hr = IPin_ReceiveConnection(peer, &pin->IPin_iface, pin->request_mt))) { pin->peer = peer; IPin_AddRef(peer); + return pin->Connect_hr; } return hr; } @@ -1711,12 +1721,50 @@ static void test_graph_builder_connect(void) IFilterGraph2_Disconnect(graph, source_pin.peer); IFilterGraph2_Disconnect(graph, &source_pin.IPin_iface); + for (source_pin.Connect_hr = 0x00040200; source_pin.Connect_hr <= 0x000402ff; + ++source_pin.Connect_hr) + { + hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); + ok(hr == source_pin.Connect_hr, "Got hr %#x for Connect() hr %#x.\n", + hr, source_pin.Connect_hr); + ok(source_pin.peer == &sink_pin.IPin_iface, "Got peer %p.\n", source_pin.peer); + IFilterGraph2_Disconnect(graph, source_pin.peer); + IFilterGraph2_Disconnect(graph, &source_pin.IPin_iface); + } + source_pin.Connect_hr = S_OK; + sink_pin.accept_mt = &sink_type; hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); -todo_wine ok(hr == VFW_E_CANNOT_CONNECT, "Got hr %#x.\n", hr); ok(!source_pin.peer, "Got peer %p.\n", source_pin.peer); + for (source_pin.Connect_hr = 0x80040200; source_pin.Connect_hr <= 0x800402ff; + ++source_pin.Connect_hr) + { + hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); + if (source_pin.Connect_hr == VFW_E_NOT_CONNECTED + || source_pin.Connect_hr == VFW_E_NO_AUDIO_HARDWARE) + ok(hr == source_pin.Connect_hr, "Got hr %#x for Connect() hr %#x.\n", + hr, source_pin.Connect_hr); + else + ok(hr == VFW_E_CANNOT_CONNECT, "Got hr %#x for Connect() hr %#x.\n", + hr, source_pin.Connect_hr); + ok(!source_pin.peer, "Got peer %p.\n", source_pin.peer); + ok(!sink_pin.peer, "Got peer %p.\n", sink_pin.peer); + } + source_pin.Connect_hr = S_OK; + + for (source_pin.EnumMediaTypes_hr = 0x80040200; source_pin.EnumMediaTypes_hr <= 0x800402ff; + ++source_pin.EnumMediaTypes_hr) + { + hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); + ok(hr == source_pin.EnumMediaTypes_hr, "Got hr %#x for EnumMediaTypes() hr %#x.\n", + hr, source_pin.EnumMediaTypes_hr); + ok(!source_pin.peer, "Got peer %p.\n", source_pin.peer); + ok(!sink_pin.peer, "Got peer %p.\n", sink_pin.peer); + } + source_pin.EnumMediaTypes_hr = S_OK; + /* Test usage of intermediate filters. Similarly to Render(), filters are * simply tried in enumeration order. */ @@ -1740,6 +1788,20 @@ todo_wine IFilterGraph2_Disconnect(graph, sink_pin.peer); IFilterGraph2_Disconnect(graph, &sink_pin.IPin_iface); + for (source_pin.Connect_hr = 0x00040200; source_pin.Connect_hr <= 0x000402ff; + ++source_pin.Connect_hr) + { + hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); + ok(hr == S_OK, "Got hr %#x for Connect() hr %#x.\n", hr, source_pin.Connect_hr); + ok(source_pin.peer == &parser2_pins[0].IPin_iface, "Got peer %p.\n", source_pin.peer); + ok(sink_pin.peer == &parser2_pins[1].IPin_iface, "Got peer %p.\n", sink_pin.peer); + IFilterGraph2_Disconnect(graph, source_pin.peer); + IFilterGraph2_Disconnect(graph, &source_pin.IPin_iface); + IFilterGraph2_Disconnect(graph, sink_pin.peer); + IFilterGraph2_Disconnect(graph, &sink_pin.IPin_iface); + } + source_pin.Connect_hr = S_OK; + IFilterGraph2_RemoveFilter(graph, &parser1.IBaseFilter_iface); IFilterGraph2_AddFilter(graph, &parser1.IBaseFilter_iface, NULL); @@ -1805,7 +1867,6 @@ todo_wine parser1_pins[1].name[0] = '~'; hr = IFilterGraph2_Connect(graph, &source_pin.IPin_iface, &sink_pin.IPin_iface); -todo_wine ok(hr == VFW_E_CANNOT_CONNECT, "Got hr %#x.\n", hr); ok(!source_pin.peer, "Got peer %p.\n", source_pin.peer);

1 0

Louis Lenders : dwmapi: Return S_OK in DwmExtendFrameIntoClientArea.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: a1f5a7699e7138a76cb9dc50e3cda1c8c5ffae57 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a1f5a7699e7138a76cb9dc50… Author: Louis Lenders <xerox.xerox2000x(a)gmail.com> Date: Fri Sep 21 18:58:24 2018 +0200 dwmapi: Return S_OK in DwmExtendFrameIntoClientArea. Signed-off-by: Louis Lenders <xerox.xerox2000x(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dwmapi/dwmapi_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/dwmapi/dwmapi_main.c b/dlls/dwmapi/dwmapi_main.c index 0e6ba3b..76e225f 100644 --- a/dlls/dwmapi/dwmapi_main.c +++ b/dlls/dwmapi/dwmapi_main.c @@ -82,7 +82,7 @@ HRESULT WINAPI DwmExtendFrameIntoClientArea(HWND hwnd, const MARGINS* margins) { FIXME("(%p, %p) stub\n", hwnd, margins); - return E_NOTIMPL; + return S_OK; } /**********************************************************************

1 0

Henri Verbeet : wined3d: Store the wined3d creation flags for NO3D adapters as well.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: b1d6d47c0c01d7d14ab74f10720758708e4f070b URL: https://source.winehq.org/git/wine.git/?a=commit;h=b1d6d47c0c01d7d14ab74f10… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Sep 25 02:13:21 2018 +0330 wined3d: Store the wined3d creation flags for NO3D adapters as well. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/directx.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index abe81de..f7ef5c0 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -2463,7 +2463,14 @@ static const struct wined3d_adapter_ops wined3d_adapter_no3d_ops = wined3d_adapter_no3d_create_context, }; -static BOOL wined3d_adapter_no3d_init(struct wined3d_adapter *adapter) +static void wined3d_adapter_no3d_init_d3d_info(struct wined3d_adapter *adapter, DWORD wined3d_creation_flags) +{ + struct wined3d_d3d_info *d3d_info = &adapter->d3d_info; + + d3d_info->wined3d_creation_flags = wined3d_creation_flags; +} + +static BOOL wined3d_adapter_no3d_init(struct wined3d_adapter *adapter, DWORD wined3d_creation_flags) { static const struct wined3d_gpu_description gpu_description = { @@ -2485,6 +2492,8 @@ static BOOL wined3d_adapter_no3d_init(struct wined3d_adapter *adapter) adapter->shader_backend = &none_shader_backend; adapter->adapter_ops = &wined3d_adapter_no3d_ops; + wined3d_adapter_no3d_init_d3d_info(adapter, wined3d_creation_flags); + return TRUE; } @@ -2510,7 +2519,7 @@ static BOOL wined3d_adapter_init(struct wined3d_adapter *adapter, unsigned int o adapter->formats = NULL; if (wined3d_creation_flags & WINED3D_NO3D) - return wined3d_adapter_no3d_init(adapter); + return wined3d_adapter_no3d_init(adapter, wined3d_creation_flags); return wined3d_adapter_gl_init(adapter, wined3d_creation_flags); }

1 0

Henri Verbeet : ddraw: Reject creating DDSCAPS_VIDEOMEMORY surfaces with DDRAW_NO3D.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 33d182dd5414c609e65c5882dc92d9cefb5faa31 URL: https://source.winehq.org/git/wine.git/?a=commit;h=33d182dd5414c609e65c5882… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Sep 25 02:13:20 2018 +0330 ddraw: Reject creating DDSCAPS_VIDEOMEMORY surfaces with DDRAW_NO3D. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/surface.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index cf09f9c..c51db76 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -6122,8 +6122,9 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ else if (desc->ddsCaps.dwCaps & DDSCAPS_3DDEVICE) usage = WINED3DUSAGE_RENDERTARGET; - if (SUCCEEDED(hr = wined3d_check_device_format(ddraw->wined3d, WINED3DADAPTER_DEFAULT, - WINED3D_DEVICE_TYPE_HAL, mode.format_id, usage, WINED3D_RTYPE_TEXTURE_2D, wined3d_desc.format))) + if (!(ddraw->flags & DDRAW_NO3D) && SUCCEEDED(hr = wined3d_check_device_format(ddraw->wined3d, + WINED3DADAPTER_DEFAULT, WINED3D_DEVICE_TYPE_HAL, mode.format_id, + usage, WINED3D_RTYPE_TEXTURE_2D, wined3d_desc.format))) desc->ddsCaps.dwCaps |= DDSCAPS_VIDEOMEMORY; else desc->ddsCaps.dwCaps |= DDSCAPS_SYSTEMMEMORY; @@ -6257,6 +6258,13 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ return DDERR_NOCOLORKEYHW; } + if ((ddraw->flags & DDRAW_NO3D) && (desc->ddsCaps.dwCaps & DDSCAPS_VIDEOMEMORY)) + { + WARN("Video memory surfaces not supported without 3D support.\n"); + heap_free(texture); + return DDERR_NODIRECTDRAWHW; + } + if (desc->ddsCaps.dwCaps & (DDSCAPS_OVERLAY)) wined3d_desc.usage |= WINED3DUSAGE_OVERLAY;

1 0

Henri Verbeet : wined3d: Do not report WINEDDSCAPS_VIDEOMEMORY for WINED3D_NO3D adapters.
by Alexandre Julliard 25 Sep '18

25 Sep '18

Module: wine Branch: master Commit: 9efa178ee6b098d4d6d171356f7f7c2da03c164d URL: https://source.winehq.org/git/wine.git/?a=commit;h=9efa178ee6b098d4d6d17135… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Sep 25 02:13:19 2018 +0330 wined3d: Do not report WINEDDSCAPS_VIDEOMEMORY for WINED3D_NO3D adapters. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/directx.c | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 925a5bc..abe81de 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -2389,24 +2389,24 @@ HRESULT CDECL wined3d_get_device_caps(const struct wined3d *wined3d, UINT adapte caps->ddraw_caps.ssb_color_key_caps = ckey_caps; caps->ddraw_caps.ssb_fx_caps = fx_caps; - caps->ddraw_caps.dds_caps = WINEDDSCAPS_ALPHA | - WINEDDSCAPS_BACKBUFFER | - WINEDDSCAPS_FLIP | - WINEDDSCAPS_FRONTBUFFER | - WINEDDSCAPS_OFFSCREENPLAIN | - WINEDDSCAPS_PALETTE | - WINEDDSCAPS_PRIMARYSURFACE | - WINEDDSCAPS_SYSTEMMEMORY | - WINEDDSCAPS_VIDEOMEMORY | - WINEDDSCAPS_VISIBLE; + caps->ddraw_caps.dds_caps = WINEDDSCAPS_ALPHA + | WINEDDSCAPS_BACKBUFFER + | WINEDDSCAPS_FLIP + | WINEDDSCAPS_FRONTBUFFER + | WINEDDSCAPS_OFFSCREENPLAIN + | WINEDDSCAPS_PALETTE + | WINEDDSCAPS_PRIMARYSURFACE + | WINEDDSCAPS_SYSTEMMEMORY + | WINEDDSCAPS_VISIBLE; if (!(wined3d->flags & WINED3D_NO3D)) { - caps->ddraw_caps.dds_caps |= WINEDDSCAPS_3DDEVICE | - WINEDDSCAPS_MIPMAP | - WINEDDSCAPS_TEXTURE | - WINEDDSCAPS_ZBUFFER; - caps->ddraw_caps.caps |= WINEDDCAPS_3D; + caps->ddraw_caps.dds_caps |= WINEDDSCAPS_3DDEVICE + | WINEDDSCAPS_MIPMAP + | WINEDDSCAPS_TEXTURE + | WINEDDSCAPS_VIDEOMEMORY + | WINEDDSCAPS_ZBUFFER; + caps->ddraw_caps.caps |= WINEDDCAPS_3D; } caps->shader_double_precision = d3d_info->shader_double_precision;

1 0

← Newer
1
...
9
10
11
12
13
14
15
...
67
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2018