From 02f0db1496eb97940a8962042d79eba39715b796 Mon Sep 17 00:00:00 2001
From: Emmanuel Lepage Vallee <elv1313@gmail.com>
Date: Fri, 28 Apr 2017 22:51:05 -0400
Subject: [PATCH] localprofile: Check the result of `open()`
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Change-Id: If958f6aa7a89ca3878482b60be5c3905502cac3e
Reviewed-by: Nicolas Jäger <nicolas.jager@savoirfairelinux.com>
---
 src/localprofilecollection.cpp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/localprofilecollection.cpp b/src/localprofilecollection.cpp
index 1949e8a3..d0bd1b72 100644
--- a/src/localprofilecollection.cpp
+++ b/src/localprofilecollection.cpp
@@ -78,7 +78,12 @@ bool LocalProfileEditor::save(const Profile* pro)
 
     qDebug() << "Saving profile in:" << filename;
     QFile file {filename};
-    file.open(QIODevice::WriteOnly);
+
+    if (Q_UNLIKELY(!file.open(QIODevice::WriteOnly))) {
+        qWarning() << "Can't write to" << filename;
+        return false;
+    }
+
     file.write(result);
     file.close();
     return true;
-- 
GitLab