mirror of
https://github.com/reactos/wine.git
synced 2024-11-25 20:59:54 +00:00
msi: Overwrite an existing read-only file when copying the install file.
This commit is contained in:
parent
a40d687133
commit
ba40c463d7
@ -638,13 +638,9 @@ static void schedule_install_files(MSIPACKAGE *package)
|
||||
}
|
||||
}
|
||||
|
||||
static UINT copy_install_file(MSIFILE *file)
|
||||
static UINT copy_file(MSIFILE *file)
|
||||
{
|
||||
BOOL ret;
|
||||
UINT gle;
|
||||
|
||||
TRACE("Copying %s to %s\n", debugstr_w(file->SourcePath),
|
||||
debugstr_w(file->TargetPath));
|
||||
|
||||
ret = CopyFileW(file->SourcePath, file->TargetPath, FALSE);
|
||||
if (ret)
|
||||
@ -653,7 +649,20 @@ static UINT copy_install_file(MSIFILE *file)
|
||||
return ERROR_SUCCESS;
|
||||
}
|
||||
|
||||
gle = GetLastError();
|
||||
return GetLastError();
|
||||
}
|
||||
|
||||
static UINT copy_install_file(MSIFILE *file)
|
||||
{
|
||||
UINT gle;
|
||||
|
||||
TRACE("Copying %s to %s\n", debugstr_w(file->SourcePath),
|
||||
debugstr_w(file->TargetPath));
|
||||
|
||||
gle = copy_file(file);
|
||||
if (gle == ERROR_SUCCESS)
|
||||
return gle;
|
||||
|
||||
if (gle == ERROR_ALREADY_EXISTS && file->state == msifs_overwrite)
|
||||
{
|
||||
TRACE("overwriting existing file\n");
|
||||
@ -665,6 +674,13 @@ static UINT copy_install_file(MSIFILE *file)
|
||||
TRACE("Source file not found\n");
|
||||
gle = ERROR_SUCCESS;
|
||||
}
|
||||
else if (gle == ERROR_ACCESS_DENIED)
|
||||
{
|
||||
SetFileAttributesW(file->TargetPath, FILE_ATTRIBUTE_NORMAL);
|
||||
|
||||
gle = copy_file(file);
|
||||
TRACE("Overwriting existing file: %d\n", gle);
|
||||
}
|
||||
else if (!(file->Attributes & msidbFileAttributesVital))
|
||||
{
|
||||
TRACE("Ignoring error for nonvital\n");
|
||||
|
@ -241,6 +241,31 @@ static const CHAR ui_custom_action_dat[] = "Action\tType\tSource\tTarget\tISComm
|
||||
"CustomAction\tAction\n"
|
||||
"SetUIProperty\t51\tHASUIRUN\t1\t\n";
|
||||
|
||||
static const CHAR rof_component_dat[] = "Component\tComponentId\tDirectory_\tAttributes\tCondition\tKeyPath\n"
|
||||
"s72\tS38\ts72\ti2\tS255\tS72\n"
|
||||
"Component\tComponent\n"
|
||||
"maximus\t\tMSITESTDIR\t0\t1\tmaximus\n";
|
||||
|
||||
static const CHAR rof_feature_dat[] = "Feature\tFeature_Parent\tTitle\tDescription\tDisplay\tLevel\tDirectory_\tAttributes\n"
|
||||
"s38\tS38\tL64\tL255\tI2\ti2\tS72\ti2\n"
|
||||
"Feature\tFeature\n"
|
||||
"feature\t\t\t\t2\t1\tTARGETDIR\t0";
|
||||
|
||||
static const CHAR rof_feature_comp_dat[] = "Feature_\tComponent_\n"
|
||||
"s38\ts72\n"
|
||||
"FeatureComponents\tFeature_\tComponent_\n"
|
||||
"feature\tmaximus";
|
||||
|
||||
static const CHAR rof_file_dat[] = "File\tComponent_\tFileName\tFileSize\tVersion\tLanguage\tAttributes\tSequence\n"
|
||||
"s72\ts72\tl255\ti4\tS72\tS20\tI2\ti2\n"
|
||||
"File\tFile\n"
|
||||
"maximus\tmaximus\tmaximus\t500\t\t\t8192\t1";
|
||||
|
||||
static const CHAR rof_media_dat[] = "DiskId\tLastSequence\tDiskPrompt\tCabinet\tVolumeLabel\tSource\n"
|
||||
"i2\ti4\tL64\tS255\tS32\tS72\n"
|
||||
"Media\tDiskId\n"
|
||||
"1\t1\t\t\tDISK1\t\n";
|
||||
|
||||
typedef struct _msi_table
|
||||
{
|
||||
const CHAR *filename;
|
||||
@ -339,6 +364,18 @@ static const msi_table ui_tables[] =
|
||||
ADD_TABLE(property),
|
||||
};
|
||||
|
||||
static const msi_table rof_tables[] =
|
||||
{
|
||||
ADD_TABLE(rof_component),
|
||||
ADD_TABLE(directory),
|
||||
ADD_TABLE(rof_feature),
|
||||
ADD_TABLE(rof_feature_comp),
|
||||
ADD_TABLE(rof_file),
|
||||
ADD_TABLE(install_exec_seq),
|
||||
ADD_TABLE(rof_media),
|
||||
ADD_TABLE(property),
|
||||
};
|
||||
|
||||
/* cabinet definitions */
|
||||
|
||||
/* make the max size large so there is only one cab file */
|
||||
@ -1120,6 +1157,56 @@ static void test_uiLevelFlags(void)
|
||||
DeleteFile(msifile);
|
||||
}
|
||||
|
||||
static BOOL file_matches(LPSTR path)
|
||||
{
|
||||
CHAR buf[MAX_PATH];
|
||||
HANDLE file;
|
||||
DWORD size;
|
||||
|
||||
file = CreateFile(path, GENERIC_READ, FILE_SHARE_READ | FILE_SHARE_WRITE,
|
||||
NULL, OPEN_EXISTING, 0, NULL);
|
||||
|
||||
ZeroMemory(buf, MAX_PATH);
|
||||
ReadFile(file, buf, 15, &size, NULL);
|
||||
CloseHandle(file);
|
||||
|
||||
return !lstrcmp(buf, "msitest\\maximus");
|
||||
}
|
||||
|
||||
static void test_readonlyfile(void)
|
||||
{
|
||||
UINT r;
|
||||
DWORD size;
|
||||
HANDLE file;
|
||||
CHAR path[MAX_PATH];
|
||||
|
||||
CreateDirectoryA("msitest", NULL);
|
||||
create_file("msitest\\maximus", 500);
|
||||
create_database(msifile, rof_tables, sizeof(rof_tables) / sizeof(msi_table));
|
||||
|
||||
MsiSetInternalUI(INSTALLUILEVEL_NONE, NULL);
|
||||
|
||||
lstrcpy(path, PROG_FILES_DIR);
|
||||
lstrcat(path, "\\msitest");
|
||||
CreateDirectory(path, NULL);
|
||||
|
||||
lstrcat(path, "\\maximus");
|
||||
file = CreateFile(path, GENERIC_READ, FILE_SHARE_READ | FILE_SHARE_WRITE,
|
||||
NULL, CREATE_NEW, FILE_ATTRIBUTE_READONLY, NULL);
|
||||
if (file == INVALID_HANDLE_VALUE) printf("didnt work here: %d\n", GetLastError());
|
||||
|
||||
WriteFile(file, "readonlyfile", 20, &size, NULL);
|
||||
CloseHandle(file);
|
||||
|
||||
r = MsiInstallProductA(msifile, NULL);
|
||||
ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r);
|
||||
ok(file_matches(path), "Expected file to be overwritten\n");
|
||||
ok(delete_pf("msitest\\maximus", TRUE), "File not installed\n");
|
||||
ok(delete_pf("msitest", FALSE), "File not installed\n");
|
||||
|
||||
DeleteFile(msifile);
|
||||
}
|
||||
|
||||
START_TEST(install)
|
||||
{
|
||||
DWORD len;
|
||||
@ -1145,6 +1232,7 @@ START_TEST(install)
|
||||
test_mixedmedia();
|
||||
test_samesequence();
|
||||
test_uiLevelFlags();
|
||||
test_readonlyfile();
|
||||
|
||||
SetCurrentDirectoryA(prev_path);
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user