Bug 207290

Summary:	plasma crashes with a 16 bit screen depth, Qt 4.6, and -graphicssystem raster
Product:	[Unmaintained] plasma4	Reporter:	Benjamin K. Stuhl <benjamin.stuhl>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	crash	CC:	asraniel
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	untested concept patch

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.