[v2 PATCH] shell32: Initialize local buffer (Coverity)

Nikolay Sivov nsivov at codeweavers.com
Wed Dec 14 13:16:11 CST 2016


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
 dlls/shell32/shellpath.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/dlls/shell32/shellpath.c b/dlls/shell32/shellpath.c
index 1fa5f23..9230623 100644
--- a/dlls/shell32/shellpath.c
+++ b/dlls/shell32/shellpath.c
@@ -4735,6 +4735,7 @@ HRESULT WINAPI SHGetKnownFolderPath(REFKNOWNFOLDERID rfid, DWORD flags, HANDLE t
             hr = E_INVALIDARG;
             break;
         case CSIDL_Type_NonExistent:
+            *tempW = 0;
             hr = S_FALSE;
             break;
         case CSIDL_Type_WindowsPath:
@@ -4785,15 +4786,19 @@ HRESULT WINAPI SHGetKnownFolderPath(REFKNOWNFOLDERID rfid, DWORD flags, HANDLE t
             break;
     }
 
+    if (FAILED(hr))
+        goto failed;
+
     /* Expand environment strings if necessary */
     if (*tempW == '%')
+    {
         hr = _SHExpandEnvironmentStrings(tempW, pathW);
+        if (FAILED(hr))
+            goto failed;
+    }
     else
         strcpyW(pathW, tempW);
 
-    if (FAILED(hr))
-        goto failed;
-
     /* if we don't care about existing directories we are ready */
     if (flags & KF_FLAG_DONT_VERIFY) goto done;
 
-- 
2.10.2




More information about the wine-patches mailing list