[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