Review Request 125734: Fix crash when exiting kscreen kcm in systemsettings

[Martin Gräßlin]

> On Oct. 21, 2015, 11:19 a.m., Christoph Feck wrote:
> > Thanks Martin for the investigation!
> > 
> > The crash is not specific to the kscreen kcm, but we also got it with other kcms that use QML, so the same (or a similiar) fix has to be applied there, too.
> 
> Martin Gräßlin wrote:
>     all right, do you know which other KCMs? I explicitly tested decorations KCM which doesn't expose the problem.
> 
> Christoph Feck wrote:
>     From remembering some comments of duplicates, "changing GL version" and "changing desktop effects" was among them. But it's possible that they previously also visited the kscreen page, and the crash is indeed only there.
> 
> Martin Gräßlin wrote:
>     "changing GL version" clearly not: it's good old QWidget based ;-)
>     
>     Going twice to desktop effects does indeed trigger a crash, but a completely different one.

> Going twice to desktop effects does indeed trigger a crash

Reported a new bug report for that: https://bugs.kde.org/show_bug.cgi?id=354164 - in case you have seen that one before please duplicate. My google skills failed to find one ;-)


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125734/#review87183
-----------------------------------------------------------


On Oct. 21, 2015, 10:37 a.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125734/
> -----------------------------------------------------------
> 
> (Updated Oct. 21, 2015, 10:37 a.m.)
> 
> 
> Review request for Plasma, Kai Uwe Broulik and Daniel Vrátil.
> 
> 
> Bugs: 344651
>     http://bugs.kde.org/show_bug.cgi?id=344651
> 
> 
> Repository: kscreen
> 
> 
> Description
> -------
> 
> When using threaded rendering loop [1] systemsettings crashed when
> clicking the back button. This crash is caused inside the shader effect
> used by the OpacityMask.
> QtQuick informed us also that something is wrong:
> "QSGDefaultLayer::bind: ShaderEffectSource: 'recursive' must be set to true when rendering recursively."
> 
> The actual problem is that orientationPanelContainer and the OpacityMask
> are children to the monitor item causing recursion. By making them
> siblings to the orientationPanelContainer the problem is resolved. The
> error message from QtQuick is no longer shown and systemsettings doesn't
> crash any more.
> 
> BUG: 344651
> FIXED-IN: 5.4.3
> 
> [1] QSG_RENDER_LOOP=threaded systemsettings5
> 
> 
> Diffs
> -----
> 
>   kcm/qml/Output.qml 7bcaf39174a748a203eab5a420cb57c3639d0024 
> 
> Diff: https://git.reviewboard.kde.org/r/125734/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151021/8e1fda05/attachment-0001.html>