iqbalrifai
/
Aegisub
mirror of https://github.com/odrling/Aegisub
1
0
Fork 0
Code Issues Releases Wiki Activity

Retry commits of file writes for up to a second to work around AV scanning 

Poorly-written antivirus software briefly lock newly written files to
scan them for viruses, which makes the rename from the temp file to
actual file fail. Work around this by retrying the rename up to ten
times.

Closes #1618.
This commit is contained in:
Thomas Goyne 2013-07-03 07:12:06 -07:00
parent 38dedfe1d9
commit 69b05d49de
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
aegisub/libaegisub/common/io.cpp
View File

@ -97,7 +97,22 @@ Save::Save(const std::string& file, bool binary)
Save::~Save() {
delete fp;
#ifndef _WIN32
util::Rename(tmp_name, file_name);
#else
for (int i = 0; i < 10; ++i) {
try {
util::Rename(tmp_name, file_name);
return;
}
catch (agi::FileNotAccessibleError const&) {
// Retry up to ten times in case it's just locked by a poorly-written antivirus scanner
if (i == 9)
throw;
Sleep(100);
}
}
#endif
}
std::ofstream& Save::Get() {