[PATCH v4 3/6] mshtml: Use a BSTR to store the detached attribute's name.

[Gabriel Ivăncescu]

From: Gabriel Ivăncescu

Signed-off-by: Gabriel Ivăncescu 
---
 dlls/mshtml/htmlattr.c       | 6 +++---
 dlls/mshtml/mshtml_private.h | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/dlls/mshtml/htmlattr.c b/dlls/mshtml/htmlattr.c
index 82d6aaf9476..a83298a7ea1 100644
--- a/dlls/mshtml/htmlattr.c
+++ b/dlls/mshtml/htmlattr.c
@@ -51,7 +51,7 @@ static HRESULT WINAPI HTMLDOMAttribute_get_nodeName(IHTMLDOMAttribute *iface, BS
             return E_FAIL;
         }

-        *p = SysAllocString(This->name);
+        *p = SysAllocStringLen(This->name, SysStringLen(This->name));
         return *p ? S_OK : E_OUTOFMEMORY;
     }

@@ -537,7 +537,7 @@ static void HTMLDOMAttribute_destructor(DispatchEx *dispex)
 {
     HTMLDOMAttribute *This = impl_from_DispatchEx(dispex);
     VariantClear(&This->value);
-    free(This->name);
+    SysFreeString(This->name);
     free(This);
 }

@@ -609,7 +609,7 @@ HRESULT HTMLDOMAttribute_Create(const WCHAR *name, HTMLElement *elem, DISPID dis

/* For detached attributes we may still do most operations if we have its name available. */
     if(name) {
-        ret->name = wcsdup(name);
+        ret->name = SysAllocString(name);
         if(!ret->name) {
             IHTMLDOMAttribute_Release(&ret->IHTMLDOMAttribute_iface);
             return E_OUTOFMEMORY;
diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h
index 4319c8b4725..ca49312fbe3 100644
--- a/dlls/mshtml/mshtml_private.h
+++ b/dlls/mshtml/mshtml_private.h
@@ -1341,7 +1341,7 @@ typedef struct {
     /* value is valid only for detached attributes (when elem == NULL). */
     VARIANT value;
     /* name must be valid for detached attributes */
-    WCHAR *name;
+    BSTR name;

HTMLDocumentNode *doc;
     HTMLElement *elem;