September 2018 - Wine-commits

Alexandre Julliard : kernel32: Move MODULE_get_binary_info implementation to process.c.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 8b492467d91d9a8c9a75d5d9f15bcbeebd25e6cc URL: https://source.winehq.org/git/wine.git/?a=commit;h=8b492467d91d9a8c9a75d5d9… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 27 13:27:28 2018 +0200 kernel32: Move MODULE_get_binary_info implementation to process.c. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/kernel_private.h | 24 ----- dlls/kernel32/module.c | 200 ------------------------------------ dlls/kernel32/process.c | 228 ++++++++++++++++++++++++++++++++++++++++- 3 files changed, 226 insertions(+), 226 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=8b492467d91d9a8c9a75…

1 0

Alexandre Julliard : kernel32: Retrieve binary information from the server for PE files.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 9d3a5a969c6c45468e012cdfc591f6485ad28ae7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9d3a5a969c6c45468e012cdf… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 27 13:37:30 2018 +0200 kernel32: Retrieve binary information from the server for PE files. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 180 ++++++++++++++++++++++-------------------------- 1 file changed, 82 insertions(+), 98 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=9d3a5a969c6c45468e01…

1 0

Alexandre Julliard : kernel32: Determine 32/ 64-bitness from the PE architecture.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: da40de3ced303178610e4da4640d1beead4096fb URL: https://source.winehq.org/git/wine.git/?a=commit;h=da40de3ced303178610e4da4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 27 14:23:52 2018 +0200 kernel32: Determine 32/64-bitness from the PE architecture. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 49e6bc2..02dce89 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -123,12 +123,9 @@ enum binary_type BINARY_UNIX_LIB }; -#define BINARY_FLAG_64BIT 0x02 - struct binary_info { enum binary_type type; - DWORD flags; pe_image_info_t pe; }; @@ -174,6 +171,15 @@ static inline unsigned int is_path_prefix( const WCHAR *prefix, const WCHAR *fil /*********************************************************************** + * is_64bit_arch + */ +static inline BOOL is_64bit_arch( WORD machine ) +{ + return (machine == IMAGE_FILE_MACHINE_AMD64 || machine == IMAGE_FILE_MACHINE_ARM64); +} + + +/*********************************************************************** * get_pe_info */ static NTSTATUS get_pe_info( HANDLE handle, pe_image_info_t *info ) @@ -251,15 +257,12 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) { case STATUS_SUCCESS: info->type = BINARY_PE; - if (info->pe.machine == IMAGE_FILE_MACHINE_AMD64 || info->pe.machine == IMAGE_FILE_MACHINE_ARM64) - info->flags |= BINARY_FLAG_64BIT; return; case STATUS_INVALID_IMAGE_WIN_32: info->type = BINARY_PE; return; case STATUS_INVALID_IMAGE_WIN_64: info->type = BINARY_PE; - info->flags |= BINARY_FLAG_64BIT; return; case STATUS_INVALID_IMAGE_WIN_16: case STATUS_INVALID_IMAGE_NE_FORMAT: @@ -279,7 +282,6 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) #else BOOL byteswap = (header.elf.data == 2); #endif - if (header.elf.class == 2) info->flags |= BINARY_FLAG_64BIT; if (byteswap) { header.elf.type = RtlUshortByteSwap( header.elf.type ); @@ -337,7 +339,6 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) else if (header.macho.magic == 0xfeedface || header.macho.magic == 0xcefaedfe || header.macho.magic == 0xfeedfacf || header.macho.magic == 0xcffaedfe) { - if ((header.macho.cputype >> 24) == 1) info->flags |= BINARY_FLAG_64BIT; if (header.macho.magic == 0xcefaedfe || header.macho.magic == 0xcffaedfe) { header.macho.filetype = RtlUlongByteSwap( header.macho.filetype ); @@ -371,7 +372,7 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil WCHAR *file_part; UINT len; void *redir_disabled = 0; - unsigned int flags = (sizeof(void*) > sizeof(int) ? BINARY_FLAG_64BIT : 0); + BOOL is_64bit = (sizeof(void*) > sizeof(int)); /* builtin names cannot be empty or contain spaces */ if (!libname[0] || strchrW( libname, ' ' ) || strchrW( libname, '\t' )) return FALSE; @@ -387,11 +388,11 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil if ((len = is_path_prefix( DIR_System, filename ))) { - if (is_wow64 && redir_disabled) flags = BINARY_FLAG_64BIT; + if (is_wow64 && redir_disabled) is_64bit = TRUE; } else if (DIR_SysWow64 && (len = is_path_prefix( DIR_SysWow64, filename ))) { - flags = 0; + is_64bit = FALSE; } else return FALSE; @@ -405,7 +406,7 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil file_part = filename + len; if (file_part > filename && file_part[-1] != '\\') *file_part++ = '\\'; strcpyW( file_part, libname ); - if (is_wow64 && redir_disabled) flags = BINARY_FLAG_64BIT; + if (is_wow64 && redir_disabled) is_64bit = TRUE; } if (ext && !strchrW( file_part, '.' )) { @@ -415,10 +416,9 @@ static BOOL get_builtin_path( const WCHAR *libname, const WCHAR *ext, WCHAR *fil } memset( binary_info, 0, sizeof(*binary_info) ); binary_info->type = BINARY_UNIX_LIB; - binary_info->flags = flags; /* assume current arch */ #if defined(__i386__) || defined(__x86_64__) - binary_info->pe.machine = (flags & BINARY_FLAG_64BIT) ? IMAGE_FILE_MACHINE_AMD64 : IMAGE_FILE_MACHINE_I386; + binary_info->pe.machine = is_64bit ? IMAGE_FILE_MACHINE_AMD64 : IMAGE_FILE_MACHINE_I386; #elif defined(__powerpc__) binary_info->pe.machine = IMAGE_FILE_MACHINE_POWERPC; #elif defined(__arm__) && !defined(__ARMEB__) @@ -2078,7 +2078,7 @@ static pid_t exec_loader( LPCWSTR cmd_line, unsigned int flags, int socketfd, argv = build_argv( cmd_line, 1 ); - if (!is_win64 ^ !(binary_info->flags & BINARY_FLAG_64BIT)) + if (!is_win64 ^ !is_64bit_arch( binary_info->pe.machine )) loader = get_alternate_loader( &wineloader ); if (exec_only || !(pid = fork())) /* child */ @@ -2678,7 +2678,7 @@ static BOOL create_process_impl( LPCWSTR app_name, LPWSTR cmd_line, LPSECURITY_A { case BINARY_PE: TRACE( "starting %s as Win%d binary (%s-%s, arch %04x)\n", - debugstr_w(name), (binary_info.flags & BINARY_FLAG_64BIT) ? 64 : 32, + debugstr_w(name), is_64bit_arch(binary_info.pe.machine) ? 64 : 32, wine_dbgstr_longlong(binary_info.pe.base), wine_dbgstr_longlong(binary_info.pe.base + binary_info.pe.map_size), binary_info.pe.machine ); @@ -2692,7 +2692,7 @@ static BOOL create_process_impl( LPCWSTR app_name, LPWSTR cmd_line, LPSECURITY_A break; case BINARY_UNIX_LIB: TRACE( "starting %s as %d-bit Winelib app\n", - debugstr_w(name), (binary_info.flags & BINARY_FLAG_64BIT) ? 64 : 32 ); + debugstr_w(name), is_64bit_arch(binary_info.pe.machine) ? 64 : 32 ); retv = create_process( hFile, name, tidy_cmdline, envW, cur_dir, process_attr, thread_attr, inherit, flags, startup_info, info, unixdir, &binary_info, FALSE ); break; @@ -2830,7 +2830,7 @@ static void exec_process( LPCWSTR name ) { case BINARY_PE: TRACE( "starting %s as Win%d binary (%s-%s, arch %04x)\n", - debugstr_w(name), (binary_info.flags & BINARY_FLAG_64BIT) ? 64 : 32, + debugstr_w(name), is_64bit_arch(binary_info.pe.machine) ? 64 : 32, wine_dbgstr_longlong(binary_info.pe.base), wine_dbgstr_longlong(binary_info.pe.base + binary_info.pe.map_size), binary_info.pe.machine );

1 0

Alexandre Julliard : kernel32: Return the binary type from get_binary_info ().
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: df637816bc3ad1afa10135fcb2b784eca49e6f06 URL: https://source.winehq.org/git/wine.git/?a=commit;h=df637816bc3ad1afa10135fc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 27 14:44:47 2018 +0200 kernel32: Return the binary type from get_binary_info(). Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 71 +++++++++++++++++++++---------------------------- 1 file changed, 31 insertions(+), 40 deletions(-) diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 02dce89..59a829b 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -207,7 +207,7 @@ static NTSTATUS get_pe_info( HANDLE handle, pe_image_info_t *info ) /*********************************************************************** * get_binary_info */ -static void get_binary_info( HANDLE hfile, struct binary_info *info ) +static enum binary_type get_binary_info( HANDLE hfile, struct binary_info *info ) { union { @@ -256,24 +256,21 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) switch (status) { case STATUS_SUCCESS: - info->type = BINARY_PE; - return; + return BINARY_PE; case STATUS_INVALID_IMAGE_WIN_32: - info->type = BINARY_PE; - return; + return BINARY_PE; case STATUS_INVALID_IMAGE_WIN_64: - info->type = BINARY_PE; - return; + return BINARY_PE; case STATUS_INVALID_IMAGE_WIN_16: case STATUS_INVALID_IMAGE_NE_FORMAT: case STATUS_INVALID_IMAGE_PROTECT: - info->type = BINARY_WIN16; - return; + return BINARY_WIN16; } /* Seek to the start of the file and read the header information. */ - if (SetFilePointer( hfile, 0, NULL, SEEK_SET ) == -1) return; - if (!ReadFile( hfile, &header, sizeof(header), &len, NULL ) || len != sizeof(header)) return; + if (SetFilePointer( hfile, 0, NULL, SEEK_SET ) == -1) return BINARY_UNKNOWN; + if (!ReadFile( hfile, &header, sizeof(header), &len, NULL ) || len != sizeof(header)) + return BINARY_UNKNOWN; if (!memcmp( header.elf.magic, "\177ELF", 4 )) { @@ -287,11 +284,19 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) header.elf.type = RtlUshortByteSwap( header.elf.type ); header.elf.machine = RtlUshortByteSwap( header.elf.machine ); } + switch(header.elf.machine) + { + case 3: info->pe.machine = IMAGE_FILE_MACHINE_I386; break; + case 20: info->pe.machine = IMAGE_FILE_MACHINE_POWERPC; break; + case 40: info->pe.machine = IMAGE_FILE_MACHINE_ARMNT; break; + case 50: info->pe.machine = IMAGE_FILE_MACHINE_IA64; break; + case 62: info->pe.machine = IMAGE_FILE_MACHINE_AMD64; break; + case 183: info->pe.machine = IMAGE_FILE_MACHINE_ARM64; break; + } switch(header.elf.type) { case 2: - info->type = BINARY_UNIX_EXE; - break; + return BINARY_UNIX_EXE; case 3: { LARGE_INTEGER phoff; @@ -309,30 +314,15 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) } while (phnum--) { - if (SetFilePointerEx( hfile, phoff, NULL, FILE_BEGIN ) == -1) return; - if (!ReadFile( hfile, &type, sizeof(type), &len, NULL ) || len < sizeof(type)) return; + if (SetFilePointerEx( hfile, phoff, NULL, FILE_BEGIN ) == -1) return BINARY_UNKNOWN; + if (!ReadFile( hfile, &type, sizeof(type), &len, NULL ) || len < sizeof(type)) + return BINARY_UNKNOWN; if (byteswap) type = RtlUlongByteSwap( type ); - if (type == 3) - { - info->type = BINARY_UNIX_EXE; - break; - } + if (type == 3) return BINARY_UNIX_EXE; phoff.QuadPart += (header.elf.class == 2) ? 56 : 32; } - if (!info->type) info->type = BINARY_UNIX_LIB; - break; - } - default: - return; + return BINARY_UNIX_LIB; } - switch(header.elf.machine) - { - case 3: info->pe.machine = IMAGE_FILE_MACHINE_I386; break; - case 20: info->pe.machine = IMAGE_FILE_MACHINE_POWERPC; break; - case 40: info->pe.machine = IMAGE_FILE_MACHINE_ARMNT; break; - case 50: info->pe.machine = IMAGE_FILE_MACHINE_IA64; break; - case 62: info->pe.machine = IMAGE_FILE_MACHINE_AMD64; break; - case 183: info->pe.machine = IMAGE_FILE_MACHINE_ARM64; break; } } /* Mach-o File with Endian set to Big Endian or Little Endian */ @@ -344,11 +334,6 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) header.macho.filetype = RtlUlongByteSwap( header.macho.filetype ); header.macho.cputype = RtlUlongByteSwap( header.macho.cputype ); } - switch(header.macho.filetype) - { - case 2: info->type = BINARY_UNIX_EXE; break; - case 8: info->type = BINARY_UNIX_LIB; break; - } switch(header.macho.cputype) { case 0x00000007: info->pe.machine = IMAGE_FILE_MACHINE_I386; break; @@ -357,7 +342,13 @@ static void get_binary_info( HANDLE hfile, struct binary_info *info ) case 0x0100000c: info->pe.machine = IMAGE_FILE_MACHINE_ARM64; break; case 0x00000012: info->pe.machine = IMAGE_FILE_MACHINE_POWERPC; break; } + switch(header.macho.filetype) + { + case 2: return BINARY_UNIX_EXE; + case 8: return BINARY_UNIX_LIB; + } } + return BINARY_UNKNOWN; } @@ -452,7 +443,7 @@ static HANDLE open_exe_file( const WCHAR *name, struct binary_info *binary_info if (contains_path( name ) && get_builtin_path( name, NULL, buffer, sizeof(buffer), binary_info )) handle = 0; } - else get_binary_info( handle, binary_info ); + else binary_info->type = get_binary_info( handle, binary_info ); return handle; } @@ -477,7 +468,7 @@ static BOOL find_exe_file( const WCHAR *name, WCHAR *buffer, int buflen, if ((*handle = CreateFileW( buffer, GENERIC_READ, FILE_SHARE_READ|FILE_SHARE_DELETE, NULL, OPEN_EXISTING, 0, 0 )) != INVALID_HANDLE_VALUE) { - get_binary_info( *handle, binary_info ); + binary_info->type = get_binary_info( *handle, binary_info ); return TRUE; } return FALSE;

1 0

Alexandre Julliard : kernel32: Get rid of the binary_info structure.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: ec13f6ec2ac84f32cc572148d542e92a57f47008 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ec13f6ec2ac84f32cc572148… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 27 14:54:01 2018 +0200 kernel32: Get rid of the binary_info structure. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/process.c | 205 +++++++++++++++++++++++------------------------- 1 file changed, 98 insertions(+), 107 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=ec13f6ec2ac84f32cc57…

1 0

Louis Lenders : mscms: Add stub for WcsGetUsePerUserProfiles.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 82c7f7977b9e13d297b8398b0902c5b3b8e630e9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=82c7f7977b9e13d297b8398b… Author: Louis Lenders <xerox.xerox2000x(a)gmail.com> Date: Thu Sep 27 13:12:28 2018 +0200 mscms: Add stub for WcsGetUsePerUserProfiles. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=45861 Signed-off-by: Louis Lenders <xerox.xerox2000x(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mscms/mscms.spec | 1 + dlls/mscms/profile.c | 10 ++++++++++ include/icm.h | 1 + 3 files changed, 12 insertions(+) diff --git a/dlls/mscms/mscms.spec b/dlls/mscms/mscms.spec index dd404f1..421a8fc 100644 --- a/dlls/mscms/mscms.spec +++ b/dlls/mscms/mscms.spec @@ -59,3 +59,4 @@ @ stdcall UninstallColorProfileW(wstr wstr long) @ stdcall UnregisterCMMA(str long) @ stdcall UnregisterCMMW(wstr long) +@ stdcall WcsGetUsePerUserProfiles(wstr long ptr) diff --git a/dlls/mscms/profile.c b/dlls/mscms/profile.c index cb4908c..53016ef 100644 --- a/dlls/mscms/profile.c +++ b/dlls/mscms/profile.c @@ -1525,3 +1525,13 @@ BOOL WINAPI CloseColorProfile( HPROFILE profile ) #endif /* HAVE_LCMS2 */ return ret; } + +/****************************************************************************** + * WcsGetUsePerUserProfiles [MSCMS.@] + */ +BOOL WINAPI WcsGetUsePerUserProfiles( const WCHAR* name, DWORD class, BOOL* use_per_user_profile ) +{ + FIXME( "%s %s %p\n", debugstr_w(name), dbgstr_tag(class), use_per_user_profile ); + SetLastError( ERROR_CALL_NOT_IMPLEMENTED ); + return FALSE; +} diff --git a/include/icm.h b/include/icm.h index 34b0b63..8a666fa 100644 --- a/include/icm.h +++ b/include/icm.h @@ -403,6 +403,7 @@ BOOL WINAPI UninstallColorProfileW(PCWSTR,PCWSTR,BOOL); BOOL WINAPI UnregisterCMMA(PCSTR,DWORD); BOOL WINAPI UnregisterCMMW(PCWSTR,DWORD); #define UnregisterCMM WINELIB_NAME_AW(UnregisterCMM) +BOOL WINAPI WcsGetUsePerUserProfiles(const WCHAR*,DWORD,BOOL*); #define PROFILE_FILENAME 1 #define PROFILE_MEMBUFFER 2

1 0

Francois Gouget : winebus.sys: Add a trailing '\n' to a TRACE().
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 90de9129cc5f3458ddb88d8d1a09c415c896a66d URL: https://source.winehq.org/git/wine.git/?a=commit;h=90de9129cc5f3458ddb88d8d… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Sep 27 09:24:35 2018 +0200 winebus.sys: Add a trailing '\n' to a TRACE(). Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winebus.sys/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/winebus.sys/main.c b/dlls/winebus.sys/main.c index 6b1949c..cee0ae6 100644 --- a/dlls/winebus.sys/main.c +++ b/dlls/winebus.sys/main.c @@ -391,7 +391,7 @@ static NTSTATUS handle_IRP_MN_QUERY_DEVICE_RELATIONS(IRP *irp) NTSTATUS status = irp->IoStatus.u.Status; IO_STACK_LOCATION *irpsp = IoGetCurrentIrpStackLocation( irp ); - TRACE("IRP_MN_QUERY_DEVICE_RELATIONS"); + TRACE("IRP_MN_QUERY_DEVICE_RELATIONS\n"); switch (irpsp->Parameters.QueryDeviceRelations.Type) { case EjectionRelations:

1 0

Francois Gouget : quartz/tests: Make testpin_{AddRef, Release}() static.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: d2dbe856d4e4e690812784df73457b3c1cfefa62 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d2dbe856d4e4e690812784df… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Sep 27 09:24:45 2018 +0200 quartz/tests: Make testpin_{AddRef,Release}() static. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/tests/filtergraph.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/quartz/tests/filtergraph.c b/dlls/quartz/tests/filtergraph.c index 74a3b68..e64a9fa 100644 --- a/dlls/quartz/tests/filtergraph.c +++ b/dlls/quartz/tests/filtergraph.c @@ -878,13 +878,13 @@ static HRESULT WINAPI testpin_QueryInterface(IPin *iface, REFIID iid, void **out return E_NOINTERFACE; } -ULONG WINAPI testpin_AddRef(IPin *iface) +static ULONG WINAPI testpin_AddRef(IPin *iface) { struct testpin *pin = impl_from_IPin(iface); return InterlockedIncrement(&pin->ref); } -ULONG WINAPI testpin_Release(IPin *iface) +static ULONG WINAPI testpin_Release(IPin *iface) { struct testpin *pin = impl_from_IPin(iface); return InterlockedDecrement(&pin->ref);

1 0

Zebediah Figura : quartz/tests: Move file source filter tests to filesource.c.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 039525bc0c30f42778a3d5defd1943cbd2995228 URL: https://source.winehq.org/git/wine.git/?a=commit;h=039525bc0c30f42778a3d5de… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Sep 26 23:36:20 2018 -0500 quartz/tests: Move file source filter tests to filesource.c. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/tests/Makefile.in | 1 + dlls/quartz/tests/avisplitter.c | 132 ---------------------------------- dlls/quartz/tests/filesource.c | 155 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 156 insertions(+), 132 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=039525bc0c30f42778a3…

1 0

Zebediah Figura : quartz/tests/filesource: Test interfaces exposed by the file source filter.
by Alexandre Julliard 27 Sep '18

27 Sep '18

Module: wine Branch: master Commit: 835a9a815340be9d6beec56cace289ff124dbe55 URL: https://source.winehq.org/git/wine.git/?a=commit;h=835a9a815340be9d6beec56c… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Sep 26 23:36:21 2018 -0500 quartz/tests/filesource: Test interfaces exposed by the file source filter. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/tests/filesource.c | 53 ++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 51 insertions(+), 2 deletions(-) diff --git a/dlls/quartz/tests/filesource.c b/dlls/quartz/tests/filesource.c index 15a091b..55c9bf2 100644 --- a/dlls/quartz/tests/filesource.c +++ b/dlls/quartz/tests/filesource.c @@ -2,6 +2,7 @@ * File source filter unit tests * * Copyright 2016 Sebastian Lackner + * Copyright 2018 Zebediah Figura * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -22,6 +23,54 @@ #include "dshow.h" #include "wine/test.h" +static IBaseFilter *create_file_source(void) +{ + IBaseFilter *filter = NULL; + HRESULT hr = CoCreateInstance(&CLSID_AsyncReader, NULL, CLSCTX_INPROC_SERVER, + &IID_IBaseFilter, (void **)&filter); + ok(hr == S_OK, "Got hr %#x.\n", hr); + return filter; +} + +#define check_interface(a, b, c) check_interface_(__LINE__, a, b, c) +static void check_interface_(unsigned int line, void *iface_ptr, REFIID iid, BOOL supported) +{ + IUnknown *iface = iface_ptr; + HRESULT hr, expected_hr; + IUnknown *unk; + + expected_hr = supported ? S_OK : E_NOINTERFACE; + + hr = IUnknown_QueryInterface(iface, iid, (void **)&unk); + ok_(__FILE__, line)(hr == expected_hr, "Got hr %#x, expected %#x.\n", hr, expected_hr); + if (SUCCEEDED(hr)) + IUnknown_Release(unk); +} + +static void test_interfaces(void) +{ + IBaseFilter *filter = create_file_source(); + + check_interface(filter, &IID_IBaseFilter, TRUE); + check_interface(filter, &IID_IFileSourceFilter, TRUE); + +todo_wine + check_interface(filter, &IID_IAMFilterMiscFlags, FALSE); + check_interface(filter, &IID_IBasicAudio, FALSE); + check_interface(filter, &IID_IBasicVideo, FALSE); + check_interface(filter, &IID_IKsPropertySet, FALSE); + check_interface(filter, &IID_IMediaPosition, FALSE); + check_interface(filter, &IID_IMediaSeeking, FALSE); + check_interface(filter, &IID_IPersistPropertyBag, FALSE); + check_interface(filter, &IID_IPin, FALSE); + check_interface(filter, &IID_IQualityControl, FALSE); + check_interface(filter, &IID_IQualProp, FALSE); + check_interface(filter, &IID_IReferenceClock, FALSE); + check_interface(filter, &IID_IVideoWindow, FALSE); + + IBaseFilter_Release(filter); +} + static void test_file_source_filter(void) { static const WCHAR prefix[] = {'w','i','n',0}; @@ -101,8 +150,7 @@ static void test_file_source_filter(void) ok(ret, "Failed to write file, error %u.\n", GetLastError()); CloseHandle(file); - CoCreateInstance(&CLSID_AsyncReader, NULL, CLSCTX_INPROC_SERVER, - &IID_IBaseFilter, (void **)&filter); + filter = create_file_source(); IBaseFilter_QueryInterface(filter, &IID_IFileSourceFilter, (void **)&filesource); olepath = (void *)0xdeadbeef; @@ -149,6 +197,7 @@ START_TEST(filesource) { CoInitialize(NULL); + test_interfaces(); test_file_source_filter(); CoUninitialize();

1 0