Summary: | System settings crashed when playing with Display configuration in QQuickWindow::MouseGrabbItem | ||
---|---|---|---|
Product: | [Applications] systemsettings | Reporter: | Sebastian <sebo.poczta> |
Component: | kcm_kscreen | Assignee: | kscreen-bugs-null <kscreen-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | anthony.gallo.1994, bugseforuns, germano.massullo, jeremy.linton, kde, linuxmetel, nate, plasma-bugs, rafalkozlowski07 |
Priority: | NOR | Keywords: | drkonqi |
Version: | 5.18.4 | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=449981 | ||
Latest Commit: | https://commits.kde.org/kscreen/cfd95e6bff4bfeadf95a7a475693b30ddfe30431 | Version Fixed In: | 5.19.0 |
Description
Sebastian
2020-04-04 10:29:30 UTC
*** Bug 411794 has been marked as a duplicate of this bug. *** A comment I don't want to get lost: Oleg Solovyov 2020-03-13 15:26:58 UTC the problem appears when upgrading kscreen from 5.16.5 to 5.17.0 I've done further investigation and found the following: in kcm/package/contents/ui/Output.qml:231, when I remove maskSource from OpacityMask item, the problem vanishes (tag v5.17.0) Problem appears after commit 6309d33 *** Bug 416991 has been marked as a duplicate of this bug. *** *** Bug 417410 has been marked as a duplicate of this bug. *** *** Bug 412367 has been marked as a duplicate of this bug. *** The opacity mask here is weird, the mask source is it's own parent. That's unusual at least. *** Bug 420331 has been marked as a duplicate of this bug. *** I have reliable steps to reproduce the crash: open system settings make a change switch to another module click discard in the prompt Git commit cfd95e6bff4bfeadf95a7a475693b30ddfe30431 by David Edmundson. Committed on 02/05/2020 at 21:52. Pushed by davidedmundson into branch 'master'. fix(kcm): avoid recursive OpacityMask which leads to teardown crash Summary: In the current code OpacityMask's source is the parent which contains itself, which doesn't make sense. Docs explicitly say this. [1] This leads to a crash in teardown. A quick reshuffle fixes it. [1] https://doc.qt.io/qt-5/qml-qtgraphicaleffects-opacitymask.html#source-prop Test Plan: Found reproducible steps (see bug) No longer crashes Reviewers: romangg Reviewed By: romangg Subscribers: romangg, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D29340 M +15 -11 kcm/package/contents/ui/Output.qml https://commits.kde.org/kscreen/cfd95e6bff4bfeadf95a7a475693b30ddfe30431 *** Bug 425927 has been marked as a duplicate of this bug. *** |