| Summary: | plasmashell hits ASSERT on startup after upgrade to 6.0.0 | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Kevin Krammer <krammer> |
| Component: | Startup process | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | faure, kde, kde, nate |
| Priority: | NOR | Keywords: | qt6 |
| Version First Reported In: | 6.0.0 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/20e6c20de00e03ebb1c4cf61246f7fbcd1e3fbbc | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: | Backtrace with debug symbol packages installed | ||
|
Description
Kevin Krammer
2024-03-08 16:05:46 UTC
Created attachment 166723 [details]
Backtrace with debug symbol packages installed
Sounds like a trivial fix but I’d like to understand how the scheme can be empty in the first place. Any chance you could look at what the url is it tries to check? Looks like a broken desktop file you have there? A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2100 (In reply to Kai Uwe Broulik from comment #2) > Sounds like a trivial fix but I’d like to understand how the scheme can be > empty in the first place. Any chance you could look at what the url is it > tries to check? Looks like a broken desktop file you have there? Apparent the URL is "://$HOME" Couldn't quite find the .desktop file this would come from. I grepped through my whole system (as root to be sure) and only found /home/kevin/.kde/share/apps/RecentDocuments/WeeklyTimeSheet-2018-06.charmreport.desktop:URL[$e]=file://$HOME/data/Charm/WeeklyTimeSheet-2018-06.charmreport /home/kevin/.kde/share/apps/RecentDocuments/IMG_20160410_155042.jpg.desktop:URL[$e]=file://$HOME/data/downloads/bluetooth/IMG_20160410_155042.jpg but those are file://$HOME Hmm, [$e] should expand environment variables. I know we dumbed down the evironment expansion for security reasons when used in desktop files which could have broken this here? @dfaure you have any idea? Either way, let's not assert in Folder Model about broken URLs. No idea, given that a KDesktopFile unittest passes. https://invent.kde.org/frameworks/kconfig/-/merge_requests/282 But the question is what foldermodel does. Git commit 0504f0e0ef2b0ebc4f1b1faa9ee8c74d2191931b by Kai Uwe Broulik. Committed on 09/03/2024 at 12:23. Pushed by broulik into branch 'master'. Folder Model: Handle invalid URL in desktop file There's no point in handling an invalid URL, and KProtocolInfo asserts that the scheme is not empty. M +4 -1 containments/desktop/plugins/folder/foldermodel.cpp https://invent.kde.org/plasma/plasma-desktop/-/commit/0504f0e0ef2b0ebc4f1b1faa9ee8c74d2191931b Git commit 0264726f8720d3093bd3ba10f6107197b4f90be3 by Kai Uwe Broulik. Committed on 09/03/2024 at 12:33. Pushed by broulik into branch 'Plasma/6.0'. Folder Model: Handle invalid URL in desktop file There's no point in handling an invalid URL, and KProtocolInfo asserts that the scheme is not empty. (cherry picked from commit 0504f0e0ef2b0ebc4f1b1faa9ee8c74d2191931b) M +4 -1 containments/desktop/plugins/folder/foldermodel.cpp https://invent.kde.org/plasma/plasma-desktop/-/commit/0264726f8720d3093bd3ba10f6107197b4f90be3 Git commit 20e6c20de00e03ebb1c4cf61246f7fbcd1e3fbbc by Kai Uwe Broulik. Committed on 17/07/2024 at 20:48. Pushed by broulik into branch 'Plasma/5.27'. Folder Model: Handle invalid URL in desktop file There's no point in handling an invalid URL, and KProtocolInfo asserts that the scheme is not empty. (cherry picked from commit 0504f0e0ef2b0ebc4f1b1faa9ee8c74d2191931b) (cherry picked from commit 0264726f8720d3093bd3ba10f6107197b4f90be3) M +4 -1 containments/desktop/plugins/folder/foldermodel.cpp https://invent.kde.org/plasma/plasma-desktop/-/commit/20e6c20de00e03ebb1c4cf61246f7fbcd1e3fbbc |