From d9330ad10da8615610454bebbff65d00bc58dd14 Mon Sep 17 00:00:00 2001 From: Carl Schwan Date: Sun, 21 Jan 2024 00:18:13 +0100 Subject: [PATCH] Fix crash in envmanager To delete an entry, we need to ensure the KConfigGroup and none of its parent config group is not const otherwise we it an assert in KConfig. --- envmanager/settings.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/envmanager/settings.cpp b/envmanager/settings.cpp index 0e4aecb2..1f1df755 100644 --- a/envmanager/settings.cpp +++ b/envmanager/settings.cpp @@ -136,8 +136,8 @@ void Settings::saveConfigSetting(const QString &fileName, const QString &group, // NOTE: this deletes the stored value from the config after loading const QString Settings::loadSavedConfigSetting(KSharedConfig::Ptr &config, const QString &fileName, const QString &group, const QString &key, bool write) { - const auto savedGroup = KConfigGroup{m_mobileConfig, SAVED_CONFIG_GROUP}; - const auto fileGroup = KConfigGroup{&savedGroup, fileName}; + auto savedGroup = KConfigGroup{m_mobileConfig, SAVED_CONFIG_GROUP}; + auto fileGroup = KConfigGroup{&savedGroup, fileName}; auto keyGroup = KConfigGroup{&fileGroup, group}; if (!keyGroup.hasKey(key)) {