comdlg32/tests: Make sure no ANSI strings are passed to Unicode functions.

dlls/comdlg32/tests/filedlg.c

@@ -86,31 +86,36 @@ static void test_DialogCancel(void)
     ok(CDERR_INITIALIZATION == CommDlgExtendedError(), "expected %d, got %d\n",
               CDERR_INITIALIZATION, CommDlgExtendedError());
 
-    SetLastError(0xdeadbeef);
-    result = GetOpenFileNameW((LPOPENFILENAMEW) &ofn);
-    if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
-        skip("GetOpenFileNameW is not implemented\n");
-    else
-    {
+    result = GetSaveFileNameA(&ofn);
     ok(0 == result, "expected %d, got %d\n", 0, result);
     ok(0 == CommDlgExtendedError(), "expected %d, got %d\n", 0,
        CommDlgExtendedError());
-    }
 
     PrintDlgA(NULL);
     ok(CDERR_INITIALIZATION == CommDlgExtendedError(), "expected %d, got %d\n",
               CDERR_INITIALIZATION, CommDlgExtendedError());
 
-    result = GetSaveFileNameA(&ofn);
-    ok(0 == result, "expected %d, got %d\n", 0, result);
-    ok(0 == CommDlgExtendedError(), "expected %d, got %d\n", 0,
-       CommDlgExtendedError());
+    /* Before passing the ofn to Unicode functions, remove the ANSI strings */
+    ofn.lpstrFilter = NULL;
+    ofn.lpstrInitialDir = NULL;
+    ofn.lpstrDefExt = NULL;
 
     PrintDlgA(NULL);
     ok(CDERR_INITIALIZATION == CommDlgExtendedError(), "expected %d, got %d\n",
               CDERR_INITIALIZATION, CommDlgExtendedError());
 
     SetLastError(0xdeadbeef);
+    result = GetOpenFileNameW((LPOPENFILENAMEW) &ofn);
+    if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
+        skip("GetOpenFileNameW is not implemented\n");
+    else
+    {
+        ok(0 == result, "expected %d, got %d\n", 0, result);
+        ok(0 == CommDlgExtendedError(), "expected %d, got %d\n", 0,
+           CommDlgExtendedError());
+    }
+
+    SetLastError(0xdeadbeef);
     result = GetSaveFileNameW((LPOPENFILENAMEW) &ofn);
     if (GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
         skip("GetSaveFileNameW is not implemented\n");