[panel] Fix crash on screen changes (!553) · Merge requests · Plasma / Plasma Workspace

David Edmundson requested to merge work/crash into master Jan 05, 2021

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.

[panel] Fix crash on screen changes