September 2019 - Wine-Commits

Alexandre Julliard : ntdll: Avoid server call when possible in NtSetContextThread() on ARM64.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: afe8467110649b303f6b59fd595bc5fea5b53910 URL: https://source.winehq.org/git/wine.git/?a=commit;h=afe8467110649b303f6b59fd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 19 19:41:14 2019 +0200 ntdll: Avoid server call when possible in NtSetContextThread() on ARM64. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/signal_arm64.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/dlls/ntdll/signal_arm64.c b/dlls/ntdll/signal_arm64.c index 25ac7dac02..7abf389bbd 100644 --- a/dlls/ntdll/signal_arm64.c +++ b/dlls/ntdll/signal_arm64.c @@ -435,12 +435,18 @@ NTSTATUS context_from_server( CONTEXT *to, const context_t *from ) */ NTSTATUS WINAPI NtSetContextThread( HANDLE handle, const CONTEXT *context ) { - NTSTATUS ret; - BOOL self; - context_t server_context; + NTSTATUS ret = STATUS_SUCCESS; + BOOL self = (handle == GetCurrentThread()); - context_to_server( &server_context, context ); - ret = set_thread_context( handle, &server_context, &self ); + if (self && (context->ContextFlags & (CONTEXT_DEBUG_REGISTERS & ~CONTEXT_ARM64))) + self = FALSE; + + if (!self) + { + context_t server_context; + context_to_server( &server_context, context ); + ret = set_thread_context( handle, &server_context, &self ); + } if (self && ret == STATUS_SUCCESS) set_cpu_context( context ); return ret; }

1 0

Alexandre Julliard : ntdll: Send first chance debug event while on signal stack on ARM64.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: 82c753a273b85eb6b30b3c9d625ea22e3dec94c3 URL: https://source.winehq.org/git/wine.git/?a=commit;h=82c753a273b85eb6b30b3c9d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 19 19:41:42 2019 +0200 ntdll: Send first chance debug event while on signal stack on ARM64. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/signal_arm64.c | 133 ++++++++++++++++++++++++---------------------- 1 file changed, 69 insertions(+), 64 deletions(-) diff --git a/dlls/ntdll/signal_arm64.c b/dlls/ntdll/signal_arm64.c index 7abf389bbd..43e5969229 100644 --- a/dlls/ntdll/signal_arm64.c +++ b/dlls/ntdll/signal_arm64.c @@ -483,52 +483,6 @@ NTSTATUS WINAPI NtGetContextThread( HANDLE handle, CONTEXT *context ) } -/*********************************************************************** - * setup_exception - * - * Setup the exception record and context on the thread stack. - */ -static struct stack_layout *setup_exception( ucontext_t *sigcontext ) -{ - struct stack_layout *stack; - DWORD exception_code = 0; - - /* push the stack_layout structure */ - stack = (struct stack_layout *)((SP_sig(sigcontext) - sizeof(*stack)) & ~15); - - stack->rec.ExceptionRecord = NULL; - stack->rec.ExceptionCode = exception_code; - stack->rec.ExceptionFlags = EXCEPTION_CONTINUABLE; - stack->rec.ExceptionAddress = (LPVOID)PC_sig(sigcontext); - stack->rec.NumberParameters = 0; - - save_context( &stack->context, sigcontext ); - save_fpu( &stack->context, sigcontext ); - return stack; -} - -/********************************************************************** - * raise_generic_exception - */ -static void WINAPI raise_generic_exception( EXCEPTION_RECORD *rec, CONTEXT *context ) -{ - NTSTATUS status = NtRaiseException( rec, context, TRUE ); - raise_status( status, rec ); -} - -/*********************************************************************** - * setup_raise_exception - * - * Modify the signal context to call the exception raise function. - */ -static void setup_raise_exception( ucontext_t *sigcontext, struct stack_layout *stack ) -{ - SP_sig(sigcontext) = (ULONG_PTR)stack; - PC_sig(sigcontext) = (ULONG_PTR)raise_generic_exception; - REGn_sig(0, sigcontext) = (ULONG_PTR)&stack->rec; /* first arg for raise_generic_exception */ - REGn_sig(1, sigcontext) = (ULONG_PTR)&stack->context; /* second arg for raise_generic_exception */ -} - /*********************************************************************** * libunwind_virtual_unwind * @@ -852,6 +806,7 @@ static NTSTATUS call_function_handlers( EXCEPTION_RECORD *rec, CONTEXT *orig_con dispatch.TargetPc = 0; dispatch.ContextRecord = &context; dispatch.HistoryTable = &table; + dispatch.NonVolatileRegisters = (BYTE *)&context.u.s.X19; for (;;) { status = virtual_unwind( UNW_FLAG_EHANDLER, &dispatch, &context ); @@ -979,19 +934,12 @@ static NTSTATUS raise_exception( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL f context->u.s.X24, context->u.s.X25, context->u.s.X26, context->u.s.X27 ); TRACE(" x28=%016lx fp=%016lx lr=%016lx sp=%016lx\n", context->u.s.X28, context->u.s.Fp, context->u.s.Lr, context->Sp ); - TRACE(" pc=%016lx\n", - context->Pc ); } - status = send_debug_event( rec, TRUE, context ); - if (status == DBG_CONTINUE || status == DBG_EXCEPTION_HANDLED) - return STATUS_SUCCESS; + if (call_vectored_handlers( rec, context ) == EXCEPTION_CONTINUE_EXECUTION) goto done; - if (call_vectored_handlers( rec, context ) == EXCEPTION_CONTINUE_EXECUTION) - return STATUS_SUCCESS; - - if ((status = call_function_handlers( rec, context )) != STATUS_UNHANDLED_EXCEPTION) - return status; + if ((status = call_function_handlers( rec, context )) == STATUS_SUCCESS) goto done; + if (status != STATUS_UNHANDLED_EXCEPTION) return status; } /* last chance exception */ @@ -1008,7 +956,62 @@ static NTSTATUS raise_exception( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL f rec->ExceptionCode, rec->ExceptionFlags, rec->ExceptionAddress ); NtTerminateProcess( NtCurrentProcess(), rec->ExceptionCode ); } - return STATUS_SUCCESS; +done: + return NtSetContextThread( GetCurrentThread(), context ); +} + +/*********************************************************************** + * setup_exception + * + * Setup the exception record and context on the thread stack. + */ +static struct stack_layout *setup_exception( ucontext_t *sigcontext ) +{ + struct stack_layout *stack; + DWORD exception_code = 0; + + /* push the stack_layout structure */ + stack = (struct stack_layout *)((SP_sig(sigcontext) - sizeof(*stack)) & ~15); + + stack->rec.ExceptionRecord = NULL; + stack->rec.ExceptionCode = exception_code; + stack->rec.ExceptionFlags = EXCEPTION_CONTINUABLE; + stack->rec.ExceptionAddress = (LPVOID)PC_sig(sigcontext); + stack->rec.NumberParameters = 0; + + save_context( &stack->context, sigcontext ); + save_fpu( &stack->context, sigcontext ); + return stack; +} + +/********************************************************************** + * raise_generic_exception + */ +static void WINAPI raise_generic_exception( EXCEPTION_RECORD *rec, CONTEXT *context ) +{ + NTSTATUS status = raise_exception( rec, context, TRUE ); + raise_status( status, rec ); +} + +/*********************************************************************** + * setup_raise_exception + * + * Modify the signal context to call the exception raise function. + */ +static void setup_raise_exception( ucontext_t *sigcontext, struct stack_layout *stack ) +{ + NTSTATUS status = send_debug_event( &stack->rec, TRUE, &stack->context ); + + if (status == DBG_CONTINUE || status == DBG_EXCEPTION_HANDLED) + { + restore_context( &stack->context, sigcontext ); + return; + } + SP_sig(sigcontext) = (ULONG_PTR)stack; + PC_sig(sigcontext) = (ULONG_PTR)raise_generic_exception; + REGn_sig(0, sigcontext) = (ULONG_PTR)&stack->rec; /* first arg for raise_generic_exception */ + REGn_sig(1, sigcontext) = (ULONG_PTR)&stack->context; /* second arg for raise_generic_exception */ + REGn_sig(18, sigcontext) = (ULONG_PTR)NtCurrentTeb(); } /********************************************************************** @@ -1790,10 +1793,10 @@ void WINAPI RtlUnwindEx( PVOID end_frame, PVOID target_ip, EXCEPTION_RECORD *rec TRACE(" x28=%016lx fp=%016lx lr=%016lx sp=%016lx\n", context->u.s.X28, context->u.s.Fp, context->u.s.Lr, context->Sp ); - dispatch.EstablisherFrame = context->Sp; dispatch.TargetPc = (ULONG64)target_ip; dispatch.ContextRecord = context; dispatch.HistoryTable = table; + dispatch.NonVolatileRegisters = (BYTE *)&context->u.s.X19; for (;;) { @@ -1891,9 +1894,13 @@ void WINAPI RtlUnwind( void *frame, void *target_ip, EXCEPTION_RECORD *rec, void */ NTSTATUS WINAPI NtRaiseException( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL first_chance ) { - NTSTATUS status = raise_exception( rec, context, first_chance ); - if (status == STATUS_SUCCESS) NtSetContextThread( GetCurrentThread(), context ); - return status; + if (first_chance) + { + NTSTATUS status = send_debug_event( rec, TRUE, context ); + if (status == DBG_CONTINUE || status == DBG_EXCEPTION_HANDLED) + NtSetContextThread( GetCurrentThread(), context ); + } + return raise_exception( rec, context, first_chance ); } /*********************************************************************** @@ -1902,12 +1909,10 @@ NTSTATUS WINAPI NtRaiseException( EXCEPTION_RECORD *rec, CONTEXT *context, BOOL void WINAPI RtlRaiseException( EXCEPTION_RECORD *rec ) { CONTEXT context; - NTSTATUS status; RtlCaptureContext( &context ); rec->ExceptionAddress = (LPVOID)context.Pc; - status = raise_exception( rec, &context, TRUE ); - if (status) raise_status( status, rec ); + RtlRaiseStatus( NtRaiseException( rec, &context, TRUE )); } /*************************************************************************

1 0

Jacek Caban : scrobj: Add registration element parser.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: e8d99613fc0abe6b9eb9fefcef0099b8fb9c2485 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e8d99613fc0abe6b9eb9fefc… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Sep 19 17:28:54 2019 +0200 scrobj: Add registration element parser. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/scrobj/scrobj.c | 142 ++++++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 140 insertions(+), 2 deletions(-) diff --git a/dlls/scrobj/scrobj.c b/dlls/scrobj/scrobj.c index 2bc746a179..83a9b099d7 100644 --- a/dlls/scrobj/scrobj.c +++ b/dlls/scrobj/scrobj.c @@ -44,6 +44,14 @@ struct scriptlet_factory IXmlReader *xml_reader; IMoniker *moniker; + + BOOL have_registration; + WCHAR *description; + WCHAR *progid; + WCHAR *versioned_progid; + WCHAR *version; + WCHAR *classid_str; + CLSID classid; }; typedef enum tid_t @@ -148,6 +156,19 @@ static BOOL is_xml_name(struct scriptlet_factory *factory, const WCHAR *name) return hres == S_OK && len == wcslen(name) && !memcmp(qname, name, len * sizeof(WCHAR)); } +static HRESULT expect_end_element(struct scriptlet_factory *factory) +{ + XmlNodeType node_type; + HRESULT hres; + hres = next_xml_node(factory, &node_type); + if (hres != S_OK || node_type != XmlNodeType_EndElement) + { + FIXME("Unexpected node %u %s\n", node_type, debugstr_xml_name(factory)); + return E_FAIL; + } + return S_OK; +} + static HRESULT expect_no_attributes(struct scriptlet_factory *factory) { UINT count; @@ -159,10 +180,122 @@ static HRESULT expect_no_attributes(struct scriptlet_factory *factory) return E_FAIL; } +static BOOL is_case_xml_name(struct scriptlet_factory *factory, const WCHAR *name) +{ + const WCHAR *qname; + UINT len; + HRESULT hres; + hres = IXmlReader_GetQualifiedName(factory->xml_reader, &qname, &len); + return hres == S_OK && len == wcslen(name) && !memicmp(qname, name, len * sizeof(WCHAR)); +} + +static HRESULT read_xml_value(struct scriptlet_factory *factory, WCHAR **ret) +{ + const WCHAR *str; + UINT len; + HRESULT hres; + hres = IXmlReader_GetValue(factory->xml_reader, &str, &len); + if (FAILED(hres)) return hres; + if (!(*ret = heap_alloc((len + 1) * sizeof(WCHAR)))) return E_OUTOFMEMORY; + memcpy(*ret, str, len * sizeof(WCHAR)); + (*ret)[len] = 0; + return S_OK; +} + static HRESULT parse_scriptlet_registration(struct scriptlet_factory *factory) { - FIXME("\n"); - return E_NOTIMPL; + HRESULT hres; + + if (factory->have_registration) + { + FIXME("duplicated registration element\n"); + return E_FAIL; + } + factory->have_registration = TRUE; + + for (;;) + { + hres = IXmlReader_MoveToNextAttribute(factory->xml_reader); + if (hres != S_OK) break; + + if (is_xml_name(factory, L"description")) + { + if (factory->description) + { + FIXME("Duplicated description\n"); + return E_FAIL; + } + hres = read_xml_value(factory, &factory->description); + if (FAILED(hres)) return hres; + } + else if (is_case_xml_name(factory, L"progid")) + { + if (factory->progid) + { + FIXME("Duplicated progid\n"); + return E_FAIL; + } + hres = read_xml_value(factory, &factory->progid); + if (FAILED(hres)) return hres; + } + else if (is_xml_name(factory, L"version")) + { + if (factory->version) + { + FIXME("Duplicated version\n"); + return E_FAIL; + } + hres = read_xml_value(factory, &factory->version); + if (FAILED(hres)) return hres; + } + else if (is_xml_name(factory, L"classid")) + { + if (factory->classid_str) + { + FIXME("Duplicated classid attribute\n"); + return E_FAIL; + } + hres = read_xml_value(factory, &factory->classid_str); + if (FAILED(hres)) return hres; + hres = IIDFromString(factory->classid_str, &factory->classid); + if (FAILED(hres)) + { + FIXME("Invalid classid %s\n", debugstr_w(factory->classid_str)); + + return E_FAIL; + } + } + else + { + FIXME("Unexpected attribute\n"); + return E_NOTIMPL; + } + } + + if (!factory->progid && !factory->classid_str) + { + FIXME("Incomplet registration element\n"); + return E_FAIL; + } + + if (factory->progid) + { + size_t progid_len = wcslen(factory->progid); + size_t version_len = wcslen(factory->version); + + if (!(factory->versioned_progid = heap_alloc((progid_len + version_len + 2) * sizeof(WCHAR)))) + return E_OUTOFMEMORY; + + memcpy(factory->versioned_progid, factory->progid, (progid_len + 1) * sizeof(WCHAR)); + if (version_len) + { + factory->versioned_progid[progid_len++] = '.'; + wcscpy(factory->versioned_progid + progid_len, factory->version); + } + else factory->versioned_progid[progid_len] = 0; + } + + return expect_end_element(factory); } static HRESULT parse_scriptlet_public(struct scriptlet_factory *factory) @@ -305,6 +438,11 @@ static ULONG WINAPI scriptlet_factory_Release(IClassFactory *iface) { if (This->moniker) IMoniker_Release(This->moniker); if (This->xml_reader) IXmlReader_Release(This->xml_reader); + heap_free(This->classid_str); + heap_free(This->description); + heap_free(This->versioned_progid); + heap_free(This->progid); + heap_free(This->version); heap_free(This); } return ref;

1 0

Jacek Caban : scrobj: Add beginning scriptlet parser implementation.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: f2fdbc3d86a8f282bff023411c74ed4a95fe44cd URL: https://source.winehq.org/git/wine.git/?a=commit;h=f2fdbc3d86a8f282bff02341… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Sep 19 17:28:44 2019 +0200 scrobj: Add beginning scriptlet parser implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/scrobj/Makefile.in | 2 +- dlls/scrobj/scrobj.c | 150 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 151 insertions(+), 1 deletion(-) diff --git a/dlls/scrobj/Makefile.in b/dlls/scrobj/Makefile.in index b6965c635a..15a4032401 100644 --- a/dlls/scrobj/Makefile.in +++ b/dlls/scrobj/Makefile.in @@ -1,5 +1,5 @@ MODULE = scrobj.dll -IMPORTS = uuid ole32 oleaut32 advapi32 +IMPORTS = uuid urlmon ole32 oleaut32 xmllite advapi32 EXTRADLLFLAGS = -mno-cygwin diff --git a/dlls/scrobj/scrobj.c b/dlls/scrobj/scrobj.c index 87c073eb46..2bc746a179 100644 --- a/dlls/scrobj/scrobj.c +++ b/dlls/scrobj/scrobj.c @@ -27,6 +27,7 @@ #include "initguid.h" #include "scrobj.h" +#include "xmllite.h" #include "wine/debug.h" #include "wine/heap.h" @@ -38,7 +39,11 @@ static HINSTANCE scrobj_instance; struct scriptlet_factory { IClassFactory IClassFactory_iface; + LONG ref; + + IXmlReader *xml_reader; + IMoniker *moniker; }; typedef enum tid_t @@ -116,6 +121,139 @@ static void release_typelib(void) ITypeLib_Release(typelib); } +static const char *debugstr_xml_name(struct scriptlet_factory *factory) +{ + const WCHAR *str; + UINT len; + HRESULT hres; + hres = IXmlReader_GetLocalName(factory->xml_reader, &str, &len); + if (FAILED(hres)) return "#err"; + return debugstr_wn(str, len); +} + +static HRESULT next_xml_node(struct scriptlet_factory *factory, XmlNodeType *node_type) +{ + HRESULT hres; + do hres = IXmlReader_Read(factory->xml_reader, node_type); + while (hres == S_OK && *node_type == XmlNodeType_Whitespace); + return hres; +} + +static BOOL is_xml_name(struct scriptlet_factory *factory, const WCHAR *name) +{ + const WCHAR *qname; + UINT len; + HRESULT hres; + hres = IXmlReader_GetQualifiedName(factory->xml_reader, &qname, &len); + return hres == S_OK && len == wcslen(name) && !memcmp(qname, name, len * sizeof(WCHAR)); +} + +static HRESULT expect_no_attributes(struct scriptlet_factory *factory) +{ + UINT count; + HRESULT hres; + hres = IXmlReader_GetAttributeCount(factory->xml_reader, &count); + if (FAILED(hres)) return hres; + if (!count) return S_OK; + FIXME("Unexpected attributes\n"); + return E_FAIL; +} + +static HRESULT parse_scriptlet_registration(struct scriptlet_factory *factory) +{ + FIXME("\n"); + return E_NOTIMPL; +} + +static HRESULT parse_scriptlet_public(struct scriptlet_factory *factory) +{ + FIXME("\n"); + return E_NOTIMPL; +} + +static HRESULT parse_scriptlet_script(struct scriptlet_factory *factory) +{ + FIXME("\n"); + return E_NOTIMPL; +} + +static HRESULT parse_scriptlet_file(struct scriptlet_factory *factory, const WCHAR *url) +{ + XmlNodeType node_type; + IBindCtx *bind_ctx; + IStream *stream; + HRESULT hres; + + hres = CreateURLMoniker(NULL, url, &factory->moniker); + if (FAILED(hres)) + { + WARN("CreateURLMoniker failed: %08x\n", hres); + return hres; + } + + hres = CreateBindCtx(0, &bind_ctx); + if(FAILED(hres)) + return hres; + + hres = IMoniker_BindToStorage(factory->moniker, bind_ctx, NULL, &IID_IStream, (void**)&stream); + IBindCtx_Release(bind_ctx); + if (FAILED(hres)) + return hres; + + hres = CreateXmlReader(&IID_IXmlReader, (void**)&factory->xml_reader, NULL); + if(SUCCEEDED(hres)) + hres = IXmlReader_SetInput(factory->xml_reader, (IUnknown*)stream); + IStream_Release(stream); + if (FAILED(hres)) + return hres; + + hres = next_xml_node(factory, &node_type); + if (hres == S_OK && node_type == XmlNodeType_XmlDeclaration) + hres = next_xml_node(factory, &node_type); + + if (node_type != XmlNodeType_Element || !is_xml_name(factory, L"component")) + { + FIXME("Unexpected %s element\n", debugstr_xml_name(factory)); + return E_FAIL; + } + + hres = expect_no_attributes(factory); + if (FAILED(hres)) return hres; + + for (;;) + { + hres = next_xml_node(factory, &node_type); + if (FAILED(hres)) return hres; + if (node_type == XmlNodeType_EndElement) break; + if (node_type != XmlNodeType_Element) + { + FIXME("Unexpected node type %u\n", node_type); + return E_FAIL; + } + + if (is_xml_name(factory, L"registration")) + hres = parse_scriptlet_registration(factory); + else if (is_xml_name(factory, L"public")) + hres = parse_scriptlet_public(factory); + else if (is_xml_name(factory, L"script")) + hres = parse_scriptlet_script(factory); + else + { + FIXME("Unexpected element %s\n", debugstr_xml_name(factory)); + return E_NOTIMPL; + } + if (FAILED(hres)) return hres; + } + + hres = next_xml_node(factory, &node_type); + if (hres != S_FALSE) + { + FIXME("Unexpected node type %u\n", node_type); + return E_FAIL; + } + return S_OK; +} + static inline struct scriptlet_factory *impl_from_IClassFactory(IClassFactory *iface) { return CONTAINING_RECORD(iface, struct scriptlet_factory, IClassFactory_iface); @@ -164,7 +302,11 @@ static ULONG WINAPI scriptlet_factory_Release(IClassFactory *iface) TRACE("(%p) ref=%d\n", This, ref); if (!ref) + { + if (This->moniker) IMoniker_Release(This->moniker); + if (This->xml_reader) IXmlReader_Release(This->xml_reader); heap_free(This); + } return ref; } @@ -194,6 +336,7 @@ static const struct IClassFactoryVtbl scriptlet_factory_vtbl = static HRESULT create_scriptlet_factory(const WCHAR *url, struct scriptlet_factory **ret) { struct scriptlet_factory *factory; + HRESULT hres; TRACE("%s\n", debugstr_w(url)); @@ -206,6 +349,13 @@ static HRESULT create_scriptlet_factory(const WCHAR *url, struct scriptlet_facto factory->IClassFactory_iface.lpVtbl = &scriptlet_factory_vtbl; factory->ref = 1; + hres = parse_scriptlet_file(factory, url); + if (FAILED(hres)) + { + IClassFactory_Release(&factory->IClassFactory_iface); + return hres; + } + *ret = factory; return S_OK; }

1 0

Jacek Caban : scrobj: Add scriptlet factory stub implementation.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: 1f848e760494b836d95d4a743b9be01ce63a8b00 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1f848e760494b836d95d4a74… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Sep 19 17:28:25 2019 +0200 scrobj: Add scriptlet factory stub implementation. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/scrobj/Makefile.in | 2 +- dlls/scrobj/scrobj.c | 129 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 130 insertions(+), 1 deletion(-) diff --git a/dlls/scrobj/Makefile.in b/dlls/scrobj/Makefile.in index b175b54fe6..b6965c635a 100644 --- a/dlls/scrobj/Makefile.in +++ b/dlls/scrobj/Makefile.in @@ -1,5 +1,5 @@ MODULE = scrobj.dll -IMPORTS = uuid ole32 oleaut32 +IMPORTS = uuid ole32 oleaut32 advapi32 EXTRADLLFLAGS = -mno-cygwin diff --git a/dlls/scrobj/scrobj.c b/dlls/scrobj/scrobj.c index c27def2b0f..87c073eb46 100644 --- a/dlls/scrobj/scrobj.c +++ b/dlls/scrobj/scrobj.c @@ -27,12 +27,20 @@ #include "initguid.h" #include "scrobj.h" + #include "wine/debug.h" +#include "wine/heap.h" WINE_DEFAULT_DEBUG_CHANNEL(scrobj); static HINSTANCE scrobj_instance; +struct scriptlet_factory +{ + IClassFactory IClassFactory_iface; + LONG ref; +}; + typedef enum tid_t { NULL_tid, @@ -108,6 +116,100 @@ static void release_typelib(void) ITypeLib_Release(typelib); } +static inline struct scriptlet_factory *impl_from_IClassFactory(IClassFactory *iface) +{ + return CONTAINING_RECORD(iface, struct scriptlet_factory, IClassFactory_iface); +} + +static HRESULT WINAPI scriptlet_factory_QueryInterface(IClassFactory *iface, REFIID riid, void **ppv) +{ + struct scriptlet_factory *This = impl_from_IClassFactory(iface); + + if (IsEqualGUID(&IID_IUnknown, riid)) + { + TRACE("(%p)->(IID_IUnknown %p)\n", iface, ppv); + *ppv = &This->IClassFactory_iface; + } + else if (IsEqualGUID(&IID_IClassFactory, riid)) + { + TRACE("(%p)->(IID_IClassFactory %p)\n", iface, ppv); + *ppv = &This->IClassFactory_iface; + } + else + { + WARN("(%p)->(%s %p)\n", iface, debugstr_guid(riid), ppv); + *ppv = NULL; + return E_NOINTERFACE; + } + + IUnknown_AddRef((IUnknown *)*ppv); + return S_OK; +} + +static ULONG WINAPI scriptlet_factory_AddRef(IClassFactory *iface) +{ + struct scriptlet_factory *This = impl_from_IClassFactory(iface); + ULONG ref = InterlockedIncrement(&This->ref); + + TRACE("(%p) ref=%d\n", This, ref); + + return ref; +} + +static ULONG WINAPI scriptlet_factory_Release(IClassFactory *iface) +{ + struct scriptlet_factory *This = impl_from_IClassFactory(iface); + ULONG ref = InterlockedDecrement(&This->ref); + + TRACE("(%p) ref=%d\n", This, ref); + + if (!ref) + heap_free(This); + return ref; +} + +static HRESULT WINAPI scriptlet_factory_CreateInstance(IClassFactory *iface, IUnknown *outer, REFIID riid, void **ppv) +{ + struct scriptlet_factory *This = impl_from_IClassFactory(iface); + FIXME("(%p)->(%p %s %p)\n", This, outer, debugstr_guid(riid), ppv); + return E_NOTIMPL; +} + +static HRESULT WINAPI scriptlet_factory_LockServer(IClassFactory *iface, BOOL fLock) +{ + struct scriptlet_factory *This = impl_from_IClassFactory(iface); + TRACE("(%p)->(%x)\n", This, fLock); + return S_OK; +} + +static const struct IClassFactoryVtbl scriptlet_factory_vtbl = +{ + scriptlet_factory_QueryInterface, + scriptlet_factory_AddRef, + scriptlet_factory_Release, + scriptlet_factory_CreateInstance, + scriptlet_factory_LockServer +}; + +static HRESULT create_scriptlet_factory(const WCHAR *url, struct scriptlet_factory **ret) +{ + struct scriptlet_factory *factory; + + TRACE("%s\n", debugstr_w(url)); + + if (!(factory = heap_alloc_zero(sizeof(*factory)))) + { + IClassFactory_Release(&factory->IClassFactory_iface); + return E_OUTOFMEMORY; + } + + factory->IClassFactory_iface.lpVtbl = &scriptlet_factory_vtbl; + factory->ref = 1; + + *ret = factory; + return S_OK; +} + struct scriptlet_typelib { IGenScriptletTLib IGenScriptletTLib_iface; @@ -544,12 +646,39 @@ static IClassFactory scriptlet_typelib_factory = { &scriptlet_typelib_factory_vt */ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID riid, void **ppv) { + WCHAR key_name[128], *p; + LONG size = 0; + LSTATUS status; + if (IsEqualGUID(&CLSID_TypeLib, rclsid)) { TRACE("(Scriptlet.TypeLib %s %p)\n", debugstr_guid(riid), ppv); return IClassFactory_QueryInterface(&scriptlet_typelib_factory, riid, ppv); } + wcscpy(key_name, L"CLSID\\"); + p = key_name + wcslen(key_name); + p += StringFromGUID2(rclsid, p, ARRAY_SIZE(key_name) - (p - key_name)) - 1; + wcscpy(p, L"\\ScriptletURL"); + status = RegQueryValueW(HKEY_CLASSES_ROOT, key_name, NULL, &size); + if (!status) + { + struct scriptlet_factory *factory; + WCHAR *url; + HRESULT hres; + + if (!(url = heap_alloc(size * sizeof(WCHAR)))) return E_OUTOFMEMORY; + status = RegQueryValueW(HKEY_CLASSES_ROOT, key_name, url, &size); + + hres = create_scriptlet_factory(url, &factory); + heap_free(url); + if (FAILED(hres)) return hres; + + hres = IClassFactory_QueryInterface(&factory->IClassFactory_iface, riid, ppv); + IClassFactory_Release(&factory->IClassFactory_iface); + return hres; + } + FIXME("%s %s %p\n", debugstr_guid(rclsid), debugstr_guid(riid), ppv); return CLASS_E_CLASSNOTAVAILABLE; }

1 0

Alistair Leslie-Hughes : wininet: Clear cached credentials in INTERNET_OPTION_END_BROWSER_SESSION.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: f8f98e6edc0229b0663dfdc84df528885a765e79 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f8f98e6edc0229b0663dfdc8… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Thu Sep 19 00:47:44 2019 +0000 wininet: Clear cached credentials in INTERNET_OPTION_END_BROWSER_SESSION. Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wininet/http.c | 31 +++++++++++++++++ dlls/wininet/internet.c | 1 + dlls/wininet/internet.h | 1 + dlls/wininet/tests/http.c | 87 +++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 120 insertions(+) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 5844a7c7d6..60dab62441 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -1019,6 +1019,37 @@ static void cache_authorization(LPWSTR host, LPWSTR scheme, LeaveCriticalSection(&authcache_cs); } +void free_authorization_cache(void) +{ + authorizationData *ad, *sa_safe; + basicAuthorizationData *basic, *basic_safe; + + EnterCriticalSection(&authcache_cs); + + LIST_FOR_EACH_ENTRY_SAFE(basic, basic_safe, &basicAuthorizationCache, basicAuthorizationData, entry) + { + heap_free(basic->host); + heap_free(basic->realm); + heap_free(basic->authorization); + + list_remove(&basic->entry); + heap_free(basic); + } + + LIST_FOR_EACH_ENTRY_SAFE(ad, sa_safe, &authorizationCache, authorizationData, entry) + { + heap_free(ad->host); + heap_free(ad->scheme); + heap_free(ad->user); + heap_free(ad->password); + heap_free(ad->domain); + list_remove(&ad->entry); + heap_free(ad); + } + + LeaveCriticalSection(&authcache_cs); +} + static BOOL HTTP_DoAuthorization( http_request_t *request, LPCWSTR pszAuthValue, struct HttpAuthInfo **ppAuthInfo, LPWSTR domain_and_username, LPWSTR password, diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c index b7aa5fe330..266100861d 100644 --- a/dlls/wininet/internet.c +++ b/dlls/wininet/internet.c @@ -2821,6 +2821,7 @@ BOOL WINAPI InternetSetOptionW(HINTERNET hInternet, DWORD dwOption, case INTERNET_OPTION_END_BROWSER_SESSION: FIXME("Option INTERNET_OPTION_END_BROWSER_SESSION: semi-stub\n"); free_cookie(); + free_authorization_cache(); break; case INTERNET_OPTION_CONNECTED_STATE: FIXME("Option INTERNET_OPTION_CONNECTED_STATE: STUB\n"); diff --git a/dlls/wininet/internet.h b/dlls/wininet/internet.h index 40e51bec32..8954d071f9 100644 --- a/dlls/wininet/internet.h +++ b/dlls/wininet/internet.h @@ -456,6 +456,7 @@ static inline req_file_t *req_file_addref(req_file_t *req_file) BOOL init_urlcache(void) DECLSPEC_HIDDEN; void free_urlcache(void) DECLSPEC_HIDDEN; void free_cookie(void) DECLSPEC_HIDDEN; +void free_authorization_cache(void) DECLSPEC_HIDDEN; void init_winsock(void) DECLSPEC_HIDDEN; diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c index b61d5c7030..f1a647346e 100644 --- a/dlls/wininet/tests/http.c +++ b/dlls/wininet/tests/http.c @@ -4502,6 +4502,92 @@ static void test_basic_auth_credentials_reuse(int port) InternetCloseHandle( ses ); } +static void test_basic_auth_credentials_end_session(int port) +{ + HINTERNET ses, con, req; + DWORD status, size; + BOOL ret; + char buffer[0x40]; + + ses = InternetOpenA( "winetest", INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, 0 ); + ok( ses != NULL, "InternetOpenA failed\n" ); + + con = InternetConnectA( ses, "localhost", port, "user", "pwd", + INTERNET_SERVICE_HTTP, 0, 0 ); + ok( con != NULL, "InternetConnectA failed %u\n", GetLastError() ); + + req = HttpOpenRequestA( con, "HEAD", "/upload.txt", NULL, NULL, NULL, 0, 0 ); + ok( req != NULL, "HttpOpenRequestA failed %u\n", GetLastError() ); + + ret = HttpSendRequestA( req, NULL, 0, NULL, 0 ); + ok( ret, "HttpSendRequestA failed %u\n", GetLastError() ); + + size = sizeof(buffer); + SetLastError(0xdeadbeef); + ret = InternetQueryOptionA(req, INTERNET_OPTION_USERNAME, buffer, &size); + ok(ret, "unexpected failure %u\n", GetLastError()); + ok(!strcmp(buffer, "user"), "got %s\n", buffer); + ok(size == 4, "got %u\n", size); + + size = sizeof(buffer); + SetLastError(0xdeadbeef); + ret = InternetQueryOptionA(req, INTERNET_OPTION_PASSWORD, buffer, &size); + ok(ret, "unexpected failure %u\n", GetLastError()); + ok(!strcmp(buffer, "pwd"), "got %s\n", buffer); + ok(size == 3, "got %u\n", size); + + status = 0xdeadbeef; + size = sizeof(status); + ret = HttpQueryInfoA( req, HTTP_QUERY_STATUS_CODE|HTTP_QUERY_FLAG_NUMBER, &status, &size, NULL ); + ok( ret, "HttpQueryInfoA failed %u\n", GetLastError() ); + ok( status == HTTP_STATUS_OK, "got %u\n", status ); + + InternetCloseHandle( req ); + InternetCloseHandle( con ); + InternetCloseHandle( ses ); + + ses = InternetOpenA( "winetest", INTERNET_OPEN_TYPE_DIRECT, NULL, NULL, 0 ); + ok( ses != NULL, "InternetOpenA failed\n" ); + + /* Clear the cached credentials */ + ret = InternetSetOptionA(ses, INTERNET_OPTION_END_BROWSER_SESSION, NULL, 0); + ok(ret, "unexpected failure %u\n", GetLastError()); + + con = InternetConnectA( ses, "localhost", port, NULL, NULL, + INTERNET_SERVICE_HTTP, 0, 0 ); + ok( con != NULL, "InternetConnectA failed %u\n", GetLastError() ); + + req = HttpOpenRequestA( con, "PUT", "/upload2.txt", NULL, NULL, NULL, 0, 0 ); + ok( req != NULL, "HttpOpenRequestA failed %u\n", GetLastError() ); + + ret = HttpSendRequestA( req, NULL, 0, NULL, 0 ); + ok( ret, "HttpSendRequestA failed %u\n", GetLastError() ); + + size = sizeof(buffer); + SetLastError(0xdeadbeef); + ret = InternetQueryOptionA(req, INTERNET_OPTION_USERNAME, buffer, &size); + ok(ret, "unexpected failure %u\n", GetLastError()); + ok(!strcmp(buffer, ""), "got %s\n", buffer); + ok(size == 0, "got %u\n", size); + + size = sizeof(buffer); + SetLastError(0xdeadbeef); + ret = InternetQueryOptionA(req, INTERNET_OPTION_PASSWORD, buffer, &size); + ok(ret, "unexpected failure %u\n", GetLastError()); + ok(!strcmp(buffer, ""), "got %s\n", buffer); + ok(size == 0, "got %u\n", size); + + status = 0xdeadbeef; + size = sizeof(status); + ret = HttpQueryInfoA( req, HTTP_QUERY_STATUS_CODE|HTTP_QUERY_FLAG_NUMBER, &status, &size, NULL ); + ok( ret, "HttpQueryInfoA failed %u\n", GetLastError() ); + ok( status == HTTP_STATUS_BAD_REQUEST, "got %u\n", status ); + + InternetCloseHandle( req ); + InternetCloseHandle( con ); + InternetCloseHandle( ses ); +} + static void test_async_read(int port) { HINTERNET ses, con, req; @@ -5696,6 +5782,7 @@ static void test_http_connection(void) test_request_content_length(si.port); test_accept_encoding(si.port); test_basic_auth_credentials_reuse(si.port); + test_basic_auth_credentials_end_session(si.port); test_async_read(si.port); test_http_read(si.port); test_connection_break(si.port);

1 0

Zebediah Figura : winex11: Track the client colormap separately.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: 22993aff94ad4b3539ed99648f8180cfc7b9253c URL: https://source.winehq.org/git/wine.git/?a=commit;h=22993aff94ad4b3539ed9964… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Sep 18 21:44:19 2019 -0500 winex11: Track the client colormap separately. If a window with an OpenGL surface attached is reparented as a child window, and then reparented as a top-level window, so that its whole window is destroyed and then recreated, it will be recreated with the colormap of its child window, which more than likely has a different visual. This violates the X11 specification, which states that a window's colormap must have the same visual as the window itself, and causes the X server to return BadMatch to the CreateWindow request. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winex11.drv/window.c | 21 +++++++++++---------- dlls/winex11.drv/x11drv.h | 3 ++- 2 files changed, 13 insertions(+), 11 deletions(-) diff --git a/dlls/winex11.drv/window.c b/dlls/winex11.drv/window.c index e1f1d30170..4e01eb201c 100644 --- a/dlls/winex11.drv/window.c +++ b/dlls/winex11.drv/window.c @@ -314,7 +314,7 @@ static unsigned long get_mwm_decorations( struct x11drv_win_data *data, static int get_window_attributes( struct x11drv_win_data *data, XSetWindowAttributes *attr ) { attr->override_redirect = !data->managed; - attr->colormap = data->colormap ? data->colormap : default_colormap; + attr->colormap = data->whole_colormap ? data->whole_colormap : default_colormap; attr->save_under = ((GetClassLongW( data->hwnd, GCL_STYLE ) & CS_SAVEBITS) != 0); attr->bit_gravity = NorthWestGravity; attr->backing_store = NotUseful; @@ -1445,12 +1445,12 @@ Window create_client_window( HWND hwnd, const XVisualInfo *visual ) TRACE( "%p reparent xwin %lx/%lx\n", data->hwnd, data->whole_window, data->client_window ); } - if (data->colormap) XFreeColormap( gdi_display, data->colormap ); - data->colormap = XCreateColormap( gdi_display, dummy_parent, visual->visual, - (visual->class == PseudoColor || - visual->class == GrayScale || - visual->class == DirectColor) ? AllocAll : AllocNone ); - attr.colormap = data->colormap; + if (data->client_colormap) XFreeColormap( gdi_display, data->client_colormap ); + data->client_colormap = XCreateColormap( gdi_display, dummy_parent, visual->visual, + (visual->class == PseudoColor || + visual->class == GrayScale || + visual->class == DirectColor) ? AllocAll : AllocNone ); + attr.colormap = data->client_colormap; attr.bit_gravity = NorthWestGravity; attr.win_gravity = NorthWestGravity; attr.backing_store = NotUseful; @@ -1509,7 +1509,7 @@ static void create_whole_window( struct x11drv_win_data *data ) data->shaped = (win_rgn != 0); if (data->vis.visualid != default_visual.visualid) - data->colormap = XCreateColormap( data->display, root_window, data->vis.visual, AllocNone ); + data->whole_colormap = XCreateColormap( data->display, root_window, data->vis.visual, AllocNone ); mask = get_window_attributes( data, &attr ); @@ -1586,9 +1586,9 @@ static void destroy_whole_window( struct x11drv_win_data *data, BOOL already_des XDeleteContext( data->display, data->whole_window, winContext ); if (!already_destroyed) XDestroyWindow( data->display, data->whole_window ); } - if (data->colormap) XFreeColormap( data->display, data->colormap ); + if (data->whole_colormap) XFreeColormap( data->display, data->whole_colormap ); data->whole_window = data->client_window = 0; - data->colormap = 0; + data->whole_colormap = 0; data->wm_state = WithdrawnState; data->net_wm_state = 0; data->mapped = FALSE; @@ -1694,6 +1694,7 @@ void CDECL X11DRV_DestroyWindow( HWND hwnd ) if (thread_data->last_xic_hwnd == hwnd) thread_data->last_xic_hwnd = 0; if (data->icon_pixmap) XFreePixmap( gdi_display, data->icon_pixmap ); if (data->icon_mask) XFreePixmap( gdi_display, data->icon_mask ); + if (data->client_colormap) XFreeColormap( data->display, data->client_colormap ); HeapFree( GetProcessHeap(), 0, data->icon_bits ); XDeleteContext( gdi_display, (XID)hwnd, win_data_context ); release_win_data( data ); diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h index 9ac83e57b1..e71156cbb0 100644 --- a/dlls/winex11.drv/x11drv.h +++ b/dlls/winex11.drv/x11drv.h @@ -553,7 +553,8 @@ struct x11drv_win_data { Display *display; /* display connection for the thread owning the window */ XVisualInfo vis; /* X visual used by this window */ - Colormap colormap; /* colormap if non-default visual */ + Colormap whole_colormap; /* colormap if non-default visual */ + Colormap client_colormap; /* colormap for the client window */ HWND hwnd; /* hwnd that this private data belongs to */ Window whole_window; /* X window for the complete window */ Window client_window; /* X window for the client area */

1 0

Zebediah Figura : winegstreamer: Introduce a callback for filter-specific initialization.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: 41fc456498c90a616ad336411e188dd37bd9dda0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=41fc456498c90a616ad33641… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Sep 18 21:37:02 2019 -0500 winegstreamer: Introduce a callback for filter-specific initialization. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/gstdemux.c | 119 ++++++++++++++++++++++++------------------ 1 file changed, 67 insertions(+), 52 deletions(-) diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c index a0c8bf6102..87a1deb6f9 100644 --- a/dlls/winegstreamer/gstdemux.c +++ b/dlls/winegstreamer/gstdemux.c @@ -68,6 +68,8 @@ struct gstdemux HANDLE no_more_pads_event; HANDLE push_thread; + + BOOL (*init_gst)(struct gstdemux *filter); }; struct gstdemux_source @@ -1068,14 +1070,12 @@ static void unknown_type(GstElement *bin, GstPad *pad, GstCaps *caps, gpointer u static HRESULT GST_Connect(struct gstdemux *This, IPin *pConnectPin, ALLOCATOR_PROPERTIES *props) { - int ret, i; - LONGLONG avail, duration; + LONGLONG avail; GstStaticPadTemplate src_template = GST_STATIC_PAD_TEMPLATE( "quartz_src", GST_PAD_SRC, GST_PAD_ALWAYS, GST_STATIC_CAPS_ANY); - GstElement *gstfilter; This->props = *props; IAsyncReader_Length(This->reader, &This->filesize, &avail); @@ -1088,68 +1088,21 @@ static HRESULT GST_Connect(struct gstdemux *This, IPin *pConnectPin, ALLOCATOR_P This->container = gst_bin_new(NULL); gst_element_set_bus(This->container, This->bus); - gstfilter = gst_element_factory_make("decodebin", NULL); - if (!gstfilter) { - ERR("Could not make source filter, are gstreamer-plugins-* installed for %u bits?\n", - 8 * (int)sizeof(void*)); - return E_FAIL; - } - - gst_bin_add(GST_BIN(This->container), gstfilter); - - g_signal_connect(gstfilter, "pad-added", G_CALLBACK(existing_new_pad_wrapper), This); - g_signal_connect(gstfilter, "pad-removed", G_CALLBACK(removed_decoded_pad_wrapper), This); - g_signal_connect(gstfilter, "autoplug-select", G_CALLBACK(autoplug_blacklist_wrapper), This); - g_signal_connect(gstfilter, "unknown-type", G_CALLBACK(unknown_type_wrapper), This); - This->my_src = gst_pad_new_from_static_template(&src_template, "quartz-src"); gst_pad_set_getrange_function(This->my_src, request_buffer_src_wrapper); gst_pad_set_query_function(This->my_src, query_function_wrapper); gst_pad_set_activatemode_function(This->my_src, activate_mode_wrapper); gst_pad_set_event_function(This->my_src, event_src_wrapper); gst_pad_set_element_private (This->my_src, This); - This->their_sink = gst_element_get_static_pad(gstfilter, "sink"); - g_signal_connect(gstfilter, "no-more-pads", G_CALLBACK(no_more_pads_wrapper), This); - ret = gst_pad_link(This->my_src, This->their_sink); - if (ret < 0) { - ERR("Returns: %i\n", ret); - return E_FAIL; - } This->start = This->nextofs = This->nextpullofs = This->stop = 0; - /* Add initial pins */ This->initial = TRUE; - ResetEvent(This->no_more_pads_event); - gst_element_set_state(This->container, GST_STATE_PLAYING); - ret = gst_element_get_state(This->container, NULL, NULL, -1); - - if (ret == GST_STATE_CHANGE_FAILURE) - { - ERR("GStreamer failed to play stream\n"); + if (!This->init_gst(This)) return E_FAIL; - } - - WaitForSingleObject(This->no_more_pads_event, INFINITE); - - gst_pad_query_duration(This->ppPins[0]->their_src, GST_FORMAT_TIME, &duration); - for (i = 0; i < This->cStreams; ++i) - { - This->ppPins[i]->seek.llDuration = This->ppPins[i]->seek.llStop = duration / 100; - This->ppPins[i]->seek.llCurrent = 0; - if (!This->ppPins[i]->seek.llDuration) - This->ppPins[i]->seek.dwCapabilities = 0; - WaitForSingleObject(This->ppPins[i]->caps_event, INFINITE); - } - *props = This->props; - - This->ignore_flush = TRUE; - gst_element_set_state(This->container, GST_STATE_READY); - gst_element_get_state(This->container, NULL, NULL, -1); - This->ignore_flush = FALSE; - This->initial = FALSE; + *props = This->props; This->nextofs = This->nextpullofs = 0; return S_OK; } @@ -1223,6 +1176,67 @@ static const BasePinFuncTable sink_ops = .pfnGetMediaType = BasePinImpl_GetMediaType, }; +static BOOL gstdecoder_init_gst(struct gstdemux *filter) +{ + GstElement *element = gst_element_factory_make("decodebin", NULL); + LONGLONG duration; + unsigned int i; + int ret; + + if (!element) + { + ERR("Failed to create decodebin; are %u-bit GStreamer \"base\" plugins installed?\n", + 8 * (int)sizeof(void*)); + return FALSE; + } + + gst_bin_add(GST_BIN(filter->container), element); + + g_signal_connect(element, "pad-added", G_CALLBACK(existing_new_pad_wrapper), filter); + g_signal_connect(element, "pad-removed", G_CALLBACK(removed_decoded_pad_wrapper), filter); + g_signal_connect(element, "autoplug-select", G_CALLBACK(autoplug_blacklist_wrapper), filter); + g_signal_connect(element, "unknown-type", G_CALLBACK(unknown_type_wrapper), filter); + g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads_wrapper), filter); + + filter->their_sink = gst_element_get_static_pad(element, "sink"); + ResetEvent(filter->no_more_pads_event); + + if ((ret = gst_pad_link(filter->my_src, filter->their_sink)) < 0) + { + ERR("Failed to link pads, error %d.\n", ret); + return FALSE; + } + + gst_element_set_state(filter->container, GST_STATE_PLAYING); + ret = gst_element_get_state(filter->container, NULL, NULL, -1); + if (ret == GST_STATE_CHANGE_FAILURE) + { + ERR("Failed to play stream.\n"); + return FALSE; + } + + WaitForSingleObject(filter->no_more_pads_event, INFINITE); + + gst_pad_query_duration(filter->ppPins[0]->their_src, GST_FORMAT_TIME, &duration); + for (i = 0; i < filter->cStreams; ++i) + { + struct gstdemux_source *pin = filter->ppPins[i]; + + pin->seek.llDuration = pin->seek.llStop = duration / 100; + pin->seek.llCurrent = 0; + if (!pin->seek.llDuration) + pin->seek.dwCapabilities = 0; + WaitForSingleObject(pin->caps_event, INFINITE); + } + + filter->ignore_flush = TRUE; + gst_element_set_state(filter->container, GST_STATE_READY); + gst_element_get_state(filter->container, NULL, NULL, -1); + filter->ignore_flush = FALSE; + + return TRUE; +} + IUnknown * CALLBACK Gstreamer_Splitter_create(IUnknown *outer, HRESULT *phr) { struct gstdemux *object; @@ -1249,6 +1263,7 @@ IUnknown * CALLBACK Gstreamer_Splitter_create(IUnknown *outer, HRESULT *phr) lstrcpynW(object->sink.name, wcsInputPinName, ARRAY_SIZE(object->sink.name)); object->sink.IPin_iface.lpVtbl = &GST_InputPin_Vtbl; object->sink.pFuncsTable = &sink_ops; + object->init_gst = gstdecoder_init_gst; *phr = S_OK; TRACE("Created GStreamer demuxer %p.\n", object);

1 0

Jeff Smith : kernelbase: Fix PathAllocCanonicalize handling segments that contain dots.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: c13a8f60bdef5b69f5db61f79efb7fd341a27303 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c13a8f60bdef5b69f5db61f7… Author: Jeff Smith <whydoubt(a)gmail.com> Date: Thu Sep 19 12:04:30 2019 +0800 kernelbase: Fix PathAllocCanonicalize handling segments that contain dots. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47766 Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Jeff Smith <whydoubt(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/path.c | 35 ++++++++++++++++++++--------------- 1 file changed, 20 insertions(+), 15 deletions(-) diff --git a/dlls/kernelbase/path.c b/dlls/kernelbase/path.c index 82708be544..e9182a764d 100644 --- a/dlls/kernelbase/path.c +++ b/dlls/kernelbase/path.c @@ -276,13 +276,8 @@ HRESULT WINAPI PathAllocCanonicalize(const WCHAR *path_in, DWORD flags, WCHAR ** continue; } - /* Keep the . if one of the following is true: - * 1. PATHCCH_DO_NOT_NORMALIZE_SEGMENTS - * 2. in form of a..b - */ - if (dst > buffer - && (((flags & PATHCCH_DO_NOT_NORMALIZE_SEGMENTS) && dst[-1] != '\\') - || (dst[-1] != '\\' && src[2] != '\\' && src[2]))) + /* Keep the .. if not surrounded by \ */ + if ((src[2] != '\\' && src[2]) || (dst > buffer && dst[-1] != '\\')) { *dst++ = *src++; *dst++ = *src++; @@ -313,14 +308,8 @@ HRESULT WINAPI PathAllocCanonicalize(const WCHAR *path_in, DWORD flags, WCHAR ** } else { - /* Keep the . if one of the following is true: - * 1. PATHCCH_DO_NOT_NORMALIZE_SEGMENTS - * 2. in form of a.b, which is used in domain names - * 3. *. - */ - if (dst > buffer - && ((flags & PATHCCH_DO_NOT_NORMALIZE_SEGMENTS && dst[-1] != '\\') - || (dst[-1] != '\\' && src[1] != '\\' && src[1]) || (dst[-1] == '*'))) + /* Keep the . if not surrounded by \ */ + if ((src[1] != '\\' && src[1]) || (dst > buffer && dst[-1] != '\\')) { *dst++ = *src++; continue; @@ -352,6 +341,22 @@ HRESULT WINAPI PathAllocCanonicalize(const WCHAR *path_in, DWORD flags, WCHAR ** /* End the path */ *dst = 0; + /* Strip multiple trailing . */ + if (!(flags & PATHCCH_DO_NOT_NORMALIZE_SEGMENTS)) + { + while (dst > buffer && dst[-1] == '.') + { + /* Keep a . after * */ + if (dst - 1 > buffer && dst[-2] == '*') + break; + /* If . follow a : at the second character, remove the . and add a \ */ + else if (dst - 1 > buffer && dst[-2] == ':' && dst - 2 == buffer + 1) + *--dst = '\\'; + else + *--dst = 0; + } + } + /* If result path is empty, fill in \ */ if (!*buffer) {

1 0

Zebediah Figura : winegstreamer: Reimplement the WAVE parser ontop of the wavparse plugin.
by Alexandre Julliard Sept. 19, 2019

Sept. 19, 2019

Module: wine Branch: master Commit: 939638b05530b10493e7a7e6699a37696bdd0e27 URL: https://source.winehq.org/git/wine.git/?a=commit;h=939638b05530b10493e7a7e6… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Sep 18 21:37:03 2019 -0500 winegstreamer: Reimplement the WAVE parser ontop of the wavparse plugin. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/Makefile.in | 3 +- dlls/quartz/main.c | 1 - dlls/quartz/quartz_private.h | 1 - dlls/quartz/quartz_strmif.idl | 7 - dlls/quartz/regsvr.c | 19 -- dlls/quartz/tests/waveparser.c | 13 +- dlls/quartz/waveparser.c | 440 --------------------------------------- dlls/winegstreamer/gst_private.h | 1 + dlls/winegstreamer/gstdemux.c | 109 ++++++++++ dlls/winegstreamer/main.c | 50 +++++ 10 files changed, 173 insertions(+), 471 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=939638b05530b10493e7…

1 0