Review Request 126961: Fix the infamous Plasma::Applet::Actions crash

David Edmundson david at
Tue Feb 2 15:00:33 UTC 2016

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 2, 2016, 3 p.m.)

Review request for Plasma.

Bugs: 351777

Repository: plasma-workspace


We were mixing KScreen and QScreen API badly.

Corona.cpp checks we are requesting a containment for a valid screen
if (screen >= 0 && screen < numScreens()) {

This fails as numScreens() is Qt API based, whereas the signal we're
adding the output for is ShellCorona::addOutput so we have an effective race condition.

BUG: 351777

Diffs (updated)

  shell/shellcorona.cpp 762e503bf59fe648fb0f5b76a36229aa43c563e5 



Started Plasma on dual screen.

Ideally we need to do more testing before backporting, as that entire codebase is a disgrace.


David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the Plasma-devel mailing list