meven accepted this revision. meven added a comment. This revision is now accepted and ready to land.
Seems hard to miss cases like for `// +1 for UDS_DISPLAY_NAME` but I will trust you did enough review to be confident for all those cases. A missing one will just cause an allocation anyway which is what this is fixing in the first place, so no arm done basically. REPOSITORY R320 KIO Extras BRANCH usemoreudsentryreserve REVISION DETAIL https://phabricator.kde.org/D28538 To: kossebau, #frameworks, sitter, meven Cc: kde-frameworks-devel, kfm-devel, nikolaik, pberestov, iasensio, fprice, LeGast00n, cblack, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov