dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land.
If the format is really such that a term must appear before any Suffix, then this patch is already better that before. Could it happen to have e.g.: a\0b1c1 If so, this code would extend the Suffix b with Suffix c. Would that be correct? Or can that never happen? Or should c be a Suffix for a? If so, this code should be improved. REPOSITORY R293 Baloo BRANCH b392878_avoid_docterms_decode_crash REVISION DETAIL https://phabricator.kde.org/D12047 To: bruns, #baloo, michaelh, ngraham, #frameworks, dhaumann Cc: dhaumann, kde-frameworks-devel, #frameworks, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns