Commit 30351482 authored by Michael Karcher's avatar Michael Karcher Committed by Alexandre Julliard

msxml3: Fail a bit earlier on removeChild(...,NULL,...).

parent 75606f35
......@@ -625,11 +625,11 @@ static HRESULT WINAPI xmlnode_removeChild(
TRACE("%p->(%p, %p)\n", This, childNode, oldChild);
if(!childNode) return E_INVALIDARG;
if(oldChild)
*oldChild = NULL;
if(!childNode) return E_INVALIDARG;
hr = IXMLDOMNode_QueryInterface(childNode, &IID_IXMLDOMNode, (LPVOID)&child);
if(FAILED(hr))
return hr;
......
......@@ -1792,7 +1792,7 @@ static void test_removeChild(void)
removed_node = (void*)0xdeadbeef;
r = IXMLDOMElement_removeChild( element, NULL, &removed_node );
ok( r == E_INVALIDARG, "ret %08x\n", r );
todo_wine ok( removed_node == (void*)0xdeadbeef, "%p\n", removed_node );
ok( removed_node == (void*)0xdeadbeef, "%p\n", removed_node );
/* ba_node is a descendant of element, but not a direct child. */
removed_node = (void*)0xdeadbeef;
......
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