Commit ed26bb09 authored by Marcus Meissner's avatar Marcus Meissner Committed by Alexandre Julliard

msxml3: Move element usage into element!=NULL check (Coverity).

parent 1fccc1b4
...@@ -1316,7 +1316,6 @@ todo_wine ...@@ -1316,7 +1316,6 @@ todo_wine
/* now traverse the tree from the root element */ /* now traverse the tree from the root element */
if (element) if (element)
{ {
IXMLDOMNode *node;
r = IXMLDOMNode_get_childNodes( element, &list ); r = IXMLDOMNode_get_childNodes( element, &list );
ok( r == S_OK, "get_childNodes returned wrong code\n"); ok( r == S_OK, "get_childNodes returned wrong code\n");
...@@ -1332,9 +1331,6 @@ todo_wine ...@@ -1332,9 +1331,6 @@ todo_wine
IXMLDOMNodeList_AddRef(list); IXMLDOMNodeList_AddRef(list);
expect_list_and_release(list, "E1.E2.D1 E2.E2.D1 E3.E2.D1 E4.E2.D1"); expect_list_and_release(list, "E1.E2.D1 E2.E2.D1 E3.E2.D1 E4.E2.D1");
ole_check(IXMLDOMNodeList_reset(list)); ole_check(IXMLDOMNodeList_reset(list));
}
else
ok( FALSE, "no element\n");
node = (void*)0xdeadbeef; node = (void*)0xdeadbeef;
r = IXMLDOMNode_selectSingleNode( element, szdl, &node ); r = IXMLDOMNode_selectSingleNode( element, szdl, &node );
...@@ -1344,6 +1340,9 @@ todo_wine ...@@ -1344,6 +1340,9 @@ todo_wine
ok( r == S_OK, "ret %08x\n", r ); ok( r == S_OK, "ret %08x\n", r );
r = IXMLDOMNode_Release( node ); r = IXMLDOMNode_Release( node );
ok( r == 0, "ret %08x\n", r ); ok( r == 0, "ret %08x\n", r );
}
else
ok( FALSE, "no element\n");
if (list) if (list)
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment