Bug 265506 - Having NET_WM::Override in window type rule overrides is plain wrong!
Summary: Having NET_WM::Override in window type rule overrides is plain wrong!
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-05 12:36 UTC by Martin Koller
Modified: 2012-03-18 14:40 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Koller 2011-02-05 12:36:22 UTC
Application: kwin (4.6.00 (4.6.0))
KDE Platform Version: 4.6.00 (4.6.0)
Qt Version: 4.7.1
Operating System: Linux 2.6.34.7-0.7-desktop i686
Distribution: "openSUSE 11.3 (i586)"

-- Information about the crash:
- What I was doing when the application crashed:

I started gimp and wanted to see how I can have the gimp-toolbox also shown in the taskbar. I defined a special kwin window rule and set the window type to Override

In kwinrulesrc I find the following for gimp:

[1]
Description=Window settings for gimp-2.6
clientmachine=eiwrk039.etm.at
clientmachinematch=0
title=Toolbox
titlematch=0
type=6
typerule=2
types=256
windowrole=gimp-toolbox
windowrolematch=1
wmclass=gimp-2.6
wmclasscomplete=false
wmclassmatch=1

Now when I start gimp again, kwin always crashes multiple times until the restart is stopped and I'm asked to select a different window manager.

The crash can be reproduced every time.

-- Backtrace:
Application: KWin (kwin), signal: Aborted
[KCrash Handler]
#7  0xffffe424 in __kernel_vsyscall ()
#8  0xb744f7ff in raise () from /lib/libc.so.6
#9  0xb7451140 in abort () from /lib/libc.so.6
#10 0xb75c8b75 in KWin::Client::detectNoBorder (this=0x8468d58) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/client.cpp:663
#11 0xb76146d9 in KWin::Client::manage (this=0x8468d58, w=88080758, isMapped=true) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/manage.cpp:127
#12 0xb75bcc8b in KWin::Workspace::createClient (this=0x81688f8, w=88080758, is_mapped=true) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/workspace.cpp:564
#13 0xb75c195c in KWin::Workspace::init (this=0x81688f8) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/workspace.cpp:435
#14 0xb75c2ad5 in KWin::Workspace::Workspace (this=0x81688f8, restore=false) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/workspace.cpp:243
#15 0xb75dda04 in KWin::Application::Application (this=0xbf947014) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/main.cpp:330
#16 0xb75df7dc in kdemain (argc=3, argv=0xbf947284) at /usr/src/debug/kdebase-workspace-4.6.0/kwin/main.cpp:516
#17 0x0804860b in main (argc=3, argv=0xbf947284) at /usr/src/debug/kdebase-workspace-4.6.0/build/kwin/kwin_dummy.cpp:3

Reported using DrKonqi
Comment 1 Thomas Lübking 2011-02-05 15:47:30 UTC
Do not set the window type to override (6) does not mark a managed window and should only be used/set by clients which wish to completely bypass the windowmanager.

-> The option should simply not exists in the config dialog (valid bug, you cannot possibly know that this is evil)

- If you wish to have no border, use the "No border" setting.
- If you wish to have a taskbar entry, set the window type to "dialog" or "normal" - this will also slightly impact the way the window is handled by the WM (eg. utility windows by default(?) hide for inactive main windows)

To sum up: DO NOT USE THE OVERRIDE TYPE. Never.
Comment 2 Thomas Lübking 2012-03-18 14:40:11 UTC
fixed that ;-)