November 2009 - Wine-commits

Rob Shearman : ole32: Fix off-by-one error in FileMonikerImpl_DecomposePath .
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: db33e8fdd9b2a24b1b35b86a0e3814fd98356810 URL: http://source.winehq.org/git/wine.git/?a=commit;h=db33e8fdd9b2a24b1b35b86a0… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Sun Nov 29 10:18:01 2009 +0000 ole32: Fix off-by-one error in FileMonikerImpl_DecomposePath. strgtable needs to be an array of the same size as str, including the nul-terminator. --- dlls/ole32/filemoniker.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/filemoniker.c b/dlls/ole32/filemoniker.c index ff543aa..72f5913 100644 --- a/dlls/ole32/filemoniker.c +++ b/dlls/ole32/filemoniker.c @@ -1048,7 +1048,7 @@ int FileMonikerImpl_DecomposePath(LPCOLESTR str, LPOLESTR** stringTable) TRACE("%s, %p\n", debugstr_w(str), *stringTable); - strgtable = CoTaskMemAlloc(len*sizeof(*strgtable)); + strgtable = CoTaskMemAlloc((len + 1)*sizeof(*strgtable)); if (strgtable==NULL) return E_OUTOFMEMORY;

1 0

Jacek Caban : mshtml: Fixed ref count leak and code clean up.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 5dd4248ff36bd88d75359dc45e79add7ffb47d14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5dd4248ff36bd88d75359dc45… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Nov 24 21:12:40 2009 +0100 mshtml: Fixed ref count leak and code clean up. --- dlls/mshtml/tests/dom.c | 25 ++++--------------------- 1 files changed, 4 insertions(+), 21 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index bd55c9f..1387b13 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -5606,8 +5606,6 @@ static void test_frame(IDispatch *disp, const char *exp_id) IHTMLDocument2 *parent_doc, *top_doc; IHTMLWindow4 *frame; IHTMLFrameBase *frame_elem; - IHTMLElement *html_elem; - BSTR bstr; HRESULT hres; hres = IDispatch_QueryInterface(disp, &IID_IHTMLWindow4, (void**)&frame); @@ -5621,17 +5619,8 @@ static void test_frame(IDispatch *disp, const char *exp_id) if(FAILED(hres)) return; - hres = IHTMLFrameBase_QueryInterface(frame_elem, &IID_IHTMLElement, (void**)&html_elem); - ok(hres == S_OK, "Could not get IHTMLElement interface: 0x%08x\n", hres); - IHTMLFrameBase_Release(frame_elem); - if(FAILED(hres)) - return; - - hres = IHTMLElement_get_id(html_elem, &bstr); - ok(hres == S_OK, "IHTMLElement_get_id failed: 0x%08x\n", hres); - ok(!strcmp_wa(bstr, exp_id), "Expected ID: \"%s\", found ID: %s\n", exp_id, wine_dbgstr_w(bstr)); - IHTMLElement_Release(html_elem); - SysFreeString(bstr); + test_elem_id((IUnknown*)frame_elem, exp_id); + IHTMLElement_Release(frame_elem); hres = IDispatch_QueryInterface(disp, &IID_IHTMLWindow2, (void**)&frame2); ok(hres == S_OK, "Could not get IHTMLWindow2 interface: 0x%08x\n", hres); @@ -5653,11 +5642,8 @@ static void test_frame(IDispatch *disp, const char *exp_id) return; } - hres = IHTMLDocument2_get_title(parent_doc, &bstr); - ok(hres == S_OK, "IHTMLDocument2_get_title failed: 0x%08x\n", hres); - ok(!strcmp_wa(bstr, "frameset test"), "Did not get the right parent. Expected \"frameset test\", found %s\n", wine_dbgstr_w(bstr)); + test_doc_title(parent_doc, "frameset test"); IHTMLDocument2_Release(parent_doc); - SysFreeString(bstr); /* test get_top */ hres = IHTMLWindow2_get_top(frame2, &top); @@ -5672,11 +5658,8 @@ static void test_frame(IDispatch *disp, const char *exp_id) if(FAILED(hres)) return; - hres = IHTMLDocument2_get_title(top_doc, &bstr); - ok(hres == S_OK, "IHTMLDocument2_get_title failed: 0x%08x\n", hres); - ok(!strcmp_wa(bstr, "frameset test"), "Did not get the right parent. Expected \"frameset test\", found %s\n", wine_dbgstr_w(bstr)); + test_doc_title(top_doc, "frameset test"); IHTMLDocument2_Release(top_doc); - SysFreeString(bstr); } static void test_frameset(IHTMLDocument2 *doc)

1 0

Jacek Caban : mshtml: Moved IHTMLFrameBase2 implementation to HTMLFrameBase object.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 7c64ab1e83ebd9dbd2c0c9152ec5327140cb06a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7c64ab1e83ebd9dbd2c0c9152… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Nov 24 21:13:12 2009 +0100 mshtml: Moved IHTMLFrameBase2 implementation to HTMLFrameBase object. --- dlls/mshtml/htmlframebase.c | 144 ++++++++++++++++++++++++++++++++++++++ dlls/mshtml/htmliframe.c | 156 +----------------------------------------- dlls/mshtml/mshtml_private.h | 4 +- 3 files changed, 148 insertions(+), 156 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7c64ab1e83ebd9dbd2c0c…

1 0

Jacek Caban : mshtml: Return content document in get_document for frame elements.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 999371724a1da36629a38caded1f011877417593 URL: http://source.winehq.org/git/wine.git/?a=commit;h=999371724a1da36629a38cade… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Nov 24 21:13:38 2009 +0100 mshtml: Return content document in get_document for frame elements. --- dlls/mshtml/htmlelem.c | 4 +++- dlls/mshtml/htmlframebase.c | 21 ++++++++++++++++++++- dlls/mshtml/htmliframe.c | 21 ++++++++++++++++++++- dlls/mshtml/mshtml_private.h | 1 + 4 files changed, 44 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index 0864e49..45338fe 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -569,9 +569,11 @@ static HRESULT WINAPI HTMLElement_get_document(IHTMLElement *iface, IDispatch ** if(!p) return E_POINTER; + if(This->node.vtbl->get_document) + return This->node.vtbl->get_document(&This->node, p); + *p = (IDispatch*)HTMLDOC(&This->node.doc->basedoc); IDispatch_AddRef(*p); - return S_OK; } diff --git a/dlls/mshtml/htmlframebase.c b/dlls/mshtml/htmlframebase.c index b474a34..2526593 100644 --- a/dlls/mshtml/htmlframebase.c +++ b/dlls/mshtml/htmlframebase.c @@ -443,11 +443,30 @@ static void HTMLFrameElement_destructor(HTMLDOMNode *iface) HTMLFrameBase_destructor(&This->framebase); } +static HRESULT HTMLFrameElement_get_document(HTMLDOMNode *iface, IDispatch **p) +{ + HTMLFrameElement *This = HTMLFRAME_NODE_THIS(iface); + + if(!This->framebase.content_window || !This->framebase.content_window->doc) { + *p = NULL; + return S_OK; + } + + *p = (IDispatch*)HTMLDOC(&This->framebase.content_window->doc->basedoc); + IDispatch_AddRef(*p); + return S_OK; +} + #undef HTMLFRAME_NODE_THIS static const NodeImplVtbl HTMLFrameElementImplVtbl = { HTMLFrameElement_QI, - HTMLFrameElement_destructor + HTMLFrameElement_destructor, + NULL, + NULL, + NULL, + NULL, + HTMLFrameElement_get_document }; HTMLElement *HTMLFrameElement_Create(HTMLDocumentNode *doc, nsIDOMHTMLElement *nselem, HTMLWindow *content_window) diff --git a/dlls/mshtml/htmliframe.c b/dlls/mshtml/htmliframe.c index 218638b..8c3c520 100644 --- a/dlls/mshtml/htmliframe.c +++ b/dlls/mshtml/htmliframe.c @@ -58,11 +58,30 @@ static void HTMLIFrame_destructor(HTMLDOMNode *iface) HTMLFrameBase_destructor(&This->framebase); } +static HRESULT HTMLIFrame_get_document(HTMLDOMNode *iface, IDispatch **p) +{ + HTMLIFrame *This = HTMLIFRAME_NODE_THIS(iface); + + if(!This->framebase.content_window || !This->framebase.content_window->doc) { + *p = NULL; + return S_OK; + } + + *p = (IDispatch*)HTMLDOC(&This->framebase.content_window->doc->basedoc); + IDispatch_AddRef(*p); + return S_OK; +} + #undef HTMLIFRAME_NODE_THIS static const NodeImplVtbl HTMLIFrameImplVtbl = { HTMLIFrame_QI, - HTMLIFrame_destructor + HTMLIFrame_destructor, + NULL, + NULL, + NULL, + NULL, + HTMLIFrame_get_document }; static const tid_t HTMLIFrame_iface_tids[] = { diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index d1fe9f8..ef4642b 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -445,6 +445,7 @@ typedef struct { HRESULT (*call_event)(HTMLDOMNode*,DWORD,BOOL*); HRESULT (*put_disabled)(HTMLDOMNode*,VARIANT_BOOL); HRESULT (*get_disabled)(HTMLDOMNode*,VARIANT_BOOL*); + HRESULT (*get_document)(HTMLDOMNode*,IDispatch**); } NodeImplVtbl; struct HTMLDOMNode {

1 0

Jacek Caban : mshtml: Added [i]frame elements content document tests.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 1fccc1b4fa59ad5570ca7a2a5d4331439a628fa8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1fccc1b4fa59ad5570ca7a2a5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Nov 24 21:13:57 2009 +0100 mshtml: Added [i]frame elements content document tests. --- dlls/mshtml/tests/dom.c | 97 +++++++++++++++++++++++++++++++++++++--------- 1 files changed, 78 insertions(+), 19 deletions(-) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 1387b13..0376ac2 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -883,6 +883,25 @@ static void _test_window_length(unsigned line, IHTMLWindow2 *window, LONG exlen) ok_(__FILE__,line)(length == exlen, "length = %d, expected %d\n", length, exlen); } +#define get_frame_content_window(e) _get_frame_content_window(__LINE__,e) +static IHTMLWindow2 *_get_frame_content_window(unsigned line, IUnknown *elem) +{ + IHTMLFrameBase2 *base2; + IHTMLWindow2 *window; + HRESULT hres; + + hres = IUnknown_QueryInterface(elem, &IID_IHTMLFrameBase2, (void**)&base2); + ok(hres == S_OK, "Could not get IHTMFrameBase2 iface: %08x\n", hres); + + window = NULL; + hres = IHTMLFrameBase2_get_contentWindow(base2, &window); + IHTMLFrameBase2_Release(base2); + ok(hres == S_OK, "get_contentWindow failed: %08x\n", hres); + ok(window != NULL, "contentWindow = NULL\n"); + + return window; +} + static void test_get_set_attr(IHTMLDocument2 *doc) { IHTMLElement *elem; @@ -2202,6 +2221,40 @@ static void _test_elem_client_rect(unsigned line, IUnknown *unk) IHTMLElement2_Release(elem); } +#define get_elem_doc(e) _get_elem_doc(__LINE__,e) +static IHTMLDocument2 *_get_elem_doc(unsigned line, IUnknown *unk) +{ + IHTMLElement *elem = _get_elem_iface(line, unk); + IHTMLDocument2 *doc; + IDispatch *disp; + HRESULT hres; + + disp = NULL; + hres = IHTMLElement_get_document(elem, &disp); + ok(hres == S_OK, "get_document failed: %08x\n", hres); + ok(disp != NULL, "disp == NULL\n"); + + hres = IDispatch_QueryInterface(disp, &IID_IHTMLDocument2, (void**)&doc); + IDispatch_Release(disp); + ok(hres == S_OK, "Could not get IHTMLDocument2 iface: %08x\n", hres); + + return doc; +} + +#define get_window_doc(e) _get_window_doc(__LINE__,e) +static IHTMLDocument2 *_get_window_doc(unsigned line, IHTMLWindow2 *window) +{ + IHTMLDocument2 *doc; + HRESULT hres; + + doc = NULL; + hres = IHTMLWindow2_get_document(window, &doc); + ok(hres == S_OK, "get_document failed: %08x\n", hres); + ok(doc != NULL, "disp == NULL\n"); + + return doc; +} + #define test_create_elem(d,t) _test_create_elem(__LINE__,d,t) static IHTMLElement *_test_create_elem(unsigned line, IHTMLDocument2 *doc, const char *tag) { @@ -4806,12 +4859,27 @@ static void doc_write(IHTMLDocument2 *doc, BOOL ln, const char *text) SafeArrayDestroy(sa); } +static void test_frame_doc(IUnknown *frame_elem) +{ + IHTMLDocument2 *window_doc, *elem_doc; + IHTMLWindow2 *content_window; + + content_window = get_frame_content_window(frame_elem); + window_doc = get_window_doc(content_window); + IHTMLWindow2_Release(content_window); + + elem_doc = get_elem_doc(frame_elem); + ok(iface_cmp((IUnknown*)window_doc, (IUnknown*)elem_doc), "content_doc != elem_doc\n"); + + IHTMLDocument2_Release(elem_doc); + IHTMLDocument2_Release(window_doc); +} + static void test_iframe_elem(IHTMLElement *elem) { IHTMLDocument2 *content_doc, *owner_doc; IHTMLElementCollection *col; IHTMLWindow2 *content_window; - IHTMLFrameBase2 *base2; IDispatch *disp; VARIANT errv; BSTR str; @@ -4825,23 +4893,13 @@ static void test_iframe_elem(IHTMLElement *elem) ET_BR }; - hres = IHTMLElement_QueryInterface(elem, &IID_IHTMLFrameBase2, (void**)&base2); - ok(hres == S_OK, "Could not get IHTMFrameBase2 iface: %08x\n", hres); - if (!base2) return; - - content_window = NULL; - hres = IHTMLFrameBase2_get_contentWindow(base2, &content_window); - IHTMLFrameBase2_Release(base2); - ok(hres == S_OK, "get_contentWindow failed: %08x\n", hres); - ok(content_window != NULL, "contentWindow = NULL\n"); + test_frame_doc((IUnknown*)elem); + content_window = get_frame_content_window((IUnknown*)elem); test_window_length(content_window, 0); - content_doc = NULL; - hres = IHTMLWindow2_get_document(content_window, &content_doc); + content_doc = get_window_doc(content_window); IHTMLWindow2_Release(content_window); - ok(hres == S_OK, "get_document failed: %08x\n", hres); - ok(content_doc != NULL, "content_doc = NULL\n"); str = a2bstr("text/html"); V_VT(&errv) = VT_ERROR; @@ -5114,7 +5172,7 @@ static void test_elems(IHTMLDocument2 *doc) elem = get_elem_by_id(doc, "s", TRUE); if(elem) { IHTMLSelectElement *select; - IHTMLDocument2 *doc_node; + IHTMLDocument2 *doc_node, *elem_doc; hres = IHTMLElement_QueryInterface(elem, &IID_IHTMLSelectElement, (void**)&select); ok(hres == S_OK, "Could not get IHTMLSelectElement interface: %08x\n", hres); @@ -5138,12 +5196,12 @@ static void test_elems(IHTMLDocument2 *doc) IHTMLSelectElement_Release(select); - hres = IHTMLElement_get_document(elem, &disp); - ok(hres == S_OK, "get_document failed: %08x\n", hres); + elem_doc = get_elem_doc((IUnknown*)elem); doc_node = get_doc_node(doc); - ok(iface_cmp((IUnknown*)disp, (IUnknown*)doc_node), "disp != doc\n"); + ok(iface_cmp((IUnknown*)elem_doc, (IUnknown*)doc_node), "disp != doc\n"); IHTMLDocument2_Release(doc_node); + IHTMLDocument2_Release(elem_doc); IHTMLElement_Release(elem); } @@ -5619,6 +5677,7 @@ static void test_frame(IDispatch *disp, const char *exp_id) if(FAILED(hres)) return; + test_frame_doc((IUnknown*)frame_elem); test_elem_id((IUnknown*)frame_elem, exp_id); IHTMLElement_Release(frame_elem); @@ -5721,7 +5780,7 @@ static void test_frameset(IHTMLDocument2 *doc) ok(hres == S_OK, "IHTMLFramesCollection2_item failed: 0x%08x\n", hres); if(SUCCEEDED(hres)) { ok(V_VT(&result_var) == VT_DISPATCH, "result type should have been VT_DISPATCH, was: 0x%x\n", V_VT(&result_var)); - test_frame((IDispatch*)V_DISPATCH(&result_var), "fr1"); + test_frame(V_DISPATCH(&result_var), "fr1"); } VariantClear(&result_var); VariantClear(&index_var);

1 0

Marcus Meissner : msxml3: Move element usage into element!=NULL check ( Coverity).
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: ed26bb09c5c155e4dafded9a6fe3343012ab8bb1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ed26bb09c5c155e4dafded9a6… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Wed Nov 25 10:14:47 2009 +0100 msxml3: Move element usage into element!=NULL check (Coverity). --- dlls/msxml3/tests/domdoc.c | 19 +++++++++---------- 1 files changed, 9 insertions(+), 10 deletions(-) diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index 43a90d7..a5b207a 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -1316,7 +1316,6 @@ todo_wine /* now traverse the tree from the root element */ if (element) { - IXMLDOMNode *node; r = IXMLDOMNode_get_childNodes( element, &list ); ok( r == S_OK, "get_childNodes returned wrong code\n"); @@ -1332,19 +1331,19 @@ todo_wine IXMLDOMNodeList_AddRef(list); expect_list_and_release(list, "E1.E2.D1 E2.E2.D1 E3.E2.D1 E4.E2.D1"); ole_check(IXMLDOMNodeList_reset(list)); + + node = (void*)0xdeadbeef; + r = IXMLDOMNode_selectSingleNode( element, szdl, &node ); + ok( r == S_FALSE, "ret %08x\n", r ); + ok( node == NULL, "node %p\n", node ); + r = IXMLDOMNode_selectSingleNode( element, szbs, &node ); + ok( r == S_OK, "ret %08x\n", r ); + r = IXMLDOMNode_Release( node ); + ok( r == 0, "ret %08x\n", r ); } else ok( FALSE, "no element\n"); - node = (void*)0xdeadbeef; - r = IXMLDOMNode_selectSingleNode( element, szdl, &node ); - ok( r == S_FALSE, "ret %08x\n", r ); - ok( node == NULL, "node %p\n", node ); - r = IXMLDOMNode_selectSingleNode( element, szbs, &node ); - ok( r == S_OK, "ret %08x\n", r ); - r = IXMLDOMNode_Release( node ); - ok( r == 0, "ret %08x\n", r ); - if (list) { r = IXMLDOMNodeList_QueryInterface(list, &IID_IDispatch, NULL);

1 0

Rob Shearman : widl: Detect interfaces that inherit from another interface as objects.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 6ed98b16a11de26145fbc24bc35b6309cc4e67b5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6ed98b16a11de26145fbc24bc… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Wed Nov 25 09:53:49 2009 +0000 widl: Detect interfaces that inherit from another interface as objects. --- tools/widl/header.c | 12 +++++++----- tools/widl/header.h | 2 +- tools/widl/parser.y | 4 ++-- tools/widl/proxy.c | 4 ++-- tools/widl/widl.c | 2 +- 5 files changed, 13 insertions(+), 11 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index c8b9613..5602b6d 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -691,10 +691,12 @@ int has_out_arg_or_return(const var_t *func) /********** INTERFACES **********/ -int is_object(const attr_list_t *list) +int is_object(const type_t *iface) { const attr_t *attr; - if (list) LIST_FOR_EACH_ENTRY( attr, list, const attr_t, entry ) + if (type_is_defined(iface) && type_iface_get_inherit(iface)) + return 1; + if (iface->attrs) LIST_FOR_EACH_ENTRY( attr, iface->attrs, const attr_t, entry ) if (attr->type == ATTR_OBJECT || attr->type == ATTR_ODL) return 1; return 0; } @@ -866,7 +868,7 @@ static void write_locals(FILE *fp, const type_t *iface, int body) = "/* WIDL-generated stub. You must provide an implementation for this. */"; const statement_t *stmt; - if (!is_object(iface->attrs)) + if (!is_object(iface)) return; STATEMENTS_FOR_EACH_FUNC(stmt, type_iface_get_stmts(iface)) { @@ -1180,7 +1182,7 @@ static void write_forward_decls(FILE *header, const statement_list_t *stmts) case STMT_TYPE: if (type_get_type(stmt->u.type) == TYPE_INTERFACE) { - if (is_object(stmt->u.type->attrs) || is_attr(stmt->u.type->attrs, ATTR_DISPINTERFACE)) + if (is_object(stmt->u.type) || is_attr(stmt->u.type->attrs, ATTR_DISPINTERFACE)) write_forward(header, stmt->u.type); } else if (type_get_type(stmt->u.type) == TYPE_COCLASS) @@ -1215,7 +1217,7 @@ static void write_header_stmts(FILE *header, const statement_list_t *stmts, cons if (type_get_type(stmt->u.type) == TYPE_INTERFACE) { type_t *iface = stmt->u.type; - if (is_attr(stmt->u.type->attrs, ATTR_DISPINTERFACE) || is_object(stmt->u.type->attrs)) + if (is_attr(stmt->u.type->attrs, ATTR_DISPINTERFACE) || is_object(stmt->u.type)) { write_com_interface_start(header, iface); write_header_stmts(header, type_iface_get_stmts(iface), stmt->u.type, TRUE); diff --git a/tools/widl/header.h b/tools/widl/header.h index d4b3b88..7d6c543 100644 --- a/tools/widl/header.h +++ b/tools/widl/header.h @@ -38,7 +38,7 @@ extern void write_type_def_or_decl(FILE *h, type_t *t, int is_field, const char extern void write_type_decl(FILE *f, type_t *t, const char *name); extern void write_type_decl_left(FILE *f, type_t *t); extern int needs_space_after(type_t *t); -extern int is_object(const attr_list_t *list); +extern int is_object(const type_t *iface); extern int is_local(const attr_list_t *list); extern int need_stub(const type_t *iface); extern int need_proxy(const type_t *iface); diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 73b48a2..dda4a37 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -841,7 +841,7 @@ dispinterfacedef: dispinterfacehdr '{' ; inherit: { $$ = NULL; } - | ':' aKNOWNTYPE { $$ = find_type_or_error2($2, 0); } + | ':' aKNOWNTYPE { $$ = find_type_or_error2($2, 0); is_object_interface = 1; } ; interface: tINTERFACE aIDENTIFIER { $$ = get_type(TYPE_INTERFACE, $2, 0); } @@ -852,9 +852,9 @@ interfacehdr: attributes interface { $$.interface = $2; $$.old_pointer_default = pointer_default; if (is_attr($1, ATTR_POINTERDEFAULT)) pointer_default = get_attrv($1, ATTR_POINTERDEFAULT); - is_object_interface = is_object($1); check_def($2); $2->attrs = check_iface_attrs($2->name, $1); + is_object_interface = is_object($2); $2->defined = TRUE; } ; diff --git a/tools/widl/proxy.c b/tools/widl/proxy.c index cc753bb..de8e466 100644 --- a/tools/widl/proxy.c +++ b/tools/widl/proxy.c @@ -698,12 +698,12 @@ static int does_any_iface(const statement_list_t *stmts, type_pred_t pred) int need_proxy(const type_t *iface) { - return is_object(iface->attrs) && !is_local(iface->attrs); + return is_object(iface) && !is_local(iface->attrs); } int need_stub(const type_t *iface) { - return !is_object(iface->attrs) && !is_local(iface->attrs); + return !is_object(iface) && !is_local(iface->attrs); } int need_proxy_file(const statement_list_t *stmts) diff --git a/tools/widl/widl.c b/tools/widl/widl.c index c1c3832..a481c56 100644 --- a/tools/widl/widl.c +++ b/tools/widl/widl.c @@ -431,7 +431,7 @@ static void write_id_data_stmts(const statement_list_t *stmts) if (type_get_type(type) == TYPE_INTERFACE) { const UUID *uuid; - if (!is_object(type->attrs) && !is_attr(type->attrs, ATTR_DISPINTERFACE)) + if (!is_object(type) && !is_attr(type->attrs, ATTR_DISPINTERFACE)) continue; uuid = get_attrp(type->attrs, ATTR_UUID); write_guid(idfile, is_attr(type->attrs, ATTR_DISPINTERFACE) ? "DIID" : "IID",

1 0

Rob Shearman : oleaut32: Fix size of memory allocated for byrefs in VARIANT_UserMarshal.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 489b73dd9e76aa4fc4c8aa6e4fdbf137d437a318 URL: http://source.winehq.org/git/wine.git/?a=commit;h=489b73dd9e76aa4fc4c8aa6e4… Author: Rob Shearman <robertshearman(a)gmail.com> Date: Wed Nov 25 09:54:27 2009 +0000 oleaut32: Fix size of memory allocated for byrefs in VARIANT_UserMarshal. --- dlls/oleaut32/usrmarshal.c | 19 +++++++++++++++++-- 1 files changed, 17 insertions(+), 2 deletions(-) diff --git a/dlls/oleaut32/usrmarshal.c b/dlls/oleaut32/usrmarshal.c index 6ad9bd6..7409fce 100644 --- a/dlls/oleaut32/usrmarshal.c +++ b/dlls/oleaut32/usrmarshal.c @@ -552,14 +552,29 @@ unsigned char * WINAPI VARIANT_UserUnmarshal(ULONG *pFlags, unsigned char *Buffe if(header->vt & VT_BYREF) { + ULONG mem_size; Pos += 4; + + switch (header->vt & ~VT_BYREF) + { + /* these types have a different memory size compared to wire size */ + case VT_UNKNOWN: + case VT_DISPATCH: + case VT_BSTR: + mem_size = sizeof(void *); + break; + default: + mem_size = type_size; + break; + } + if (V_VT(pvar) != header->vt) { VariantClear(pvar); - V_BYREF(pvar) = CoTaskMemAlloc(type_size); + V_BYREF(pvar) = CoTaskMemAlloc(mem_size); } else if (!V_BYREF(pvar)) - V_BYREF(pvar) = CoTaskMemAlloc(type_size); + V_BYREF(pvar) = CoTaskMemAlloc(mem_size); memcpy(V_BYREF(pvar), Pos, type_size); if((header->vt & VT_TYPEMASK) != VT_VARIANT) Pos += type_size;

1 0

Paul Vriens : shell32/tests: Fix some test failures on Windows 7.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 1da30597e41b7b05c98c4648e99ca723aa951926 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1da30597e41b7b05c98c4648e… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Wed Nov 25 11:15:48 2009 +0100 shell32/tests: Fix some test failures on Windows 7. --- dlls/shell32/tests/shlfolder.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index ed3a8b5..831205a 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -472,7 +472,8 @@ static void test_GetDisplayName(void) /* WinXP and up store the filenames as both ANSI and UNICODE in the pidls */ if (pidlLast->mkid.cb >= 76) { ok(!lstrcmpW((WCHAR*)&pidlLast->mkid.abID[46], wszFileName) || - (pidlLast->mkid.cb >= 94 && !lstrcmpW((WCHAR*)&pidlLast->mkid.abID[64], wszFileName)), /* Vista */ + (pidlLast->mkid.cb >= 94 && !lstrcmpW((WCHAR*)&pidlLast->mkid.abID[64], wszFileName)) || /* Vista */ + (pidlLast->mkid.cb >= 98 && !lstrcmpW((WCHAR*)&pidlLast->mkid.abID[68], wszFileName)), /* Win7 */ "Filename should be stored as wchar-string at this position!\n"); } @@ -1529,7 +1530,8 @@ static void test_ITEMIDLIST_format(void) { "Last write time should match last access time!\n"); ok (!lstrcmpW(wszFile[i], pFileStructW->wszName) || - !lstrcmpW(wszFile[i], (WCHAR *)(pFileStructW->abFooBar2 + 22)), /* Vista */ + !lstrcmpW(wszFile[i], (WCHAR *)(pFileStructW->abFooBar2 + 22)) || /* Vista */ + !lstrcmpW(wszFile[i], (WCHAR *)(pFileStructW->abFooBar2 + 26)), /* Win7 */ "The filename should be stored in unicode at this position!\n"); } }

1 0

Maarten Lankhorst : openal32: Add alBufferDataStatic.
by Alexandre Julliard 30 Nov '09

30 Nov '09

Module: wine Branch: master Commit: 49b39a88155117a6bf73a5531d3da4402b45bad7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=49b39a88155117a6bf73a5531… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Tue Nov 24 12:39:39 2009 +0100 openal32: Add alBufferDataStatic. --- dlls/openal32/openal.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/dlls/openal32/openal.c b/dlls/openal32/openal.c index 3d2bb00..04d0d35 100644 --- a/dlls/openal32/openal.c +++ b/dlls/openal32/openal.c @@ -44,6 +44,7 @@ typedef struct wine_ALCcontext { ALboolean been_current; + ALvoid (AL_APIENTRY*alBufferDataStatic)(const ALuint bid, ALenum format, const ALvoid* data, ALsizei size, ALsizei freq); ALvoid (AL_APIENTRY*alGenFilters)(ALsizei n, ALuint* filters); ALvoid (AL_APIENTRY*alDeleteFilters)(ALsizei n, const ALuint* filters); ALboolean (AL_APIENTRY*alIsFilter)(ALuint fid); @@ -190,6 +191,7 @@ ALCboolean CDECL wine_alcMakeContextCurrent(ALCcontext *context) CurrentCtx->been_current = AL_TRUE; #define LOADFUNC(x) CurrentCtx->x = alGetProcAddress(#x) + LOADFUNC(alBufferDataStatic); LOADFUNC(alGenFilters); LOADFUNC(alDeleteFilters); LOADFUNC(alIsFilter); @@ -620,6 +622,11 @@ ALvoid CDECL wine_alBufferData(ALuint bid, ALenum format, const ALvoid* data, AL alBufferData(bid, format, data, size, freq); } +ALvoid CDECL wine_alBufferDataStatic(const ALuint bid, ALenum format, const ALvoid* data, ALsizei size, ALsizei freq) +{ + CurrentCtx->alBufferDataStatic(bid, format, data, size, freq); +} + ALvoid CDECL wine_alGetBufferf(ALuint bid, ALenum param, ALfloat* value) { alGetBufferf(bid, param, value); @@ -1008,6 +1015,7 @@ static const struct FuncList ALFuncs[] = { { "alDeleteBuffers", wine_alDeleteBuffers }, { "alIsBuffer", wine_alIsBuffer }, { "alBufferData", wine_alBufferData }, + { "alBufferDataStatic", wine_alBufferDataStatic }, { "alBufferf", wine_alBufferf }, { "alBuffer3f", wine_alBuffer3f }, { "alBufferfv", wine_alBufferfv },

1 0