Avoid in-place convertion of images with multiple references
authorBenjamin Poulain <benjamin.poulain@nokia.com>
Fri, 8 Oct 2010 12:21:10 +0000 (14:21 +0200)
committerJesper Thomschutz <jesper.thomschutz@nokia.com>
Sun, 10 Oct 2010 17:52:34 +0000 (19:52 +0200)
commit6f596b07a7c035b36bc1005a83b283057669e4ac
treea6b6902cae3d98e0ee9099741496fb7fa45a4bef
parentdc5299e1a59e066e4b804c081114edaa8b104438
Avoid in-place convertion of images with multiple references

The decoding from image reader was assuming the image reader do not
keep the image internally. This is not true for the GIF plugins because
the previous image can be used to compose the current image.

This was causing crash on ARM because the 16 bits color depth causes
the image memory to be reduce by half. When the plugin was accessing
the memory, it assumes the images has not changed and is on 32 bits.

This patch disable the in-place conversion if a detach is required.
Regular conversion is the correct solution in this case, and it can
also be made faster by converting while copying.

Reviewed-by: Andreas Kling
(cherry picked from commit 4d974ff0a748b22e668a4cb7ef38101122c85b3b)
src/gui/image/qimage.cpp
tests/auto/qpixmap/tst_qpixmap.cpp