| Summary: | Plasma can crash in LegendModel::update() when displaying a notification that does *not* include a chart in it | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Nate Graham <nate> |
| Component: | Notifications | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | 4wy78uwh, bandhanpramanik06.foss, bugs.box, bugseforuns, ddrs, e.bachmakov, germano.massullo, ismailsahillioglu, jeremystartm, jrcornford, kalzwayed, kde, kdedev, mark, mds_foss, meven, nicolas.fella, oleg, percesvrzic, postix, pxq43a8b5, q9hc26ia, serdarthtux, slartibart70, stahlbergindustries, sw4 |
| Priority: | VHI | Keywords: | regression |
| Version First Reported In: | 6.3.90 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| See Also: | https://bugs.kde.org/show_bug.cgi?id=509192 | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/810b39163c8c835450d0a8a39d3bc768d3344c39 | Version Fixed In: | 6.4.5 |
| Sentry Crash Report: | https://crash-reports.kde.org/organizations/kde/issues/201624/?project=9&query=is%3Aunresolved%20issue.priority%3A%5Bhigh%2C%20medium%5D&stream_index=1 | ||
|
Description
Nate Graham
2025-04-21 11:01:17 UTC
*** Bug 505276 has been marked as a duplicate of this bug. *** *** Bug 504322 has been marked as a duplicate of this bug. *** Git commit fed717c4c52fc4b06b027130b39e9fc9d61a2760 by Arjen Hiemstra. Committed on 10/06/2025 at 15:02. Pushed by ahiemstra into branch 'master'. controls: Handle Chart::destroyed() in LegendModel * controls: Handle Chart::destroyed() in LegendModel Otherwise we risk trying to access objects that no longer exist. M +15 -2 controls/LegendModel.cpp M +1 -0 controls/LegendModel.h https://invent.kde.org/frameworks/kquickcharts/-/commit/fed717c4c52fc4b06b027130b39e9fc9d61a2760 We think that may fix the issue. Please test/verify with Frameworks 6.15 or later. Thanks folks! Unfortunately this crash doesn't seem to be fixed by that commit. On today's git master, I just got a crash with the same backtrace when I clicked on the Notifications icon in my system tray. [KCrash Handler] #4 0x00007fafa0144985 in QJSEngine::handle (this=0x0) at /usr/src/debug/qt6-qtdeclarative-6.9.1-1.fc42.x86_64/src/qml/jsapi/qjsengine.h:298 #5 QQmlListModel::engine (this=this@entry=0x1f25eaa0) at /usr/src/debug/qt6-qtdeclarative-6.9.1-1.fc42.x86_64/src/qmlmodels/qqmllistmodel.cpp:2128 #6 0x00007fafa0145c46 in QQmlListModel::data (this=0x1f25eaa0, index=<optimized out>, role=0) at /usr/src/debug/qt6-qtdeclarative-6.9.1-1.fc42.x86_64/src/qmlmodels/qqmllistmodel.cpp:2319 #7 0x00007fafa01461f4 in QQmlListModel::data (this=<optimized out>, index=..., role=<optimized out>) at /usr/include/qt6/QtCore/qabstractitemmodel.h:130 #8 0x00007faeed9b6643 in ModelSource::item (this=0x19845790, index=0) at /home/nate/kde/src/kquickcharts/src/datasource/ModelSource.cpp:93 #9 0x00007faeed9b251c in ChartDataSource::first (this=<optimized out>) at /home/nate/kde/src/kquickcharts/src/datasource/ChartDataSource.cpp:20 #10 0x00007faeed9638f7 in LegendModel::getValueForItem (this=this@entry=0x1e543530, item=item@entry=0) at /home/nate/kde/src/kquickcharts/controls/LegendModel.cpp:263 #11 0x00007faeed964620 in LegendModel::update (this=0x1e543530) at /home/nate/kde/src/kquickcharts/controls/LegendModel.cpp:164 *** Bug 505994 has been marked as a duplicate of this bug. *** *** Bug 505966 has been marked as a duplicate of this bug. *** *** Bug 506006 has been marked as a duplicate of this bug. *** *** Bug 506922 has been marked as a duplicate of this bug. *** *** Bug 507204 has been marked as a duplicate of this bug. *** *** Bug 507536 has been marked as a duplicate of this bug. *** *** Bug 507783 has been marked as a duplicate of this bug. *** *** Bug 506843 has been marked as a duplicate of this bug. *** *** Bug 507861 has been marked as a duplicate of this bug. *** A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5737 Git commit 8f81580780253f98210731c8962f33c67de08e98 by Arjen Hiemstra. Committed on 07/08/2025 at 07:55. Pushed by ahiemstra into branch 'master'. applets/notifications: Use a proper component for the speed chart Children of a Loader are not part of the loader's contents, they're just children of the loader. To make sure we actually have the chart as Loader contents, wrap it in an item and set it as sourceComponent. This ensures we don't try to load the chart when it's not actually visible or needed. M +99 -95 applets/notifications/components/SpeedChart.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/8f81580780253f98210731c8962f33c67de08e98 A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5740 Git commit 810b39163c8c835450d0a8a39d3bc768d3344c39 by Arjen Hiemstra. Committed on 07/08/2025 at 09:03. Pushed by ahiemstra into branch 'Plasma/6.4'. applets/notifications: Use a proper component for the speed chart Children of a Loader are not part of the loader's contents, they're just children of the loader. To make sure we actually have the chart as Loader contents, wrap it in an item and set it as sourceComponent. This ensures we don't try to load the chart when it's not actually visible or needed. (cherry picked from commit 8f81580780253f98210731c8962f33c67de08e98) Co-authored-by: Arjen Hiemstra <ahiemstra@heimr.nl> M +99 -95 applets/notifications/package/contents/ui/components/SpeedChart.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/810b39163c8c835450d0a8a39d3bc768d3344c39 *** Bug 507973 has been marked as a duplicate of this bug. *** That should fix it. We'll re-open if we get any new reports from people using Plasma 6.4.5 or later. *** Bug 508100 has been marked as a duplicate of this bug. *** *** Bug 508051 has been marked as a duplicate of this bug. *** *** Bug 508649 has been marked as a duplicate of this bug. *** *** Bug 507805 has been marked as a duplicate of this bug. *** *** Bug 508931 has been marked as a duplicate of this bug. *** *** Bug 509246 has been marked as a duplicate of this bug. *** *** Bug 508783 has been marked as a duplicate of this bug. *** |