msi: Translate the resulting string only if the previous call succeeded.

0d13a99e · Dmitry Timoshkov · Alexandre Julliard · bcd6e9f3 · 0d13a99e
Commit 0d13a99e authored May 25, 2006 by Dmitry Timoshkov Committed by Alexandre Julliard May 25, 2006
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

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

No files found.
--- a/dlls/msi/msi.c
+++ b/dlls/msi/msi.c
@@ -1610,8 +1610,10 @@ UINT WINAPI MsiProvideQualifiedComponentA( LPCSTR szComponent,

    msi_free(szwComponent);
    msi_free(szwQualifier);
-    *pcchPathBuf = WideCharToMultiByte(CP_ACP, 0, lpwPathBuf, pcchwPathBuf,
-                    lpPathBuf, *pcchPathBuf, NULL, NULL);
+
+    if (rc == ERROR_SUCCESS)
+        *pcchPathBuf = WideCharToMultiByte(CP_ACP, 0, lpwPathBuf, pcchwPathBuf,
+                                           lpPathBuf, *pcchPathBuf, NULL, NULL);

    msi_free(lpwPathBuf);
    return rc;