aacid created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. aacid requested review of this revision.
REVISION SUMMARY We were passing handledextralen instead of n to parseExtraField We could also do a n < parseExtraField check earlier and fail, but given that parseExtraField always returns true and it has good checks for not over-reading from the buffer it seemed better just to pass n and let it parse as much as possible REPOSITORY R243 KArchive BRANCH master REVISION DETAIL https://phabricator.kde.org/D20208 AFFECTED FILES src/kzip.cpp To: aacid Cc: kde-frameworks-devel, michaelh, ngraham, bruns