[PATCH v4 0/1] MR8914: msxml: IXMLDOMNamedNodeMap::getNamedItem to handle global namespace.

[Alistair Leslie-Hughes (＠alesliehughes)]

--
  v4: msxml: IXMLDOMNamedNodeMap::getNamedItem to handle global namespace.

https://gitlab.winehq.org/wine/wine/-/merge_requests/8914

[PATCH v4 1/1] msxml: IXMLDOMNamedNodeMap::getNamedItem to handle global namespace.

[Alistair Leslie-Hughes]

From: Alistair Leslie-Hughes

---
 dlls/msxml3/element.c      | 26 +++++++++++++++++++++++++-
 dlls/msxml3/tests/domdoc.c | 25 ++++++++++++++++++++++++-
 2 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/dlls/msxml3/element.c b/dlls/msxml3/element.c
index fd9a2973cd8..1f5ede06f25 100644
--- a/dlls/msxml3/element.c
+++ b/dlls/msxml3/element.c
@@ -1661,7 +1661,31 @@ static HRESULT domelem_get_named_item(const xmlNodePtr node, BSTR name, IXMLDOMN
         return domelem_get_qualified_item(node, name, NULL, item);

/* try to get namespace uri for supplied qualified name */
-    ns = xmlSearchNs(node->doc, node, prefix);
+    if (xmlStrEqual(prefix, BAD_CAST "xmlns") )
+    {
+        IUnknown *unk;
+
+        xmlFree(prefix);
+
+        ns = xmlSearchNs(node->doc, node, local);
+        xmlFree(local);
+        if (!ns)
+        {
+            if (item) *item = NULL;
+            return item ? S_FALSE : E_INVALIDARG;
+        }
+
+        unk = create_attribute(node, FALSE);
+        if (!unk)
+        {
+            return E_OUTOFMEMORY;
+        }
+        hr = IUnknown_QueryInterface(unk, &IID_IXMLDOMNode, (void**)item);
+        IUnknown_Release(unk);
+        return hr;
+    }
+    else
+        ns = xmlSearchNs(node->doc, node, prefix);

xmlFree(prefix);

diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c
index a08506fad9b..1910e4bf0aa 100644
--- a/dlls/msxml3/tests/domdoc.c
+++ b/dlls/msxml3/tests/domdoc.c
@@ -10264,6 +10264,7 @@ static void test_get_attributes(void)
     IXMLDOMDocument *doc, *doc2;
     IXMLDOMNode *node, *node2;
     IXMLDOMElement *elem;
+    DOMNodeType type;
     VARIANT_BOOL b;
     HRESULT hr;
     VARIANT v;
@@ -10303,7 +10304,6 @@ static void test_get_attributes(void)
     if (hr == S_OK && length == 1)
     {
         IXMLDOMAttribute *attr;
-        DOMNodeType type;
         VARIANT v;

node2 = NULL;
@@ -10579,6 +10579,29 @@ static void test_get_attributes(void)
         IXMLDOMNode_Release(node2);
     }

+    hr = IXMLDOMNamedNodeMap_getNamedItem(map, _bstr_("xmlns:foaf"), &node2);
+    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
+    if (hr == S_OK)
+    {
+        type = -1;
+        hr = IXMLDOMNode_get_nodeType(node2, &type);
+        ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
+        ok(type == NODE_ATTRIBUTE, "Unexpected type %d.\n", type);
+        IXMLDOMNode_Release(node2);
+    }
+
+    hr = IXMLDOMNamedNodeMap_getNamedItem(map, _bstr_("dcterms:created"), &node2);
+    ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
+    if (hr == S_OK)
+    {
+        type = -1;
+        hr = IXMLDOMNode_get_nodeType(node2, &type);
+        ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
+        ok(type == NODE_ATTRIBUTE, "Unexpected type %d.\n", type);
+        IXMLDOMNode_Release(node2);
+    }
+
+
     IXMLDOMNamedNodeMap_Release(map);
     IXMLDOMElement_Release(elem);