blumia/dde-application-manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: add checks before reading from value

Browse Source 
Signed-off-by: ComixHe <heyuming@deepin.org>
This commit is contained in:
ComixHe 2023-11-06 12:31:23 +08:00 committed by Comix
parent ff5f2062c3
commit e61c8ae17a
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/applicationmanagerstorage.cpp
View File

@ -165,16 +165,25 @@ QVariant ApplicationManager1Storage::readApplicationValue(const QString &appId,
const QString &groupName, const QString &groupName,
const QString &valueKey) const noexcept const QString &valueKey) const noexcept
{ {
if (!m_data.contains(appId)) {
return {};
}
auto app = m_data.constFind(appId)->toObject(); auto app = m_data.constFind(appId)->toObject();
if (app.isEmpty()) { if (app.isEmpty()) {
return {}; return {};
} }
if (!app.contains(groupName)) {
return {};
}
auto group = app.constFind(groupName)->toObject(); auto group = app.constFind(groupName)->toObject();
if (group.isEmpty()) { if (group.isEmpty()) {
return {}; return {};
} }
if (!group.contains(valueKey)) {
return {};
}
auto val = group.constFind(valueKey); auto val = group.constFind(valueKey);
if (val->isNull()) { if (val->isNull()) {
return {}; return {};