wine-commits September 2011

wine-commits@winehq.org

2 participants
996 discussions

Piotr Caban : jscript: Use hash table when accessing object properties by name.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: f1a237d682a116ebbbb995637c16d14b10f69257 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1a237d682a116ebbbb995637… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Sep 2 12:24:25 2011 +0200 jscript: Use hash table when accessing object properties by name. --- dlls/jscript/dispex.c | 168 ++++++++++++++++++++++++++++++++++++------------- 1 files changed, 124 insertions(+), 44 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f1a237d682a116ebbbb99…

1 0

Piotr Caban : jscript: Added Object.hasOwnProperty implementation.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 71b58e56c24c06db589605cd53fc37aca5e2d747 URL: http://source.winehq.org/git/wine.git/?a=commit;h=71b58e56c24c06db589605cd5… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Sep 2 12:24:12 2011 +0200 jscript: Added Object.hasOwnProperty implementation. --- dlls/jscript/dispex.c | 14 ++++++++++++++ dlls/jscript/jscript.h | 1 + dlls/jscript/object.c | 43 +++++++++++++++++++++++++++++++++++++++++-- dlls/jscript/tests/api.js | 11 +++++++++++ dlls/jscript/tests/run.c | 30 ++++++++++++++++++++++++++++++ 5 files changed, 97 insertions(+), 2 deletions(-) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index 653857c..3a0eabf 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -1132,3 +1132,17 @@ HRESULT jsdisp_delete_idx(jsdisp_t *obj, DWORD idx) return delete_prop(prop); } + +VARIANT_BOOL jsdisp_is_own_prop(jsdisp_t *obj, BSTR name) +{ + dispex_prop_t *prop; + HRESULT hres; + + hres = find_prop_name(obj, name, &prop); + if(FAILED(hres)) + return VARIANT_FALSE; + else if(!prop) + return VARIANT_FALSE; + + return prop->type==PROP_VARIANT ? VARIANT_TRUE : VARIANT_FALSE; +} diff --git a/dlls/jscript/jscript.h b/dlls/jscript/jscript.h index c0b60ae..ee03e90 100644 --- a/dlls/jscript/jscript.h +++ b/dlls/jscript/jscript.h @@ -217,6 +217,7 @@ HRESULT jsdisp_propget_name(jsdisp_t*,LPCWSTR,VARIANT*,jsexcept_t*,IServiceProvi HRESULT jsdisp_get_idx(jsdisp_t*,DWORD,VARIANT*,jsexcept_t*,IServiceProvider*) DECLSPEC_HIDDEN; HRESULT jsdisp_get_id(jsdisp_t*,const WCHAR*,DWORD,DISPID*) DECLSPEC_HIDDEN; HRESULT jsdisp_delete_idx(jsdisp_t*,DWORD) DECLSPEC_HIDDEN; +VARIANT_BOOL jsdisp_is_own_prop(jsdisp_t *obj, BSTR name) DECLSPEC_HIDDEN; HRESULT create_builtin_function(script_ctx_t*,builtin_invoke_t,const WCHAR*,const builtin_info_t*,DWORD, jsdisp_t*,jsdisp_t**) DECLSPEC_HIDDEN; diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 9f2e4d5..c1137c1 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -111,8 +111,47 @@ static HRESULT Object_valueOf(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DI static HRESULT Object_hasOwnProperty(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISPPARAMS *dp, VARIANT *retv, jsexcept_t *ei, IServiceProvider *sp) { - FIXME("\n"); - return E_NOTIMPL; + BSTR name; + BOOL result; + DISPID id; + HRESULT hres; + + TRACE("\n"); + + if(!arg_cnt(dp)) { + if(retv) { + V_VT(retv) = VT_BOOL; + V_BOOL(retv) = VARIANT_FALSE; + } + + return S_OK; + } + + hres = to_string(ctx, get_arg(dp, 0), ei, &name); + if(FAILED(hres)) + return hres; + + if(is_jsdisp(jsthis)) { + result = jsdisp_is_own_prop(jsthis->u.jsdisp, name); + if(retv) { + V_VT(retv) = VT_BOOL; + V_BOOL(retv) = result; + } + + return S_OK; + } else if(is_dispex(jsthis)) { + hres = IDispatchEx_GetDispID(jsthis->u.dispex, name, + make_grfdex(ctx, fdexNameCaseSensitive), &id); + } else { + hres = IDispatch_GetIDsOfNames(jsthis->u.disp, &IID_NULL, + &name, 1, ctx->lcid, &id); + } + + if(retv) { + V_VT(retv) = VT_BOOL; + V_BOOL(retv) = SUCCEEDED(hres) ? VARIANT_TRUE : VARIANT_FALSE; + } + return S_OK; } static HRESULT Object_propertyIsEnumerable(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISPPARAMS *dp, diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index c5f0c4f..09c3360 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -1864,11 +1864,22 @@ ok(err.message === 4, "err.message = " + err.message); ok(!("number" in Error), "number is in Error"); tmp = new Object(); +ok(tmp.hasOwnProperty("toString") === false, "toString property should be inherited"); tmp.toString = function() { return "test"; }; +ok(tmp.hasOwnProperty("toString") === true, "toString own property should exist"); +ok(tmp.hasOwnProperty("nonExisting") === false, "nonExisting property should not exist"); tmp = Error.prototype.toString.call(tmp); ok(tmp === "[object Error]", "Error.prototype.toString.call(tmp) = " + tmp); +tmp = function() { return 0; }; +tmp[0] = true; +ok(tmp.hasOwnProperty("toString") === false, "toString property should be inherited"); +ok(tmp.hasOwnProperty("0") === true, "hasOwnProperty(0) returned false"); +ok(tmp.hasOwnProperty() === false, "hasOwnProperty() returned true"); + +ok(Object.prototype.hasOwnProperty.call(testObj) === false, "hasOwnProperty without name returned true"); + if(invokeVersion >= 2) { obj = new Object(); obj.name = "test"; diff --git a/dlls/jscript/tests/run.c b/dlls/jscript/tests/run.c index 88c69ef..fbbff96 100644 --- a/dlls/jscript/tests/run.c +++ b/dlls/jscript/tests/run.c @@ -64,6 +64,8 @@ DEFINE_EXPECT(global_propput_i); DEFINE_EXPECT(global_success_d); DEFINE_EXPECT(global_success_i); DEFINE_EXPECT(global_notexists_d); +DEFINE_EXPECT(puredisp_prop_d); +DEFINE_EXPECT(puredisp_noprop_d); DEFINE_EXPECT(testobj_delete); DEFINE_EXPECT(testobj_value); DEFINE_EXPECT(testobj_prop_d); @@ -173,6 +175,18 @@ static HRESULT WINAPI DispatchEx_GetIDsOfNames(IDispatchEx *iface, REFIID riid, LPOLESTR *rgszNames, UINT cNames, LCID lcid, DISPID *rgDispId) { + ok(IsEqualGUID(riid, &IID_NULL), "Expected IID_NULL\n"); + ok(cNames==1, "cNames = %d\n", cNames); + + if(!strcmp_wa(*rgszNames, "prop")) { + CHECK_EXPECT(puredisp_prop_d); + *rgDispId = DISPID_TESTOBJ_PROP; + return S_OK; + } else if(!strcmp_wa(*rgszNames, "noprop")) { + CHECK_EXPECT(puredisp_noprop_d); + return DISP_E_UNKNOWNNAME; + } + ok(0, "unexpected call\n"); return E_NOTIMPL; } @@ -1476,6 +1490,22 @@ static void run_tests(void) parse_script_a("ok(('noprop' in testObj) === false, 'noprop is in testObj');"); CHECK_CALLED(testobj_noprop_d); + SET_EXPECT(testobj_prop_d); + parse_script_a("ok(Object.prototype.hasOwnProperty.call(testObj, 'prop') === true, 'hasOwnProperty(\\\"prop\\\") returned false');"); + CHECK_CALLED(testobj_prop_d); + + SET_EXPECT(testobj_noprop_d); + parse_script_a("ok(Object.prototype.hasOwnProperty.call(testObj, 'noprop') === false, 'hasOwnProperty(\\\"noprop\\\") returned true');"); + CHECK_CALLED(testobj_noprop_d); + + SET_EXPECT(puredisp_prop_d); + parse_script_a("ok(Object.prototype.hasOwnProperty.call(pureDisp, 'prop') === true, 'hasOwnProperty(\\\"noprop\\\") returned false');"); + CHECK_CALLED(puredisp_prop_d); + + SET_EXPECT(puredisp_noprop_d); + parse_script_a("ok(Object.prototype.hasOwnProperty.call(pureDisp, 'noprop') === false, 'hasOwnProperty(\\\"noprop\\\") returned true');"); + CHECK_CALLED(puredisp_noprop_d); + SET_EXPECT(testobj_value); parse_script_a("ok(String(testObj) === '1', 'wrong testObj value');"); CHECK_CALLED(testobj_value);

1 0

Alexandre Julliard : gdi32: Make the BITMAPINFO parameters non-const in driver entry points since we are always passing a copy .
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: a0f1b204649268bea8a2f99c384faee2813a7fac URL: http://source.winehq.org/git/wine.git/?a=commit;h=a0f1b204649268bea8a2f99c3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 2 15:53:10 2011 +0200 gdi32: Make the BITMAPINFO parameters non-const in driver entry points since we are always passing a copy. --- dlls/gdi32/dib.c | 2 +- dlls/gdi32/driver.c | 4 ++-- dlls/gdi32/enhmfdrv/bitblt.c | 4 ++-- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 5 ++--- dlls/gdi32/gdi_private.h | 2 +- dlls/gdi32/mfdrv/bitblt.c | 5 ++--- dlls/gdi32/mfdrv/metafiledrv.h | 5 ++--- dlls/wineps.drv/bitmap.c | 2 +- dlls/wineps.drv/psdrv.h | 2 +- dlls/winex11.drv/dib.c | 4 ++-- dlls/winex11.drv/x11drv.h | 4 ++-- include/wine/gdi_driver.h | 6 +++--- 12 files changed, 21 insertions(+), 24 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a0f1b204649268bea8a2f…

1 0

André Hentschel : urlmon/tests: Print out the wrong mime.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 62dca77bb3341846af010397c26adad34c66041b URL: http://source.winehq.org/git/wine.git/?a=commit;h=62dca77bb3341846af010397c… Author: André Hentschel <nerv(a)dawncrow.de> Date: Thu Sep 1 22:43:28 2011 +0200 urlmon/tests: Print out the wrong mime. --- dlls/urlmon/tests/misc.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/urlmon/tests/misc.c b/dlls/urlmon/tests/misc.c index ecb0c87..cfa04f0 100644 --- a/dlls/urlmon/tests/misc.c +++ b/dlls/urlmon/tests/misc.c @@ -666,7 +666,7 @@ static void test_FindMimeFromData(void) hres = pFindMimeFromData(NULL, mime_tests[i].url, NULL, 0, NULL, 0, &mime, 0); if(mime_tests[i].mime) { ok(hres == S_OK, "[%d] FindMimeFromData failed: %08x\n", i, hres); - ok(!lstrcmpW(mime, mime_tests[i].mime), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mime_tests[i].mime), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); CoTaskMemFree(mime); }else { ok(hres == E_FAIL || hres == mime_tests[i].hres, @@ -678,13 +678,13 @@ static void test_FindMimeFromData(void) mime = (LPWSTR)0xf0f0f0f0; hres = pFindMimeFromData(NULL, mime_tests[i].url, NULL, 0, mimeTextPlain, 0, &mime, 0); ok(hres == S_OK, "[%d] FindMimeFromData failed: %08x\n", i, hres); - ok(!lstrcmpW(mime, mimeTextPlain), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mimeTextPlain), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); CoTaskMemFree(mime); mime = (LPWSTR)0xf0f0f0f0; hres = pFindMimeFromData(NULL, mime_tests[i].url, NULL, 0, mimeAppOctetStream, 0, &mime, 0); ok(hres == S_OK, "[%d] FindMimeFromData failed: %08x\n", i, hres); - ok(!lstrcmpW(mime, mimeAppOctetStream), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mimeAppOctetStream), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); CoTaskMemFree(mime); } @@ -700,16 +700,16 @@ static void test_FindMimeFromData(void) ok(hres == S_OK, "[%d] FindMimeFromData failed: %08x\n", i, hres); if(!lstrcmpW(mimeAppOctetStream, mime_tests2[i].mime) || !lstrcmpW(mimeTextPlain, mime_tests2[i].mime)) - ok(!lstrcmpW(mime, mimeTextHtml), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mimeTextHtml), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); else - ok(!lstrcmpW(mime, mime_tests2[i].mime), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mime_tests2[i].mime), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); CoTaskMemFree(mime); hres = pFindMimeFromData(NULL, NULL, mime_tests2[i].data, mime_tests2[i].size, mimeImagePjpeg, 0, &mime, 0); ok(hres == S_OK, "[%d] FindMimeFromData failed: %08x\n", i, hres); if(!lstrcmpW(mimeAppOctetStream, mime_tests2[i].mime) || i == 17) - ok(!lstrcmpW(mime, mimeImagePjpeg), "[%d] wrong mime\n", i); + ok(!lstrcmpW(mime, mimeImagePjpeg), "[%d] wrong mime: %s\n", i, wine_dbgstr_w(mime)); else ok(!lstrcmpW(mime, mime_tests2[i].mime) || (mime_tests2[i].mime_alt && !lstrcmpW(mime, mime_tests2[i].mime_alt)), @@ -720,17 +720,17 @@ static void test_FindMimeFromData(void) hres = pFindMimeFromData(NULL, url1, data1, sizeof(data1), NULL, 0, &mime, 0); ok(hres == S_OK, "FindMimeFromData failed: %08x\n", hres); - ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime\n"); + ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime: %s\n", wine_dbgstr_w(mime)); CoTaskMemFree(mime); hres = pFindMimeFromData(NULL, url1, data1, sizeof(data1), mimeAppOctetStream, 0, &mime, 0); ok(hres == S_OK, "FindMimeFromData failed: %08x\n", hres); - ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime\n"); + ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime: %s\n", wine_dbgstr_w(mime)); CoTaskMemFree(mime); hres = pFindMimeFromData(NULL, url4, data1, sizeof(data1), mimeAppOctetStream, 0, &mime, 0); ok(hres == S_OK, "FindMimeFromData failed: %08x\n", hres); - ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime\n"); + ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime: %s\n", wine_dbgstr_w(mime)); CoTaskMemFree(mime); hres = pFindMimeFromData(NULL, NULL, NULL, 0, NULL, 0, &mime, 0); @@ -747,7 +747,7 @@ static void test_FindMimeFromData(void) hres = pFindMimeFromData(NULL, NULL, data1, 0, mimeTextPlain, 0, &mime, 0); ok(hres == S_OK, "FindMimeFromData failed: %08x\n", hres); - ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime\n"); + ok(!lstrcmpW(mime, mimeTextPlain), "wrong mime: %s\n", wine_dbgstr_w(mime)); CoTaskMemFree(mime); hres = pFindMimeFromData(NULL, NULL, data1, 0, mimeTextPlain, 0, NULL, 0);

1 0

Jacek Caban : vbscript: Moved includes required by vbscript.h to the header .
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 4f5b2a4c3acbd23c1da981523b674aae5f4c653c URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f5b2a4c3acbd23c1da981523… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 13:16:27 2011 +0200 vbscript: Moved includes required by vbscript.h to the header. --- dlls/vbscript/vbscript.c | 10 +--------- dlls/vbscript/vbscript.h | 8 ++++++++ dlls/vbscript/vbscript_main.c | 11 +---------- 3 files changed, 10 insertions(+), 19 deletions(-) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index 04a513f..b50cbd6 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -16,19 +16,11 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include "config.h" -#include <stdarg.h> #include <assert.h> -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" -#include "ole2.h" -#include "objsafe.h" - #include "vbscript.h" +#include "objsafe.h" #include "wine/debug.h" diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h index 98a4486..29909f2 100644 --- a/dlls/vbscript/vbscript.h +++ b/dlls/vbscript/vbscript.h @@ -16,7 +16,15 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include <stdarg.h> + +#define COBJMACROS + +#include "windef.h" +#include "winbase.h" +#include "ole2.h" #include "activscp.h" + #include "vbscript_classes.h" typedef struct { diff --git a/dlls/vbscript/vbscript_main.c b/dlls/vbscript/vbscript_main.c index 71eaa86..fc31aed 100644 --- a/dlls/vbscript/vbscript_main.c +++ b/dlls/vbscript/vbscript_main.c @@ -16,20 +16,11 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include "config.h" - -#include <stdarg.h> - -#define COBJMACROS - -#include "windef.h" -#include "winbase.h" #include "initguid.h" -#include "ole2.h" +#include "vbscript.h" #include "objsafe.h" #include "rpcproxy.h" -#include "vbscript.h" #include "vbscript_classes.h" #include "wine/debug.h"

1 0

Jacek Caban : vbscript: Added IObjectSafety tests.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 286c9cee4a2e78c9f07d69238eacab4679e489e0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=286c9cee4a2e78c9f07d69238… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 13:16:18 2011 +0200 vbscript: Added IObjectSafety tests. --- dlls/vbscript/tests/vbscript.c | 96 +++++++++++++++++++++++++++++++++++++++- 1 files changed, 95 insertions(+), 1 deletions(-) diff --git a/dlls/vbscript/tests/vbscript.c b/dlls/vbscript/tests/vbscript.c index 9d0dbdf..ff167b7 100644 --- a/dlls/vbscript/tests/vbscript.c +++ b/dlls/vbscript/tests/vbscript.c @@ -22,9 +22,12 @@ #include <initguid.h> #include <ole2.h> #include <activscp.h> +#include <objsafe.h> #include "wine/test.h" +DEFINE_GUID(GUID_NULL,0,0,0,0,0,0,0,0,0,0,0); + #define DEFINE_EXPECT(func) \ static BOOL expect_ ## func = FALSE, called_ ## func = FALSE @@ -188,6 +191,97 @@ static const IActiveScriptSiteVtbl ActiveScriptSiteVtbl = { static IActiveScriptSite ActiveScriptSite = { &ActiveScriptSiteVtbl }; +static void test_safety(IActiveScript *script) +{ + IObjectSafety *safety; + DWORD supported, enabled; + HRESULT hres; + + hres = IActiveScript_QueryInterface(script, &IID_IObjectSafety, (void**)&safety); + ok(hres == S_OK, "Could not get IObjectSafety: %08x\n", hres); + if(FAILED(hres)) + return; + + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_NULL, &supported, NULL); + ok(hres == E_POINTER, "GetInterfaceSafetyOptions failed: %08x, expected E_POINTER\n", hres); + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_NULL, NULL, &enabled); + ok(hres == E_POINTER, "GetInterfaceSafetyOptions failed: %08x, expected E_POINTER\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_NULL, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScript, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER + |INTERFACESAFE_FOR_UNTRUSTED_CALLER, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER); + ok(hres == E_FAIL, "SetInterfaceSafetyOptions failed: %08x, expected E_FAIL\n", hres); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, INTERFACESAFE_FOR_UNTRUSTED_DATA, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == (INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + hres = IObjectSafety_SetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, + INTERFACE_USES_DISPEX, 0); + ok(hres == S_OK, "SetInterfaceSafetyOptions failed: %08x\n", hres); + + supported = enabled = 0xdeadbeef; + hres = IObjectSafety_GetInterfaceSafetyOptions(safety, &IID_IActiveScriptParse, &supported, &enabled); + ok(hres == S_OK, "GetInterfaceSafetyOptions failed: %08x\n", hres); + ok(supported == (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER), + "supported=%x\n", supported); + ok(enabled == INTERFACE_USES_DISPEX, "enabled=%x\n", enabled); + + IObjectSafety_Release(safety); +} + static IActiveScript *create_vbscript(void) { IActiveScript *ret; @@ -213,6 +307,7 @@ static void test_vbscript(void) ok(hres == S_OK, "Could not get IActiveScriptParse iface: %08x\n", hres); test_state(vbscript, SCRIPTSTATE_UNINITIALIZED); + test_safety(vbscript); SET_EXPECT(GetLCID); hres = IActiveScript_SetScriptSite(vbscript, &ActiveScriptSite); @@ -244,7 +339,6 @@ static void test_vbscript(void) ok(!ref, "ref = %d\n", ref); } - static BOOL check_vbscript(void) { IActiveScript *vbscript;

1 0

Jacek Caban : vbscript: Added GetInterfaceSafetyOptions implementation.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: dd7ee024afd8b1879d437d63a2c6eeec1d9403f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dd7ee024afd8b1879d437d63a… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 13:16:08 2011 +0200 vbscript: Added GetInterfaceSafetyOptions implementation. --- dlls/vbscript/vbscript.c | 11 +++++++++-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index 70b35f2..04a513f 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -438,8 +438,15 @@ static HRESULT WINAPI VBScriptSafety_GetInterfaceSafetyOptions(IObjectSafety *if DWORD *pdwSupportedOptions, DWORD *pdwEnabledOptions) { VBScript *This = impl_from_IObjectSafety(iface); - FIXME("(%p)->(%s %p %p)\n", This, debugstr_guid(riid), pdwSupportedOptions, pdwEnabledOptions); - return E_NOTIMPL; + + TRACE("(%p)->(%s %p %p)\n", This, debugstr_guid(riid), pdwSupportedOptions, pdwEnabledOptions); + + if(!pdwSupportedOptions || !pdwEnabledOptions) + return E_POINTER; + + *pdwSupportedOptions = SUPPORTED_OPTIONS; + *pdwEnabledOptions = This->safeopt; + return S_OK; } static HRESULT WINAPI VBScriptSafety_SetInterfaceSafetyOptions(IObjectSafety *iface, REFIID riid,

1 0

Jacek Caban : vbscript: Added SetInterfaceSafetyOptions implementation.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: a34ece4cbe10321ed3c1f1b8c724cddaf18b7fe4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a34ece4cbe10321ed3c1f1b8c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 13:16:00 2011 +0200 vbscript: Added SetInterfaceSafetyOptions implementation. --- dlls/vbscript/vbscript.c | 14 ++++++++++++-- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index 3f963a3..70b35f2 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -53,6 +53,7 @@ struct VBScript { LONG ref; + DWORD safeopt; SCRIPTSTATE state; IActiveScriptSite *site; script_ctx_t *ctx; @@ -431,6 +432,8 @@ static ULONG WINAPI VBScriptSafety_Release(IObjectSafety *iface) return IActiveScript_Release(&This->IActiveScript_iface); } +#define SUPPORTED_OPTIONS (INTERFACESAFE_FOR_UNTRUSTED_DATA|INTERFACE_USES_DISPEX|INTERFACE_USES_SECURITY_MANAGER) + static HRESULT WINAPI VBScriptSafety_GetInterfaceSafetyOptions(IObjectSafety *iface, REFIID riid, DWORD *pdwSupportedOptions, DWORD *pdwEnabledOptions) { @@ -443,8 +446,14 @@ static HRESULT WINAPI VBScriptSafety_SetInterfaceSafetyOptions(IObjectSafety *if DWORD dwOptionSetMask, DWORD dwEnabledOptions) { VBScript *This = impl_from_IObjectSafety(iface); - FIXME("(%p)->(%s %x %x)\n", This, debugstr_guid(riid), dwOptionSetMask, dwEnabledOptions); - return E_NOTIMPL; + + TRACE("(%p)->(%s %x %x)\n", This, debugstr_guid(riid), dwOptionSetMask, dwEnabledOptions); + + if(dwOptionSetMask & ~SUPPORTED_OPTIONS) + return E_FAIL; + + This->safeopt = (dwEnabledOptions & dwOptionSetMask) | (This->safeopt & ~dwOptionSetMask) | INTERFACE_USES_DISPEX; + return S_OK; } static const IObjectSafetyVtbl VBScriptSafetyVtbl = { @@ -472,6 +481,7 @@ HRESULT WINAPI VBScriptFactory_CreateInstance(IClassFactory *iface, IUnknown *pU ret->ref = 1; ret->state = SCRIPTSTATE_UNINITIALIZED; + ret->safeopt = INTERFACE_USES_DISPEX; hres = IActiveScript_QueryInterface(&ret->IActiveScript_iface, riid, ppv); IActiveScript_Release(&ret->IActiveScript_iface);

1 0

Jacek Caban : vbscript: Added IObjectSafety stub implementation.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 0491b0326013f6f529fe13866094e208b1dd0f56 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0491b0326013f6f529fe13866… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 13:15:51 2011 +0200 vbscript: Added IObjectSafety stub implementation. --- dlls/vbscript/vbscript.c | 53 +++++++++++++++++++++++++++++++++++++++++ dlls/vbscript/vbscript_main.c | 1 + 2 files changed, 54 insertions(+), 0 deletions(-) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index caee4d4..3f963a3 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -26,6 +26,7 @@ #include "windef.h" #include "winbase.h" #include "ole2.h" +#include "objsafe.h" #include "vbscript.h" @@ -48,6 +49,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(vbscript); struct VBScript { IActiveScript IActiveScript_iface; IActiveScriptParse IActiveScriptParse_iface; + IObjectSafety IObjectSafety_iface; LONG ref; @@ -135,6 +137,9 @@ static HRESULT WINAPI VBScript_QueryInterface(IActiveScript *iface, REFIID riid, }else if(IsEqualGUID(riid, &IID_IActiveScriptParse)) { TRACE("(%p)->(IID_IActiveScriptParse %p)\n", This, ppv); *ppv = &This->IActiveScriptParse_iface; + }else if(IsEqualGUID(riid, &IID_IObjectSafety)) { + TRACE("(%p)->(IID_IObjectSafety %p)\n", This, ppv); + *ppv = &This->IObjectSafety_iface; }else { FIXME("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppv); *ppv = NULL; @@ -403,6 +408,53 @@ static const IActiveScriptParseVtbl VBScriptParseVtbl = { VBScriptParse_ParseScriptText }; +static inline VBScript *impl_from_IObjectSafety(IObjectSafety *iface) +{ + return CONTAINING_RECORD(iface, VBScript, IObjectSafety_iface); +} + +static HRESULT WINAPI VBScriptSafety_QueryInterface(IObjectSafety *iface, REFIID riid, void **ppv) +{ + VBScript *This = impl_from_IObjectSafety(iface); + return IActiveScript_QueryInterface(&This->IActiveScript_iface, riid, ppv); +} + +static ULONG WINAPI VBScriptSafety_AddRef(IObjectSafety *iface) +{ + VBScript *This = impl_from_IObjectSafety(iface); + return IActiveScript_AddRef(&This->IActiveScript_iface); +} + +static ULONG WINAPI VBScriptSafety_Release(IObjectSafety *iface) +{ + VBScript *This = impl_from_IObjectSafety(iface); + return IActiveScript_Release(&This->IActiveScript_iface); +} + +static HRESULT WINAPI VBScriptSafety_GetInterfaceSafetyOptions(IObjectSafety *iface, REFIID riid, + DWORD *pdwSupportedOptions, DWORD *pdwEnabledOptions) +{ + VBScript *This = impl_from_IObjectSafety(iface); + FIXME("(%p)->(%s %p %p)\n", This, debugstr_guid(riid), pdwSupportedOptions, pdwEnabledOptions); + return E_NOTIMPL; +} + +static HRESULT WINAPI VBScriptSafety_SetInterfaceSafetyOptions(IObjectSafety *iface, REFIID riid, + DWORD dwOptionSetMask, DWORD dwEnabledOptions) +{ + VBScript *This = impl_from_IObjectSafety(iface); + FIXME("(%p)->(%s %x %x)\n", This, debugstr_guid(riid), dwOptionSetMask, dwEnabledOptions); + return E_NOTIMPL; +} + +static const IObjectSafetyVtbl VBScriptSafetyVtbl = { + VBScriptSafety_QueryInterface, + VBScriptSafety_AddRef, + VBScriptSafety_Release, + VBScriptSafety_GetInterfaceSafetyOptions, + VBScriptSafety_SetInterfaceSafetyOptions +}; + HRESULT WINAPI VBScriptFactory_CreateInstance(IClassFactory *iface, IUnknown *pUnkOuter, REFIID riid, void **ppv) { VBScript *ret; @@ -416,6 +468,7 @@ HRESULT WINAPI VBScriptFactory_CreateInstance(IClassFactory *iface, IUnknown *pU ret->IActiveScript_iface.lpVtbl = &VBScriptVtbl; ret->IActiveScriptParse_iface.lpVtbl = &VBScriptParseVtbl; + ret->IObjectSafety_iface.lpVtbl = &VBScriptSafetyVtbl; ret->ref = 1; ret->state = SCRIPTSTATE_UNINITIALIZED; diff --git a/dlls/vbscript/vbscript_main.c b/dlls/vbscript/vbscript_main.c index 093019a..71eaa86 100644 --- a/dlls/vbscript/vbscript_main.c +++ b/dlls/vbscript/vbscript_main.c @@ -27,6 +27,7 @@ #include "initguid.h" #include "ole2.h" +#include "objsafe.h" #include "rpcproxy.h" #include "vbscript.h" #include "vbscript_classes.h"

1 0

Jacek Caban : mshtml: Added support for text/vbscript script type.
by Alexandre Julliard 02 Sep '11

02 Sep '11

Module: wine Branch: master Commit: 3a268796b82a3ffc4f11c986d69a80066f620453 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3a268796b82a3ffc4f11c986d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Sep 2 12:03:17 2011 +0200 mshtml: Added support for text/vbscript script type. --- dlls/mshtml/script.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c index 9b1a81e..25ae0b6 100644 --- a/dlls/mshtml/script.c +++ b/dlls/mshtml/script.c @@ -48,6 +48,8 @@ static const WCHAR emptyW[] = {0}; static const CLSID CLSID_JScript = {0xf414c260,0x6ac0,0x11cf,{0xb6,0xd1,0x00,0xaa,0x00,0xbb,0xbb,0x58}}; +static const CLSID CLSID_VBScript = + {0xb54f3741,0x5b07,0x11cf,{0xa4,0xb0,0x00,0xaa,0x00,0x4a,0x55,0xe8}}; struct ScriptHost { IActiveScriptSite IActiveScriptSite_iface; @@ -718,10 +720,14 @@ static BOOL get_guid_from_type(LPCWSTR type, GUID *guid) { const WCHAR text_javascriptW[] = {'t','e','x','t','/','j','a','v','a','s','c','r','i','p','t',0}; + const WCHAR text_vbscriptW[] = + {'t','e','x','t','/','v','b','s','c','r','i','p','t',0}; /* FIXME: Handle more types */ if(!strcmpiW(type, text_javascriptW)) { *guid = CLSID_JScript; + }else if(!strcmpiW(type, text_vbscriptW)) { + *guid = CLSID_VBScript; }else { FIXME("Unknown type %s\n", debugstr_w(type)); return FALSE;

1 0

← Newer
1
...
92
93
94
95
96
97
98
99
100
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2011