Commit 12a14a78 authored by David Redondo's avatar David Redondo 🏎

[kcms/keys] Really ignore unknown action

Summary:
Otherwise the end() iterator is dereferenced resulting in a crash
BUG: 421016

Test Plan: Import a predefined scheme

Reviewers: ngraham, #plasma, mart

Reviewed By: ngraham, #plasma, mart

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D29579
parent fcb04768
......@@ -400,6 +400,7 @@ void ShortcutsModel::setShortcuts(const KConfigBase &config)
});
if (shortcut == component->shortcuts.end()) {
qCWarning(KCMKEYS) << "Ignoring unknown action" << key;
continue;
}
const auto shortcuts = QKeySequence::listFromString(shortcutsGroup.readEntry(key));
shortcut->activeShortcuts = QSet<QKeySequence>(shortcuts.cbegin(), shortcuts.cend());
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment