[PATCH 1/1] msi: Fix an use-after-free (Coverity).

[Zhiyi Zhang]

From: Zhiyi Zhang

---
 dlls/msi/files.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/dlls/msi/files.c b/dlls/msi/files.c
index 96c6c5eaa4b..1dbdc72f4f8 100644
--- a/dlls/msi/files.c
+++ b/dlls/msi/files.c
@@ -988,7 +988,7 @@ static BOOL move_files_wildcard( MSIPACKAGE *package, const WCHAR *source, WCHAR
     HANDLE hfile;
     LPWSTR path;
     BOOL res;
-    struct file_list files, *file;
+    struct file_list files, *file, *file2;
     DWORD size;

hfile = msi_find_first_file( package, source, &wfd );
@@ -1032,10 +1032,8 @@ static BOOL move_files_wildcard( MSIPACKAGE *package, const WCHAR *source, WCHAR
     file->dest[size - 1] = '\0';
     lstrcpyW(wcsrchr(file->dest, '\') + 1, file->destname);

-    while (!list_empty(&files.entry))
+    LIST_FOR_EACH_ENTRY_SAFE(file, file2, &files.entry, struct file_list, entry)
     {
-        file = LIST_ENTRY(list_head(&files.entry), struct file_list, entry);
-
         move_file( package, file->source, file->dest, options );

list_remove(&file->entry);