Prevent accessing freed memory in failure case.

diff --git a/dlls/msi/action.c b/dlls/msi/action.c
index e773fea0fb854536e5a83b85a65ed00af82c4119..92d9c987d4063fe67eddf406c57384a1aa7261da 100644 (file)
--- a/dlls/msi/action.c
+++ b/dlls/msi/action.c
@@ -3206,7 +3206,6 @@ static UINT ACTION_DuplicateFiles(MSIPACKAGE *package)
         }
 
         dest = build_directory_name(2, dest_path, dest_name);
-        HeapFree(GetProcessHeap(), 0, dest_path);
            
         TRACE("Duplicating file %s to %s\n",debugstr_w(file_source),
               debugstr_w(dest)); 
@@ -3222,6 +3221,7 @@ static UINT ACTION_DuplicateFiles(MSIPACKAGE *package)
         FIXME("We should track these duplicate files as well\n");   
  
         msiobj_release(&row->hdr);
+        HeapFree(GetProcessHeap(),0,dest_path);
         HeapFree(GetProcessHeap(),0,dest);
         HeapFree(GetProcessHeap(),0,file_source);
     }