Fix #2161

src/persistence/profile.cpp

@@ -332,8 +332,18 @@ void Profile::saveToxSave(QByteArray data)
     }
 
     saveFile.write(data);
-    saveFile.commit();
-    newProfile = false;
+
+    // check if everything got written
+    if(saveFile.flush())
+    {
+        saveFile.commit();
+        newProfile = false;
+    }
+    else
+    {
+        saveFile.cancelWriting();
+        qCritical() << "Failed to write, can't save!";
+    }
 }
 
 QString Profile::avatarPath(const QString &ownerId, bool forceUnencrypted)

src/persistence/settingsserializer.cpp

@@ -22,6 +22,7 @@
 #include "src/nexus.h"
 #include "src/persistence/profile.h"
 #include "src/core/core.h"
+#include <QSaveFile>
 #include <QFile>
 #include <QDebug>
 #include <memory>
@@ -219,7 +220,7 @@ void SettingsSerializer::load()
 
 void SettingsSerializer::save()
 {
-    QFile f(path);
+    QSaveFile f(path);
     if (!f.open(QIODevice::Truncate | QIODevice::WriteOnly))
     {
         qWarning() << "Couldn't open file";
@@ -280,7 +281,17 @@ void SettingsSerializer::save()
     }
 
     f.write(data);
-    f.close();
+
+    // check if everything got written
+    if(f.flush())
+    {
+        f.commit();
+    }
+    else
+    {
+        f.cancelWriting();
+        qCritical() << "Failed to write, can't save!";
+    }
 }
 
 void SettingsSerializer::readSerialized()