Merge pull request #3371 from Be-ing/fix_new_config

ConfigObject: fix creation of new config file

1 files changed, 7 insertions, 4 deletions

diff --git a/src/preferences/configobject.cpp b/src/preferences/configobject.cpp
index 2e25ea076e..74610868cf 100644
--- a/src/preferences/configobject.cpp
+++ b/src/preferences/configobject.cpp
@@ -237,10 +237,13 @@ bool ConfigObject<ValueType>::save() {
     }
 
     QFile oldConfig(m_filename);
-    if (!oldConfig.remove()) {
-        qWarning().nospace() << "Could not remove old config file: "
-                             << oldConfig.fileName() << ": " << oldConfig.errorString();
-        return false;
+    // Trying to remove a file that does not exist would fail
+    if (oldConfig.exists()) {
+        if (!oldConfig.remove()) {
+            qWarning().nospace() << "Could not remove old config file: "
+                                 << oldConfig.fileName() << ": " << oldConfig.errorString();
+            return false;
+        }
     }
     if (!tmpFile.rename(m_filename)) {
         qWarning().nospace() << "Could not rename tmp file to config file: "