Fix for AVP-01-006 - inscure app config permissions

client/amnezia_application.cpp

@@ -1,6 +1,7 @@
 #include "amnezia_application.h"
 
 #include <QFontDatabase>
+#include <QStandardPaths>
 #include <QTimer>
 #include <QTranslator>
 
@@ -49,6 +50,24 @@
 #endif
 {
     setQuitOnLastWindowClosed(false);
+
+    // Fix config file permissions
+#ifdef Q_OS_LINUX
+    {
+        QSettings s(ORGANIZATION_NAME, APPLICATION_NAME);
+        s.setValue("permFixed", true);
+    }
+
+    QString configLoc1 = QStandardPaths::standardLocations(QStandardPaths::ConfigLocation).first() + "/"
+            + ORGANIZATION_NAME + "/" + APPLICATION_NAME + ".conf";
+    QFile::setPermissions(configLoc1, QFileDevice::ReadOwner | QFileDevice::WriteOwner);
+
+    QString configLoc2 = QStandardPaths::standardLocations(QStandardPaths::ConfigLocation).first() + "/"
+            + ORGANIZATION_NAME + "/" + APPLICATION_NAME + "/" + APPLICATION_NAME + ".conf";
+    QFile::setPermissions(configLoc2, QFileDevice::ReadOwner | QFileDevice::WriteOwner);
+
+#endif
+
     m_settings = std::shared_ptr<Settings>(new Settings);
     m_serverController = std::shared_ptr<ServerController>(new ServerController(m_settings, this));
     m_configurator = std::shared_ptr<VpnConfigurator>(new VpnConfigurator(m_settings, m_serverController, this));

client/secure_qsettings.cpp

@@ -162,7 +162,10 @@ QByteArray SecureQSettings::decryptText(const QByteArray& ba) const
 
 bool SecureQSettings::encryptionRequired() const
 {
-    // TODO: review on linux
+#ifdef Q_OS_LINUX
+    // QtKeyChain failing on Linux
+    return false;
+#endif
     return true;
 }