GH-887 fix patch file removal

1 files changed, 16 insertions, 1 deletions

diff --git a/logic/minecraft/OneSixProfileStrategy.cpp b/logic/minecraft/OneSixProfileStrategy.cpp
index 4f596d12..12503a80 100644
--- a/logic/minecraft/OneSixProfileStrategy.cpp
+++ b/logic/minecraft/OneSixProfileStrategy.cpp
@@ -186,6 +186,15 @@ bool OneSixProfileStrategy::removePatch(ProfilePatchPtr patch)
 	bool ok = true;
 	// first, remove the patch file. this ensures it's not used anymore
 	auto fileName = patch->getPatchFilename();
+	if(fileName.size())
+	{
+		QFile patchFile(fileName);
+		if(patchFile.exists() && !patchFile.remove())
+		{
+			qCritical() << "File" << fileName << "could not be removed because:" << patchFile.errorString();
+			return false;
+		}
+	}
 
 
 	auto preRemoveJarMod = [&](JarmodPtr jarMod) -> bool
@@ -194,7 +203,13 @@ bool OneSixProfileStrategy::removePatch(ProfilePatchPtr patch)
 		QFileInfo finfo (fullpath);
 		if(finfo.exists())
 		{
-			return QFile::remove(fullpath);
+			QFile jarModFile(fullpath);
+			if(!jarModFile.remove())
+			{
+				qCritical() << "File" << fullpath << "could not be removed because:" << jarModFile.errorString();
+				return false;
+			}
+			return true;
 		}
 		return true;
 	};