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