[PATCH v2 3/4] kernelbase: Fix DeleteFileA doesn't support long path.

[yaoyongjie]

From: yaoyongjie

If alloc is FALSE, file_name_AtoW will uses the TEB static buffer,
but the buffer size is MAX_PATH+1 . If DeleteFileA
is to support long paths prefixed with "\\?\", alloc must be set to TRUE.
---
 dlls/kernel32/tests/file.c | 4 ++--
 dlls/kernelbase/file.c     | 7 +++++--
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c
index 987faf7468f..f9c4d5583c5 100644
--- a/dlls/kernel32/tests/file.c
+++ b/dlls/kernel32/tests/file.c
@@ -1311,9 +1311,9 @@ static void test_CopyFileEx(void)
     retok = CopyFileExA(long_path_1, long_path_2, NULL, NULL, NULL, 0);
     ok(retok, "CopyFileExA failed, got %ld, copy %s -> %s failed\n", GetLastError(), long_path_1, long_path_2);
     ret = DeleteFileA(long_path_1);
-    todo_wine ok(ret, "DeleteFileA failed with error %ld\n", GetLastError());
+    ok(ret, "DeleteFileA failed with error %ld\n", GetLastError());
     ret = DeleteFileA(long_path_2);
-    todo_wine ok(ret, "DeleteFileA failed with error %ld\n", GetLastError());
+    ok(ret, "DeleteFileA failed with error %ld\n", GetLastError());

ret = DeleteFileA(source);
     ok(ret, "DeleteFileA failed with error %ld\n", GetLastError());
diff --git a/dlls/kernelbase/file.c b/dlls/kernelbase/file.c
index c3d87a84fd5..ab601246d28 100644
--- a/dlls/kernelbase/file.c
+++ b/dlls/kernelbase/file.c
@@ -1003,9 +1003,12 @@ BOOLEAN WINAPI /* DECLSPEC_HOTPATCH */ CreateSymbolicLinkW( LPCWSTR link, LPCWST
 BOOL WINAPI DECLSPEC_HOTPATCH DeleteFileA( LPCSTR path )
 {
     WCHAR *pathW;
+    BOOL res;

-    if (!(pathW = file_name_AtoW( path, FALSE ))) return FALSE;
-    return DeleteFileW( pathW );
+    if (!(pathW = file_name_AtoW( path, TRUE ))) return FALSE;
+    res = DeleteFileW( pathW );
+    HeapFree( GetProcessHeap(), 0, pathW );
+    return res;
 }