archive/dolphin
1
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2024-09-20 19:31:53 +02:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #11197 from shuffle2/updater

Browse source 
WinUpdater: fix renaming of Updater
This commit is contained in:
JosJuice 2022-10-23 10:44:24 +02:00 committed by GitHub
commit 100121c0b1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
Source/Core/UpdaterCommon/UpdaterCommon.cpp
View file

@ -464,16 +464,10 @@ bool UpdateFiles(const std::vector<TodoList::UpdateOp>& to_update,
// after entire update has completed, and have it delete our relocated executable. For now we
// just let the relocated file hang around.
// It is enough to match based on filename, don't need File/VolumeId etc.
if (op.filename == self_filename)
{
auto reloc_path = self_path;
reloc_path.replace_filename("Updater.2.exe");
if (!MoveFile(self_path.wstring().c_str(), reloc_path.wstring().c_str()))
{
fprintf(log_fp, "Failed to move %s.\n", op.filename.c_str());
// Just let the Copy fail, later.
}
}
const bool is_self = op.filename == self_filename;
#else
// On other platforms, the renaming is handled by Dolphin before running the Updater.
const bool is_self = false;
#endif
std::string contents;
@ -488,7 +482,7 @@ bool UpdateFiles(const std::vector<TodoList::UpdateOp>& to_update,
fprintf(log_fp, "File %s was already up to date. Partial update?\n", op.filename.c_str());
continue;
}
else if (!op.old_hash || contents_hash != *op.old_hash)
else if (!op.old_hash || contents_hash != *op.old_hash || is_self)
{
if (!BackupFile(path))
return false;