Summary: | System Settings crash on deleting cursor themes | ||
---|---|---|---|
Product: | [Applications] systemsettings | Reporter: | Shaarav <shaaravgunsi> |
Component: | kcm_cursortheme | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | kde, nate, sg7e0f7m |
Priority: | NOR | Keywords: | drkonqi |
Version: | 5.22.1 | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/commit/59a352e19041eec9c0a521e4aaea1eaf95b30873 | Version Fixed In: | 5.23 |
Sentry Crash Report: | |||
Attachments: | New crash information added by DrKonqi |
Description
Shaarav
2021-06-22 14:32:41 UTC
Created attachment 140241 [details]
New crash information added by DrKonqi
systemsettings5 (5.22.3) using Qt 5.15.3
- What I was doing when the application crashed:
I deleted some cursor themes and when I hit apply system settings crashes.
-- Backtrace (Reduced):
#4 QString::QString (other=..., this=0x7fff1ef28f50) at /usr/include/x86_64-linux-gnu/qt5/QtCore/qstring.h:1093
#5 CursorTheme::path (this=0x0) at ./kcms/cursortheme/xcursor/cursortheme.h:84
#6 CursorThemeConfig::removeThemes (this=this@entry=0x55c106679880) at ./kcms/cursortheme/kcmcursortheme.cpp:511
#7 0x00007fd6916164bc in CursorThemeConfig::save (this=0x55c106679880) at ./kcms/cursortheme/kcmcursortheme.cpp:328
[...]
#9 0x00007fd6c9db527f in KCModuleProxy::save() () from /lib/x86_64-linux-gnu/libKF5KCMUtils.so.5
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/998 Git commit 59a352e19041eec9c0a521e4aaea1eaf95b30873 by David Edmundson. Committed on 15/09/2021 at 23:01. Pushed by davidedmundson into branch 'master'. Fix potential crash removing cursorthemes m_themeModel->removeTheme(idx) will adjust the model. Therefore any model indexes we had stored will be invalidated. If we try and remove 2 cursor themes this can end up either crashing or even worse potentially removing the wrong theme. M +6 -5 kcms/cursortheme/kcmcursortheme.cpp https://invent.kde.org/plasma/plasma-workspace/commit/59a352e19041eec9c0a521e4aaea1eaf95b30873 |