[PATCH] shell32: Don't return a path from SHGetKnownFolderPath when it can not be created.

Sven Baars sven.wine at gmail.com
Wed Feb 6 04:48:17 CST 2019


Signed-off-by: Sven Baars <sven.wine at gmail.com>
---
 dlls/shell32/shellpath.c       | 2 +-
 dlls/shell32/tests/shellpath.c | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/dlls/shell32/shellpath.c b/dlls/shell32/shellpath.c
index f08474b363..8c6e8a89b2 100644
--- a/dlls/shell32/shellpath.c
+++ b/dlls/shell32/shellpath.c
@@ -4970,7 +4970,7 @@ HRESULT WINAPI SHGetKnownFolderPath(REFKNOWNFOLDERID rfid, DWORD flags, HANDLE t
     if (!(flags & KF_FLAG_CREATE))
     {
         hr = HRESULT_FROM_WIN32(ERROR_PATH_NOT_FOUND);
-        goto done;
+        goto failed;
     }
 
     /* create directory/directories */
diff --git a/dlls/shell32/tests/shellpath.c b/dlls/shell32/tests/shellpath.c
index 38b56f01d3..f768f44c8b 100644
--- a/dlls/shell32/tests/shellpath.c
+++ b/dlls/shell32/tests/shellpath.c
@@ -1875,7 +1875,10 @@ if (0) { /* crashes */
         path = NULL;
         hr = pSHGetKnownFolderPath(folder_id, KF_FLAG_DEFAULT, NULL, &path);
         if (FAILED(hr))
+        {
+            ok(path == NULL, "expected path == NULL\n");
             continue;
+        }
         ok(hr == S_OK, "expected S_OK, got 0x%08x\n", hr);
         ok(path != NULL, "expected path != NULL\n");
 
-- 
2.17.1




More information about the wine-devel mailing list