D15818: [Exe Thumbnailer] Ignore depth > 32

2018-09-28 Thread Kai Uwe Broulik
broulik created this revision.
broulik added a reviewer: anthonyfieroni.
Herald added projects: Dolphin, Frameworks.
Herald added subscribers: kfm-devel, kde-frameworks-devel.
broulik requested review of this revision.

REVISION SUMMARY
  The depth information is read unaltered from the `ico` file and might be 
bogus. `QImage::depth()` claims that 32 is the largest supported value, so cap 
here.

REPOSITORY
  R320 KIO Extras

REVISION DETAIL
  https://phabricator.kde.org/D15818

AFFECTED FILES
  thumbnail/icoutils_common.cpp

To: broulik, anthonyfieroni
Cc: kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, 
navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp


D15818: [Exe Thumbnailer] Ignore depth > 32

2018-09-28 Thread Anthony Fieroni
anthonyfieroni accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R320 KIO Extras

REVISION DETAIL
  https://phabricator.kde.org/D15818

To: broulik, anthonyfieroni
Cc: kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, 
navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp


D15818: [Exe Thumbnailer] Ignore depth > 32

2018-10-01 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes.
Closed by commit R320:5943efcfed8d: [Exe Thumbnailer] Ignore depth > 32 
(authored by broulik).

REPOSITORY
  R320 KIO Extras

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15818?vs=42492&id=42638

REVISION DETAIL
  https://phabricator.kde.org/D15818

AFFECTED FILES
  thumbnail/icoutils_common.cpp

To: broulik, anthonyfieroni
Cc: kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, 
navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp