https://bugs.kde.org/show_bug.cgi?id=395712

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/2693e288c57a66facbe6b9932
                   |                            |ce8628a53afb246
         Resolution|---                         |FIXED

--- Comment #2 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 2693e288c57a66facbe6b9932ce8628a53afb246 by David Edmundson.
Committed on 27/06/2018 at 00:36.
Pushed by davidedmundson into branch 'Plasma/5.13'.

Avoid potential assert in SM saving

Summary:
Sesison Manager stores all relevant clients. There's an assert if the
window type is outside of the standard client window types. It assumed
that all windows outside this would be Unmanaged windows rather than
Client objects, something probably true but not something enforced.

This particular crash was probably cased as we have a new window type in
Plasma OSD, which does not set BypassWindowManager in Qt window flags.

Test Plan:
Set to restore session
Logged out and back in
Saw some windows

Set to restore manually saved session
Hit save
No crash

Reviewers: #kwin, graesslin

Reviewed By: #kwin, graesslin

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D13715

M  +8    -0    sm.cpp

https://commits.kde.org/kwin/2693e288c57a66facbe6b9932ce8628a53afb246

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to