mshtml: Don't free a string too early.

c4a51b0b · Zhenbo Li · Alexandre Julliard · 531a6ceb · c4a51b0b
Commit c4a51b0b authored Apr 06, 2014 by Zhenbo Li Committed by Alexandre Julliard Apr 08, 2014
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

htmltablerow.c dlls/mshtml/htmltablerow.c +1 -1

No files found.
--- a/dlls/mshtml/htmltablerow.c
+++ b/dlls/mshtml/htmltablerow.c
@@ -200,7 +200,6 @@ static HRESULT WINAPI HTMLTableRow_get_bgColor(IHTMLTableRow *iface, VARIANT *p)

    nsAString_Init(&strColor, NULL);
    nsres = nsIDOMHTMLTableRowElement_GetBgColor(This->nsrow, &strColor);
-    nsAString_Finish(&strColor);

    if(NS_SUCCEEDED(nsres)) {
       nsAString_GetData(&strColor, &color);
@@ -211,6 +210,7 @@ static HRESULT WINAPI HTMLTableRow_get_bgColor(IHTMLTableRow *iface, VARIANT *p)
       hres = E_FAIL;
    }

+    nsAString_Finish(&strColor);
    return hres;
 }