From 33cc5a2017b0bea38b43af6d0d60fe014658d8a9 Mon Sep 17 00:00:00 2001 From: Camila Ayres Date: Mon, 28 Oct 2024 20:07:28 +0100 Subject: [PATCH] Validate http_user value before assigning it to user in the config file. Signed-off-by: Camila Ayres --- src/gui/accountmanager.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gui/accountmanager.cpp b/src/gui/accountmanager.cpp index 57dfd097bb..8cb16779c9 100644 --- a/src/gui/accountmanager.cpp +++ b/src/gui/accountmanager.cpp @@ -393,12 +393,16 @@ void AccountManager::saveAccountHelper(Account *acc, QSettings &settings, bool s const auto settingsMapKeys = acc->_settingsMap.keys(); for (const auto &key : settingsMapKeys) { + if (!acc->_settingsMap.value(key).isValid()) { + continue; + } + settings.setValue(key, acc->_settingsMap.value(key)); } settings.setValue(QLatin1String(authTypeC), acc->_credentials->authType()); // HACK: Save http_user also as user - if (acc->_settingsMap.contains(httpUserC)) { + if (acc->_settingsMap.contains(httpUserC) && acc->_settingsMap.value(httpUserC).isValid()) { settings.setValue(userC, acc->_settingsMap.value(httpUserC)); } }