Git-Mirrors/keepassxc
1
0
Fork 0
mirror of https://github.com/keepassxreboot/keepassxc.git synced 2025-02-03 10:00:04 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix potential crash in search if an entry doesn't have a group (#9633)

Browse Source
This commit is contained in:
Jonathan White 2023-07-19 16:11:59 -04:00 committed by GitHub
parent bb37cf3c32
commit 534b61e9e8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/core/EntrySearcher.cpp
View File

@ -148,7 +148,10 @@ bool EntrySearcher::searchEntryImpl(const Entry* entry)
auto attributes = QStringList(attributes_keys + entry->attributes()->values(attributes_keys));
auto attachments = QStringList(entry->attachments()->keys());
// Build a group hierarchy to allow searching for e.g. /group1/subgroup*
auto hierarchy = entry->group()->hierarchy().join('/').prepend("/");
QString hierarchy;
if (entry->group()) {
hierarchy = entry->group()->hierarchy().join('/').prepend("/");
}
// By default, empty term matches every entry.
// However when skipping protected fields, we will reject everything instead
@ -190,7 +193,7 @@ bool EntrySearcher::searchEntryImpl(const Entry* entry)
// Match against the full hierarchy if the word contains a '/' otherwise just the group name
if (term.word.contains('/')) {
found = term.regex.match(hierarchy).hasMatch();
} else {
} else if (entry->group()) {
found = term.regex.match(entry->group()->name()).hasMatch();
}
break;