Skip to content

[panel] Fix crash on screen changes

David Edmundson requested to merge work/crash into master

There is an error handling path when we fetch the relevant config()

KConfigGroup PanelView::panelConfig(...
{
    if (!containment || !screen) {
        return KConfigGroup();
    }

which we indiscrimiately call parent() on.

This patch guards that case, which is presumably screen being temporarily null.

This code is also Plasma 5.8 compatibility fallback, so arguably we could get rid of it.

BUG: 425711

Merge request reports

Loading