msi: Use the return value of IXMLDOMNode_get_text (clang).

98c621c1 · Hans Leidekker · Alexandre Julliard · f8b29a06 · 98c621c1
Commit 98c621c1 authored Feb 06, 2012 by Hans Leidekker Committed by Alexandre Julliard Feb 06, 2012
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

msi.c dlls/msi/msi.c +5 -2

No files found.
--- a/dlls/msi/msi.c
+++ b/dlls/msi/msi.c
@@ -613,8 +613,11 @@ static UINT MSI_ApplicablePatchXML( MSIPACKAGE *package, IXMLDOMDocument *desc )
    {
        hr = IXMLDOMNode_get_text( node, &s );
        IXMLDOMNode_Release( node );
-        if (!strcmpW( s, product_code )) r = ERROR_SUCCESS;
-        SysFreeString(s);
+        if (hr == S_OK)
+        {
+            if (!strcmpW( s, product_code )) r = ERROR_SUCCESS;
+            SysFreeString( s );
+        }
    }
    IXMLDOMNodeList_Release( list );