Bug 139225

Summary:	konsole crashes after zooming text to really large size
Product:	[Applications] konsole	Reporter:	Yuriy Kozlov <yuriy.kozlov>
Component:	general	Assignee:	Robert Knight <robertknight>
Status:	RESOLVED FIXED
Severity:	crash
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Backtrace Backtrace Patch to fix bug #139225, restricts maximum font size to that which will fit in the display. Better patch to fix the crash.

Description Yuriy Kozlov 2006-12-26 04:57:55 UTC

Version:            (using KDE KDE 3.5.5)
Installed from:    Ubuntu Packages
OS:                Linux

Originally reported here: https://launchpad.net/distros/ubuntu/+source/kdebase/+bug/53159

Enlarging the font in konsole so much that a letter is bigger than the window causes konsole to crash.

One shouldn't even be able to enlarge the font to such an unreasonable size.

Comment 1 Yuriy Kozlov 2006-12-26 04:58:33 UTC

Created attachment 19034 [details]
Backtrace

Comment 2 Yuriy Kozlov 2006-12-26 04:58:34 UTC

Created attachment 19035 [details]
Backtrace

Comment 3 Robert Knight 2006-12-26 18:45:40 UTC

Created attachment 19036 [details]
Patch to fix bug #139225, restricts maximum font size to that which will fit in the display.

Fix for bug #139225.  Ignores attempts to change the font to a size larger than
will fit in the display, adds an assertion to catch errors in this area
earlier.

Comment 4 Robert Knight 2006-12-26 19:00:46 UTC

Marking bug as fixed.

Comment 5 Robert Knight 2006-12-26 19:02:15 UTC

Should have mentioned in previous comment:  Fix committed with SVN revision 616760.

Comment 6 Yuriy Kozlov 2007-01-26 08:19:36 UTC

I can still reproduce the crash in KDE 3.5.6

Was the fix committed to 3.5?

Comment 7 Robert Knight 2007-01-26 10:02:57 UTC

Yes, confirmed here as well.  The patch didn't completely solve the problem it seems.

Comment 8 Robert Knight 2007-01-26 10:04:24 UTC

Created attachment 19420 [details]
Better patch to fix the crash.

Better patch to fix the bug.  Use in addition to the previous patch.

Comment 9 Robert Knight 2007-05-08 01:06:23 UTC

Right, should be fixed for good in KDE 3.5.7