Bug 207290 - plasma crashes with a 16 bit screen depth, Qt 4.6, and -graphicssystem raster
Summary: plasma crashes with a 16 bit screen depth, Qt 4.6, and -graphicssystem raster
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR crash
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-13 22:26 UTC by Benjamin K. Stuhl
Modified: 2009-11-08 12:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
untested concept patch (1.36 KB, patch)
2009-09-14 01:27 UTC, Benjamin K. Stuhl 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin K. Stuhl 2009-09-13 22:26:38 UTC 
Version:            (using KDE 4.3.1)
Compiler:          gcc 4.4.1 
OS:                Linux
Installed from:    Fedora RPMs

In 4.6, Qt now supports 16-bit depth QImages, and uses them for QPixmap::toImage() if you're using the Raster graphics system and a 16-bit screen depth. However, plasma's expblur() assumes a 32 bit depth QImage (without checking for it) and so causes memory corruption when it is used with a 16-bit depth screen.
Comment 1 Benjamin K. Stuhl 2009-09-14 01:27:53 UTC 
Created attachment 36930 [details]
untested concept patch

Here's an (untested) concept patch. (Untested because my Qt 4.6 install is sufficiently broken that I can't get KDE to configure on that system ATM...)
Comment 2 Beat Wolf 2009-09-14 10:05:13 UTC 
could you submit it to reviewboard.kde.org? thank you
Comment 3 Benjamin K. Stuhl 2009-09-14 17:01:30 UTC 
Submitted as http://reviewboard.kde.org/r/1608/ . I hope I filled it out correctly, I've never used reviewboard before.