October 2018 - Wine-Commits

Hans Leidekker : wbemprox: Make name and value optional in IWbemClassObject::Next.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 492a9653f5e297bdc49d66c31435eb17f958868d URL: https://source.winehq.org/git/wine.git/?a=commit;h=492a9653f5e297bdc49d66c3… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Oct 9 15:25:03 2018 +0200 wbemprox: Make name and value optional in IWbemClassObject::Next. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wbemprox/class.c | 5 +++- dlls/wbemprox/query.c | 61 ++++++++++++++++++++++++++++++++------------- dlls/wbemprox/tests/query.c | 2 ++ 3 files changed, 49 insertions(+), 19 deletions(-) diff --git a/dlls/wbemprox/class.c b/dlls/wbemprox/class.c index 3f299ea..cc54fd5 100644 --- a/dlls/wbemprox/class.c +++ b/dlls/wbemprox/class.c @@ -532,8 +532,11 @@ static HRESULT WINAPI class_object_Next( SysFreeString( prop ); return hr; } + obj->index_property = i + 1; - *strName = prop; + if (strName) *strName = prop; + else SysFreeString( prop ); + return S_OK; } return WBEM_S_NO_MORE_DATA; diff --git a/dlls/wbemprox/query.c b/dlls/wbemprox/query.c index 8a51266..d193574 100644 --- a/dlls/wbemprox/query.c +++ b/dlls/wbemprox/query.c @@ -668,50 +668,71 @@ static HRESULT get_system_propval( const struct view *view, UINT index, const WC if (!strcmpiW( name, classW )) { - V_VT( ret ) = VT_BSTR; - V_BSTR( ret ) = build_classname( view ); + if (ret) + { + V_VT( ret ) = VT_BSTR; + V_BSTR( ret ) = build_classname( view ); + } if (type) *type = CIM_STRING; return S_OK; } if (!strcmpiW( name, genusW )) { - V_VT( ret ) = VT_I4; - V_I4( ret ) = WBEM_GENUS_INSTANCE; /* FIXME */ + if (ret) + { + V_VT( ret ) = VT_I4; + V_I4( ret ) = WBEM_GENUS_INSTANCE; /* FIXME */ + } if (type) *type = CIM_SINT32; return S_OK; } else if (!strcmpiW( name, namespaceW )) { - V_VT( ret ) = VT_BSTR; - V_BSTR( ret ) = build_namespace( view ); + if (ret) + { + V_VT( ret ) = VT_BSTR; + V_BSTR( ret ) = build_namespace( view ); + } if (type) *type = CIM_STRING; return S_OK; } else if (!strcmpiW( name, pathW )) { - V_VT( ret ) = VT_BSTR; - V_BSTR( ret ) = build_path( view, index, name ); + if (ret) + { + V_VT( ret ) = VT_BSTR; + V_BSTR( ret ) = build_path( view, index, name ); + } if (type) *type = CIM_STRING; return S_OK; } if (!strcmpiW( name, propcountW )) { - V_VT( ret ) = VT_I4; - V_I4( ret ) = count_selected_properties( view ); + if (ret) + { + V_VT( ret ) = VT_I4; + V_I4( ret ) = count_selected_properties( view ); + } if (type) *type = CIM_SINT32; return S_OK; } else if (!strcmpiW( name, relpathW )) { - V_VT( ret ) = VT_BSTR; - V_BSTR( ret ) = build_relpath( view, index, name ); + if (ret) + { + V_VT( ret ) = VT_BSTR; + V_BSTR( ret ) = build_relpath( view, index, name ); + } if (type) *type = CIM_STRING; return S_OK; } else if (!strcmpiW( name, serverW )) { - V_VT( ret ) = VT_BSTR; - V_BSTR( ret ) = build_servername( view ); + if (ret) + { + V_VT( ret ) = VT_BSTR; + V_BSTR( ret ) = build_servername( view ); + } if (type) *type = CIM_STRING; return S_OK; } @@ -835,6 +856,11 @@ HRESULT get_propval( const struct view *view, UINT index, const WCHAR *name, VAR hr = get_value( view->table, row, column, &val ); if (hr != S_OK) return hr; + if (type) *type = view->table->columns[column].type & COL_TYPE_MASK; + if (flavor) *flavor = 0; + + if (!ret) return S_OK; + vartype = view->table->columns[column].vartype; if (view->table->columns[column].type & CIM_FLAG_ARRAY) { @@ -843,8 +869,10 @@ HRESULT get_propval( const struct view *view, UINT index, const WCHAR *name, VAR val_ptr = to_safearray( (const struct array *)(INT_PTR)val, basetype ); if (!val_ptr) vartype = VT_NULL; else if (!vartype) vartype = to_vartype( basetype ) | VT_ARRAY; - goto done; + set_variant( vartype, val, val_ptr, ret ); + return S_OK; } + switch (view->table->columns[column].type & COL_TYPE_MASK) { case CIM_BOOLEAN: @@ -891,10 +919,7 @@ HRESULT get_propval( const struct view *view, UINT index, const WCHAR *name, VAR return WBEM_E_FAILED; } -done: set_variant( vartype, val, val_ptr, ret ); - if (type) *type = view->table->columns[column].type & COL_TYPE_MASK; - if (flavor) *flavor = 0; return S_OK; } diff --git a/dlls/wbemprox/tests/query.c b/dlls/wbemprox/tests/query.c index b2f5d68..387952d 100644 --- a/dlls/wbemprox/tests/query.c +++ b/dlls/wbemprox/tests/query.c @@ -1194,6 +1194,8 @@ static void test_Win32_OperatingSystem( IWbemServices *services ) hr = IWbemClassObject_BeginEnumeration( obj, 0 ); ok( hr == S_OK, "got %08x\n", hr ); + while (IWbemClassObject_Next( obj, 0, NULL, NULL, NULL, NULL ) == S_OK) {} + hr = IWbemClassObject_EndEnumeration( obj ); ok( hr == S_OK, "got %08x\n", hr );

1 0

Hans Leidekker : wbemdisp: Generate DISPIDs for object methods too.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 6db5a3a11ff26856a4ed8df3b64d1a21d53eb610 URL: https://source.winehq.org/git/wine.git/?a=commit;h=6db5a3a11ff26856a4ed8df3… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Oct 9 15:25:04 2018 +0200 wbemdisp: Generate DISPIDs for object methods too. Tests based on a patch by Alistair Leslie-Hughes. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=44474 Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wbemdisp/locator.c | 102 +++++++++++++++++++++++++++++++---------- dlls/wbemdisp/tests/wbemdisp.c | 35 ++++++++++++++ 2 files changed, 112 insertions(+), 25 deletions(-) diff --git a/dlls/wbemdisp/locator.c b/dlls/wbemdisp/locator.c index 1c7f56a..78b81d8 100644 --- a/dlls/wbemdisp/locator.c +++ b/dlls/wbemdisp/locator.c @@ -488,11 +488,13 @@ static HRESULT SWbemPropertySet_create( IWbemClassObject *wbem_object, ISWbemPro return S_OK; } -#define DISPID_BASE 0x1800000 +#define DISPID_BASE 0x1800000 +#define DISPID_BASE_METHOD 0x1000000 struct member { BSTR name; + BOOL is_method; DISPID dispid; }; @@ -504,6 +506,7 @@ struct object struct member *members; UINT nb_members; DISPID last_dispid; + DISPID last_dispid_method; }; static inline struct object *impl_from_ISWbemObject( @@ -585,41 +588,83 @@ static HRESULT WINAPI object_GetTypeInfo( static HRESULT init_members( struct object *object ) { - LONG bound, i; - SAFEARRAY *sa; + IWbemClassObject *sig_in, *sig_out; + LONG i = 0, count = 0; + BSTR name; HRESULT hr; if (object->members) return S_OK; - hr = IWbemClassObject_GetNames( object->object, NULL, 0, NULL, &sa ); - if (FAILED( hr )) return hr; - hr = SafeArrayGetUBound( sa, 1, &bound ); - if (FAILED( hr )) + hr = IWbemClassObject_BeginEnumeration( object->object, 0 ); + if (SUCCEEDED( hr )) { - SafeArrayDestroy( sa ); - return hr; + while (IWbemClassObject_Next( object->object, 0, NULL, NULL, NULL, NULL ) == S_OK) count++; + IWbemClassObject_EndEnumeration( object->object ); } - if (!(object->members = heap_alloc( sizeof(struct member) * (bound + 1) ))) + + hr = IWbemClassObject_BeginMethodEnumeration( object->object, 0 ); + if (SUCCEEDED( hr )) { - SafeArrayDestroy( sa ); - return E_OUTOFMEMORY; + while (IWbemClassObject_NextMethod( object->object, 0, &name, &sig_in, &sig_out ) == S_OK) + { + count++; + SysFreeString( name ); + IWbemClassObject_Release( sig_in ); + IWbemClassObject_Release( sig_out ); + } + IWbemClassObject_EndMethodEnumeration( object->object ); } - for (i = 0; i <= bound; i++) + + if (!(object->members = heap_alloc( sizeof(struct member) * count ))) return E_OUTOFMEMORY; + + hr = IWbemClassObject_BeginEnumeration( object->object, 0 ); + if (SUCCEEDED( hr )) { - hr = SafeArrayGetElement( sa, &i, &object->members[i].name ); - if (FAILED( hr )) + while (IWbemClassObject_Next( object->object, 0, &name, NULL, NULL, NULL ) == S_OK) { - for (i--; i >= 0; i--) SysFreeString( object->members[i].name ); - SafeArrayDestroy( sa ); - heap_free( object->members ); - object->members = NULL; - return E_OUTOFMEMORY; + object->members[i].name = name; + object->members[i].is_method = FALSE; + object->members[i].dispid = 0; + if (++i > count) + { + IWbemClassObject_EndEnumeration( object->object ); + goto error; + } + TRACE( "added property %s\n", debugstr_w(name) ); } - object->members[i].dispid = 0; + IWbemClassObject_EndEnumeration( object->object ); } - object->nb_members = bound + 1; - SafeArrayDestroy( sa ); + + hr = IWbemClassObject_BeginMethodEnumeration( object->object, 0 ); + if (SUCCEEDED( hr )) + { + while (IWbemClassObject_NextMethod( object->object, 0, &name, &sig_in, &sig_out ) == S_OK) + { + object->members[i].name = name; + object->members[i].is_method = TRUE; + object->members[i].dispid = 0; + if (++i > count) + { + IWbemClassObject_EndMethodEnumeration( object->object ); + goto error; + } + IWbemClassObject_Release( sig_in ); + IWbemClassObject_Release( sig_out ); + TRACE( "added method %s\n", debugstr_w(name) ); + } + IWbemClassObject_EndMethodEnumeration( object->object ); + } + + object->nb_members = count; + TRACE( "added %u members\n", object->nb_members ); return S_OK; + +error: + for (--i; i >= 0; i--) SysFreeString( object->members[i].name ); + heap_free( object->members ); + object->members = NULL; + object->nb_members = 0; + return E_FAIL; } static DISPID get_member_dispid( struct object *object, const WCHAR *name ) @@ -629,7 +674,13 @@ static DISPID get_member_dispid( struct object *object, const WCHAR *name ) { if (!strcmpiW( object->members[i].name, name )) { - if (!object->members[i].dispid) object->members[i].dispid = ++object->last_dispid; + if (!object->members[i].dispid) + { + if (object->members[i].is_method) + object->members[i].dispid = ++object->last_dispid_method; + else + object->members[i].dispid = ++object->last_dispid; + } return object->members[i].dispid; } } @@ -701,7 +752,7 @@ static HRESULT WINAPI object_Invoke( TRACE( "%p, %x, %s, %u, %x, %p, %p, %p, %p\n", object, member, debugstr_guid(riid), lcid, flags, params, result, excep_info, arg_err ); - if (member <= DISPID_BASE) + if (member <= DISPID_BASE_METHOD) { hr = get_typeinfo( ISWbemObject_tid, &typeinfo ); if (SUCCEEDED(hr)) @@ -1044,6 +1095,7 @@ static HRESULT SWbemObject_create( IWbemClassObject *wbem_object, ISWbemObject * object->members = NULL; object->nb_members = 0; object->last_dispid = DISPID_BASE; + object->last_dispid_method = DISPID_BASE_METHOD; *obj = &object->ISWbemObject_iface; TRACE( "returning iface %p\n", *obj ); diff --git a/dlls/wbemdisp/tests/wbemdisp.c b/dlls/wbemdisp/tests/wbemdisp.c index 1ce0456..68c0e3b 100644 --- a/dlls/wbemdisp/tests/wbemdisp.c +++ b/dlls/wbemdisp/tests/wbemdisp.c @@ -48,6 +48,11 @@ static void test_ParseDisplayName(void) static const WCHAR name4[] = {'w','i','n','m','g','m','t','s',':','\\','\\','.','\\','r','o','o','t','\\','c','i','m','v','2',':', 'W','i','n','3','2','_','S','e','r','v','i','c','e',0}; + static const WCHAR stdregprovW[] = + {'w','i','n','m','g','m','t','s',':','\\','\\','.','\\','r','o','o','t','\\','d','e','f','a','u','l','t',':', + 'S','t','d','R','e','g','P','r','o','v',0}; + static const WCHAR getstringvalueW[] = + {'G','e','t','S','t','r','i','n','g','V','a','l','u','e',0}; static const struct { const WCHAR *name; @@ -242,6 +247,36 @@ static void test_ParseDisplayName(void) } IBindCtx_Release( ctx ); + + hr = CreateBindCtx( 0, &ctx ); + ok( hr == S_OK, "got %x\n", hr ); + + str = SysAllocString( stdregprovW ); + hr = IParseDisplayName_ParseDisplayName( displayname, NULL, str, &eaten, &moniker ); + ok( hr == S_OK, "got %x\n", hr ); + SysFreeString( str ); + + if (moniker) + { + ISWbemObject *sobj = NULL; + hr = IMoniker_BindToObject( moniker, ctx, NULL, &IID_ISWbemObject, (void **)&sobj ); + ok( hr == S_OK, "got %x\n",hr ); + if (sobj) + { + DISPID dispid = 0xdeadbeef; + + str = SysAllocString( getstringvalueW ); + hr = ISWbemObject_GetIDsOfNames( sobj, &IID_NULL, &str, 1, english, &dispid ); + ok( hr == S_OK, "got %x\n", hr ); + ok( dispid == 0x1000001, "got %x\n", dispid ); + + ISWbemObject_Release( sobj ); + SysFreeString( str ); + } + IMoniker_Release( moniker ); + } + + IBindCtx_Release(ctx); IParseDisplayName_Release( displayname ); }

1 0

Brendan McGrath : quartz/filtergraph: Iterate over all source pins in ExploreGraph().
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 9f0534301321c9192c9e3a705b2bae84a2081745 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9f0534301321c9192c9e3a70… Author: Brendan McGrath <brendan(a)redmandi.com> Date: Tue Oct 9 10:07:36 2018 -0500 quartz/filtergraph: Iterate over all source pins in ExploreGraph(). Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/filtergraph.c | 91 +++++++---------------------------------------- 1 file changed, 13 insertions(+), 78 deletions(-) diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c index 07d40db..7abffa5 100644 --- a/dlls/quartz/filtergraph.c +++ b/dlls/quartz/filtergraph.c @@ -974,68 +974,6 @@ static HRESULT GetFilterInfo(IMoniker* pMoniker, VARIANT* pvar) return hr; } -static HRESULT GetInternalConnections(IBaseFilter* pfilter, IPin* pinputpin, IPin*** pppins, ULONG* pnb) -{ - HRESULT hr; - ULONG nb = 0; - - TRACE("(%p, %p, %p, %p)\n", pfilter, pinputpin, pppins, pnb); - hr = IPin_QueryInternalConnections(pinputpin, NULL, &nb); - if (hr == S_OK) { - /* Rendered input */ - } else if (hr == S_FALSE) { - *pppins = CoTaskMemAlloc(sizeof(IPin*)*nb); - hr = IPin_QueryInternalConnections(pinputpin, *pppins, &nb); - if (hr != S_OK) { - WARN("Error (%x)\n", hr); - } - } else if (hr == E_NOTIMPL) { - /* Input connected to all outputs */ - IEnumPins* penumpins; - IPin* ppin; - int i = 0; - TRACE("E_NOTIMPL\n"); - hr = IBaseFilter_EnumPins(pfilter, &penumpins); - if (FAILED(hr)) { - WARN("filter Enumpins failed (%x)\n", hr); - return hr; - } - i = 0; - /* Count output pins */ - while(IEnumPins_Next(penumpins, 1, &ppin, &nb) == S_OK) { - PIN_DIRECTION pindir; - IPin_QueryDirection(ppin, &pindir); - if (pindir == PINDIR_OUTPUT) - i++; - IPin_Release(ppin); - } - *pppins = CoTaskMemAlloc(sizeof(IPin*)*i); - /* Retrieve output pins */ - IEnumPins_Reset(penumpins); - i = 0; - while(IEnumPins_Next(penumpins, 1, &ppin, &nb) == S_OK) { - PIN_DIRECTION pindir; - IPin_QueryDirection(ppin, &pindir); - if (pindir == PINDIR_OUTPUT) - (*pppins)[i++] = ppin; - else - IPin_Release(ppin); - } - IEnumPins_Release(penumpins); - nb = i; - if (FAILED(hr)) { - WARN("Next failed (%x)\n", hr); - return hr; - } - } else if (FAILED(hr)) { - WARN("Cannot get internal connection (%x)\n", hr); - return hr; - } - - *pnb = nb; - return S_OK; -} - /* Attempt to connect one of the output pins on filter to sink. Helper for * FilterGraph2_Connect(). */ static HRESULT connect_output_pin(IFilterGraphImpl *graph, IBaseFilter *filter, IPin *sink) @@ -2087,12 +2025,12 @@ static HRESULT ExploreGraph(IFilterGraphImpl* pGraph, IPin* pOutputPin, fnFoundF { IAMFilterMiscFlags *flags; IMediaSeeking *seeking; + IEnumPins *enumpins; + PIN_DIRECTION dir; HRESULT hr; IPin* pInputPin; - IPin** ppPins; - ULONG nb; - ULONG i; PIN_INFO PinInfo; + IPin *pin; TRACE("%p %p\n", pGraph, pOutputPin); PinInfo.pFilter = NULL; @@ -2102,26 +2040,23 @@ static HRESULT ExploreGraph(IFilterGraphImpl* pGraph, IPin* pOutputPin, fnFoundF if (SUCCEEDED(hr)) { hr = IPin_QueryPinInfo(pInputPin, &PinInfo); - if (SUCCEEDED(hr)) - hr = GetInternalConnections(PinInfo.pFilter, pInputPin, &ppPins, &nb); IPin_Release(pInputPin); } if (SUCCEEDED(hr)) + hr = IBaseFilter_EnumPins(PinInfo.pFilter, &enumpins); + + if (SUCCEEDED(hr)) { - if (nb) + while (IEnumPins_Next(enumpins, 1, &pin, NULL) == S_OK) { - for(i = 0; i < nb; i++) - { - /* Explore the graph downstream from this pin - * FIXME: We should prevent exploring from a pin more than once. This can happens when - * several input pins are connected to the same output (a MUX for instance). */ - ExploreGraph(pGraph, ppPins[i], FoundFilter, data); - IPin_Release(ppPins[i]); - } - - CoTaskMemFree(ppPins); + IPin_QueryDirection(pin, &dir); + if (dir == PINDIR_OUTPUT) + ExploreGraph(pGraph, pin, FoundFilter, data); + IPin_Release(pin); } + + IEnumPins_Release(enumpins); TRACE("Doing stuff with filter %p\n", PinInfo.pFilter); if (SUCCEEDED(IBaseFilter_QueryInterface(PinInfo.pFilter,

1 0

Alexandre Julliard : opengl32: Update to the current OpenGL spec.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: d900547f58405935c230b2530b06fff6938318f7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d900547f58405935c230b253… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 9 12:59:07 2018 +0200 opengl32: Update to the current OpenGL spec. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/opengl32/make_opengl | 4 +- dlls/opengl32/opengl_ext.c | 196 +++++++++++++++++++++++++++++++++++++++++++- dlls/opengl32/opengl_norm.c | 56 ++++++++++++- include/wine/wgl.h | 95 ++++++++++++++++++++- include/wine/wgl_driver.h | 28 ++++++- 5 files changed, 364 insertions(+), 15 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d900547f58405935c230…

1 0

Jacek Caban : mshtml: Support registry configuration for maximal allowed compatibility mode.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 5489529f4e20efeec908af38c2f0d1f2974aa59c URL: https://source.winehq.org/git/wine.git/?a=commit;h=5489529f4e20efeec908af38… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Oct 9 12:22:16 2018 +0200 mshtml: Support registry configuration for maximal allowed compatibility mode. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/main.c | 125 +++++++++++++++++++++++++++++++++++++++++++ dlls/mshtml/mshtml_private.h | 3 ++ dlls/mshtml/mutation.c | 60 +++++++++++++-------- 3 files changed, 166 insertions(+), 22 deletions(-) diff --git a/dlls/mshtml/main.c b/dlls/mshtml/main.c index 6ce8198..2ba648f 100644 --- a/dlls/mshtml/main.c +++ b/dlls/mshtml/main.c @@ -36,6 +36,7 @@ #include "rpcproxy.h" #include "shlguid.h" #include "mlang.h" +#include "wininet.h" #include "wine/debug.h" @@ -53,6 +54,14 @@ DWORD mshtml_tls = TLS_OUT_OF_INDEXES; static HINSTANCE shdoclc = NULL; static WCHAR *status_strings[IDS_STATUS_LAST-IDS_STATUS_FIRST+1]; static IMultiLanguage2 *mlang; +static unsigned global_max_compat_mode = COMPAT_MODE_IE11; +static struct list compat_config = LIST_INIT(compat_config); + +typedef struct { + struct list entry; + compat_mode_t max_compat_mode; + WCHAR host[1]; +} compat_config_t; static BOOL ensure_mlang(void) { @@ -109,6 +118,114 @@ BSTR charset_string_from_cp(UINT cp) return SysAllocString(info.wszWebCharset); } +static BOOL read_compat_mode(HKEY key, compat_mode_t *r) +{ + WCHAR version[32]; + DWORD type, size; + LSTATUS status; + + static const WCHAR max_compat_modeW[] = {'M','a','x','C','o','m','p','a','t','M','o','d','e',0}; + + size = sizeof(version); + status = RegQueryValueExW(key, max_compat_modeW, NULL, &type, (BYTE*)version, &size); + if(status != ERROR_SUCCESS || type != REG_SZ) + return FALSE; + + return parse_compat_version(version, r); +} + +static BOOL WINAPI load_compat_settings(INIT_ONCE *once, void *param, void **context) +{ + WCHAR key_name[INTERNET_MAX_HOST_NAME_LENGTH]; + DWORD index = 0, name_size; + compat_config_t *new_entry; + compat_mode_t max_compat_mode; + HKEY key, host_key; + DWORD res; + + static const WCHAR key_nameW[] = { + 'S','o','f','t','w','a','r','e', + '\\','W','i','n','e', + '\\','M','S','H','T','M','L', + '\\','C','o','m','p','a','t','M','o','d','e',0}; + + /* @@ Wine registry key: HKCU\Software\Wine\MSHTML\CompatMode */ + res = RegOpenKeyW(HKEY_CURRENT_USER, key_nameW, &key); + if(res != ERROR_SUCCESS) + return TRUE; + + if(read_compat_mode(key, &max_compat_mode)) { + TRACE("Setting global max compat mode to %u\n", max_compat_mode); + global_max_compat_mode = max_compat_mode; + } + + while(1) { + res = RegEnumKeyW(key, index, key_name, ARRAY_SIZE(key_name)); + if(res == ERROR_NO_MORE_ITEMS) + break; + index++; + if(res != ERROR_SUCCESS) { + WARN("RegEnumKey failed: %u\n", GetLastError()); + continue; + } + + name_size = strlenW(key_name) + 1; + new_entry = heap_alloc(FIELD_OFFSET(compat_config_t, host[name_size])); + if(!new_entry) + continue; + + new_entry->max_compat_mode = COMPAT_MODE_IE11; + memcpy(new_entry->host, key_name, name_size * sizeof(WCHAR)); + list_add_tail(&compat_config, &new_entry->entry); + + res = RegOpenKeyW(key, key_name, &host_key); + if(res != ERROR_SUCCESS) + continue; + + if(read_compat_mode(host_key, &max_compat_mode)) { + TRACE("Setting max compat mode for %s to %u\n", debugstr_w(key_name), max_compat_mode); + new_entry->max_compat_mode = max_compat_mode; + } + + RegCloseKey(host_key); + } + + RegCloseKey(key); + return TRUE; +} + +compat_mode_t get_max_compat_mode(IUri *uri) +{ + compat_config_t *iter; + size_t len, iter_len; + BSTR host; + HRESULT hres; + + static INIT_ONCE init_once = INIT_ONCE_STATIC_INIT; + InitOnceExecuteOnce(&init_once, load_compat_settings, NULL, NULL); + + if(!uri) + return global_max_compat_mode; + hres = IUri_GetHost(uri, &host); + if(FAILED(hres)) + return global_max_compat_mode; + len = SysStringLen(host); + + LIST_FOR_EACH_ENTRY(iter, &compat_config, compat_config_t, entry) { + iter_len = strlenW(iter->host); + /* If configured host starts with '.', we also match subdomains */ + if((len == iter_len || (iter->host[0] == '.' && len > iter_len)) + && !memcmp(host + len - iter_len, iter->host, iter_len * sizeof(WCHAR))) { + TRACE("Found max mode %u\n", iter->max_compat_mode); + return iter->max_compat_mode; + } + } + + SysFreeString(host); + TRACE("Using global max mode %u\n", global_max_compat_mode); + return global_max_compat_mode; +} + static void thread_detach(void) { thread_data_t *thread_data; @@ -132,9 +249,17 @@ static void free_strings(void) static void process_detach(void) { + compat_config_t *config; + close_gecko(); release_typelib(); + while(!list_empty(&compat_config)) { + config = LIST_ENTRY(list_head(&compat_config), compat_config_t, entry); + list_remove(&config->entry); + heap_free(config); + } + if(shdoclc) FreeLibrary(shdoclc); if(mshtml_tls != TLS_OUT_OF_INDEXES) diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index ddc8df5..89a56bd 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -1167,6 +1167,8 @@ void remove_target_tasks(LONG) DECLSPEC_HIDDEN; HRESULT set_task_timer(HTMLInnerWindow*,LONG,BOOL,IDispatch*,LONG*) DECLSPEC_HIDDEN; HRESULT clear_task_timer(HTMLInnerWindow*,DWORD) DECLSPEC_HIDDEN; +BOOL parse_compat_version(const WCHAR*,compat_mode_t*) DECLSPEC_HIDDEN; + const char *debugstr_mshtml_guid(const GUID*) DECLSPEC_HIDDEN; DEFINE_GUID(CLSID_AboutProtocol, 0x3050F406, 0x98B5, 0x11CF, 0xBB,0x82, 0x00,0xAA,0x00,0xBD,0xCE,0x0B); @@ -1337,6 +1339,7 @@ static inline VARIANT_BOOL variant_bool(BOOL b) extern void *call_thiscall_func; #endif +compat_mode_t get_max_compat_mode(IUri*) DECLSPEC_HIDDEN; UINT cp_from_charset_string(BSTR) DECLSPEC_HIDDEN; BSTR charset_string_from_cp(UINT) DECLSPEC_HIDDEN; HINSTANCE get_shdoclc(void) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/mutation.c b/dlls/mshtml/mutation.c index dd46dd8..1a1cfe3 100644 --- a/dlls/mshtml/mutation.c +++ b/dlls/mshtml/mutation.c @@ -377,6 +377,8 @@ compat_mode_t lock_document_mode(HTMLDocumentNode *doc) static void set_document_mode(HTMLDocumentNode *doc, compat_mode_t document_mode, BOOL lock) { + compat_mode_t max_compat_mode; + if(doc->document_mode_locked) { WARN("attempting to set document mode %d on locked document %p\n", document_mode, doc); return; @@ -384,35 +386,31 @@ static void set_document_mode(HTMLDocumentNode *doc, compat_mode_t document_mode TRACE("%p: %d\n", doc, document_mode); + max_compat_mode = doc->window && doc->window->base.outer_window + ? get_max_compat_mode(doc->window->base.outer_window->uri) + : COMPAT_MODE_IE11; + if(max_compat_mode < document_mode) { + WARN("Tried to set compat mode %u higher than maximal configured %u\n", + document_mode, max_compat_mode); + document_mode = max_compat_mode; + } + doc->document_mode = document_mode; if(lock) lock_document_mode(doc); } -static BOOL parse_ua_compatible(const WCHAR *p, compat_mode_t *r) +BOOL parse_compat_version(const WCHAR *version_string, compat_mode_t *r) { - int v = 0; - - static const WCHAR ie_eqW[] = {'I','E','='}; - static const WCHAR edgeW[] = {'e','d','g','e',0}; + DWORD version = 0; + const WCHAR *p; - TRACE("%s\n", debugstr_w(p)); - - if(strncmpiW(ie_eqW, p, ARRAY_SIZE(ie_eqW))) + for(p = version_string; '0' <= *p && *p <= '9'; p++) + version = version * 10 + *p-'0'; + if(*p || p == version_string) return FALSE; - p += 3; - if(!strcmpiW(p, edgeW)) { - *r = COMPAT_MODE_IE11; - return TRUE; - } - - while('0' <= *p && *p <= '9') - v = v*10 + *(p++)-'0'; - if(*p || !v) - return FALSE; - - switch(v){ + switch(version){ case 5: case 6: *r = COMPAT_MODE_IE5; @@ -430,12 +428,30 @@ static BOOL parse_ua_compatible(const WCHAR *p, compat_mode_t *r) *r = COMPAT_MODE_IE10; break; default: - *r = v < 5 ? COMPAT_MODE_QUIRKS : COMPAT_MODE_IE11; + *r = version < 5 ? COMPAT_MODE_QUIRKS : COMPAT_MODE_IE11; } - return TRUE; } +static BOOL parse_ua_compatible(const WCHAR *p, compat_mode_t *r) +{ + static const WCHAR ie_eqW[] = {'I','E','='}; + static const WCHAR edgeW[] = {'e','d','g','e',0}; + + TRACE("%s\n", debugstr_w(p)); + + if(strncmpiW(ie_eqW, p, ARRAY_SIZE(ie_eqW))) + return FALSE; + p += 3; + + if(!strcmpiW(p, edgeW)) { + *r = COMPAT_MODE_IE11; + return TRUE; + } + + return parse_compat_version(p, r); +} + void process_document_response_headers(HTMLDocumentNode *doc, IBinding *binding) { IWinInetHttpInfo *http_info;

1 0

Zebediah Figura : widl: Properly handle nesting of pointers and arrays when parsing.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: f12cc887f8a584a5931d11b189a26ad62d598709 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f12cc887f8a584a5931d11b1… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Oct 5 16:57:50 2018 -0500 widl: Properly handle nesting of pointers and arrays when parsing. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/header.h | 1 - tools/widl/parser.y | 120 +++++++++++++++++++++++++++---------------------- tools/widl/widltypes.h | 2 - 3 files changed, 66 insertions(+), 57 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=f12cc887f8a584a5931d…

1 0

Zebediah Figura : widl: Check for positive array dimension when parsing array.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: a2d0038045233a1830c61c4816304c33e2f92087 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a2d0038045233a1830c61c48… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Oct 5 16:57:47 2018 -0500 widl: Check for positive array dimension when parsing array. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 53a5ec0..3b99d93 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -468,8 +468,8 @@ arg: attributes decl_spec m_any_declarator { if ($2->stgclass != STG_NONE && $ ; array: '[' expr ']' { $$ = $2; - if (!$$->is_const) - error_loc("array dimension is not an integer constant\n"); + if (!$$->is_const || $$->cval <= 0) + error_loc("array dimension is not a positive integer constant\n"); } | '[' '*' ']' { $$ = make_expr(EXPR_VOID); } | '[' ']' { $$ = make_expr(EXPR_VOID); } @@ -1554,9 +1554,6 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl { if (dim->is_const) { - if (dim->cval <= 0) - error_loc("%s: array dimension must be positive\n", v->name); - /* FIXME: should use a type_memsize that allows us to pass in a pointer size */ if (0) {

1 0

Zebediah Figura : widl: The default pointer attribute for an array is always ref.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 8f463a575ddce64b51db7df59d344faf01d0ab01 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8f463a575ddce64b51db7df5… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Oct 5 16:57:49 2018 -0500 widl: The default pointer attribute for an array is always ref. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 1fe7129..36a5fae 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1552,10 +1552,12 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl ptype = &v->type; if (arr) LIST_FOR_EACH_ENTRY_REV(dim, arr, expr_t, entry) { + /* An array is always a reference pointer unless explicitly marked otherwise + * (regardless of what the default pointer attribute is). */ *ptype = type_new_array(NULL, *ptype, FALSE, dim->is_const ? dim->cval : 0, dim->is_const ? NULL : dim, NULL, - pointer_default); + FC_RP); } ptype = &v->type;

1 0

Zebediah Figura : widl: Remove dead code.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: 8118a45a6aacea24d1c4de1b52e00b968e598dde URL: https://source.winehq.org/git/wine.git/?a=commit;h=8118a45a6aacea24d1c4de1b… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Oct 5 16:57:48 2018 -0500 widl: Remove dead code. MIDL doesn't warn on this, and it's not a particularly salient mistake to make. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 3b99d93..1fe7129 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1552,18 +1552,6 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl ptype = &v->type; if (arr) LIST_FOR_EACH_ENTRY_REV(dim, arr, expr_t, entry) { - if (dim->is_const) - { - /* FIXME: should use a type_memsize that allows us to pass in a pointer size */ - if (0) - { - unsigned int size = type_memsize(v->type); - - if (0xffffffffu / size < dim->cval) - error_loc("%s: total array size is too large\n", v->name); - } - } - *ptype = type_new_array(NULL, *ptype, FALSE, dim->is_const ? dim->cval : 0, dim->is_const ? NULL : dim, NULL,

1 0

Zebediah Figura : widl: Allow all dimensions of an array to be sizeless.
by Alexandre Julliard Oct. 9, 2018

Oct. 9, 2018

Module: wine Branch: master Commit: bd37ddfa967136cbaee7102ae147bdd411dd605f URL: https://source.winehq.org/git/wine.git/?a=commit;h=bd37ddfa967136cbaee7102a… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Fri Oct 5 16:57:46 2018 -0500 widl: Allow all dimensions of an array to be sizeless. MIDL achieves this by setting the size of the inner arrays to a constant value. widl actually already handles this (but uses 0 instead of 1.) Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 7 ------- 1 file changed, 7 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index bf715c8..53a5ec0 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1456,7 +1456,6 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl var_t *v = decl->var; expr_list_t *sizes = get_attrp(attrs, ATTR_SIZEIS); expr_list_t *lengs = get_attrp(attrs, ATTR_LENGTHIS); - int sizeless; expr_t *dim; type_t **ptype; array_dims_t *arr = decl ? decl->array : NULL; @@ -1551,12 +1550,8 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl v->name); ptype = &v->type; - sizeless = FALSE; if (arr) LIST_FOR_EACH_ENTRY_REV(dim, arr, expr_t, entry) { - if (sizeless) - error_loc("%s: only the first array dimension can be unspecified\n", v->name); - if (dim->is_const) { if (dim->cval <= 0) @@ -1571,8 +1566,6 @@ static var_t *declare_var(attr_list_t *attrs, decl_spec_t *decl_spec, const decl error_loc("%s: total array size is too large\n", v->name); } } - else - sizeless = TRUE; *ptype = type_new_array(NULL, *ptype, FALSE, dim->is_const ? dim->cval : 0,

1 0