minor fix

diff --git a/device-agent/daemon/settingshandler.cpp b/device-agent/daemon/settingshandler.cpp
index b926b0b..71d5e7a 100644 (file)
--- a/device-agent/daemon/settingshandler.cpp
+++ b/device-agent/daemon/settingshandler.cpp
@@ -36,12 +36,16 @@ void SettingsHandler::accept(NetworkManager::Event& event)
         Json::Value root;
 
         if (reader.parse(event.getContent(), root)) {
+
+            bool modified = false;
+
             if (root.isMember(TIMEOUT_KEY)) {
                 int timeout = root[TIMEOUT_KEY].asInt();
 
                 if (timeout > 0) {
                     LOG_D(TAG, "SettingsHandler: keepalive timeout set to %d", timeout);
                     NetworkManager::Settings::instance().setKeepAliveTimeout(std::chrono::milliseconds(timeout));
+                    modified = true;
                 } else {
                     LOG_E(TAG, "SettingsHandler: wrong timeout value %d", timeout);
                 }
@@ -51,9 +55,13 @@ void SettingsHandler::accept(NetworkManager::Event& event)
                 bool lock = root[LOCK_KEY].asBool();
                 LOG_D(TAG, "SettingsHandler: lock set to %d", int(lock));
                 NetworkManager::Settings::instance().setLock(lock);
+                modified = true;
+            }
+
+            if (modified) {
+                NetworkManager::Settings::instance().save();
             }
 
-            NetworkManager::Settings::instance().save();
             event.confirm();
         }
     } catch (std::exception& e) {