D24367: Some sanity verification
Martin Tobias Holmedahl Sandsmark
noreply at phabricator.kde.org
Wed Oct 2 16:45:41 BST 2019
sandsmark created this revision.
sandsmark added reviewers: aacid, cfeck.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
sandsmark requested review of this revision.
REVISION SUMMARY
Since QImage does sanity checking for overflows and stuff wrt. dimensions and depth, check for QImage::isNull() as early as possible to see if there's some funky business going on.
Also tried to add some checks wherever we wrote to "raw" memory.
TEST PLAN
Unit tests pass, and tested converting some files from https://samples.ffmpeg.org/image-samples/ to pngs, and that seemed to work.
REPOSITORY
R287 KImageFormats
REVISION DETAIL
https://phabricator.kde.org/D24367
AFFECTED FILES
src/imageformats/exr.cpp
src/imageformats/pcx.cpp
src/imageformats/pic.cpp
src/imageformats/psd.cpp
src/imageformats/rgb.cpp
src/imageformats/tga.cpp
To: sandsmark, aacid, cfeck
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191002/46a7cae3/attachment.html>
More information about the Kde-frameworks-devel
mailing list