Bug 454539 - Widgets shouldn't be able to crash plasmashell
Summary: Widgets shouldn't be able to crash plasmashell
Status: RESOLVED INTENTIONAL
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (other bugs)
Version First Reported In: 5.24.5
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: usability
: 464405 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-05-28 16:57 UTC by vincentpamatat
Modified: 2023-05-07 13:26 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Plasmashell crashreport (4.34 KB, text/vnd.kde.kcrash-report)
2022-05-28 16:57 UTC, vincentpamatat
Details

Note You need to log in before you can comment on or make changes to this bug.
Description vincentpamatat 2022-05-28 16:57:41 UTC
Created attachment 149293 [details]
Plasmashell crashreport

STEPS TO REPRODUCE
1. Add a widget to the panel that only works under X11 (e.g https://github.com/wsdfhjxc/virtual-desktop-bar)
2. Log out and switch to wayland 

OBSERVED RESULT
Plasmashell crashes and continuously tries to restart itself.

EXPECTED RESULT
Plasmashell automatically disabling the crashing widget and/ or telling the user which widget crashes.

SOFTWARE/OS VERSIONS

Linux/KDE Plasma: Arch Linux
(available in About System)
KDE Plasma Version: 5.24.5
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.4
Comment 1 Nate Graham 2022-06-01 17:43:32 UTC
The root cause would be a misbehaving widget, but Plasmashell should definitely be more robust here.
Comment 2 Nate Graham 2023-04-27 18:38:55 UTC
*** Bug 464405 has been marked as a duplicate of this bug. ***
Comment 3 Nate Graham 2023-05-07 13:26:06 UTC
We discussed this at the Plasma 6 sprint and decided that it would not be worth the effort. Anything we could do that would accomplish this goal would hugely complicate the code and reduce robustness in a different way. So I'm afraid this is going to have to stay the way it is for the time being, sorry.