|Summary:||plasmashell segmentation fault in KScreen::GetConfigOperation::start when logging into Plasma 5.18.2 on Wayland|
|Product:||[Plasma] KScreen||Reporter:||Matt Fagnani <matt.fagnani>|
|Severity:||crash||CC:||kde, plasma-bugs, rdieter, subdiff|
|Latest Commit:||Version Fixed In:|
Description Matt Fagnani 2020-02-27 03:13:40 UTC
Comment 1 Matt Fagnani 2020-02-27 04:53:42 UTC
I should've written that after the first such crash, the desktop icons were in four columns while they were normally in three columns. I've seen this crash 4/4 times when logging into Plasma 5.18.2 on Wayland. These plasmashell crashes might be related to the patch for the kded5 crashes when logging into Plasma 5.17.90 and later on Wayland at https://bugs.kde.org/show_bug.cgi?id=416566#c6 The kded5 crashing thread had functions with Config in their names in libkscreen-qt5 in frame #8 KScreen::WaylandConfig::applyConfig at /usr/src/debug/libkscreen-qt5-5.18.0-1.fc32.x86_64/backends/kwayland/waylandconfig.cpp:330 and #9 KScreen::SetConfigOperation::start() at /usr/src/debug/libkscreen-qt5-5.18.0-1.fc32.x86_64/src/setconfigoperation.cpp:129 https://bugs.kde.org/show_bug.cgi?id=416566#c2
Comment 2 Matt Fagnani 2020-02-27 20:55:03 UTC
A patch in GetConfigOperation::start at getconfigoperation.cpp:174-176 to fix a null pointer dereference where the segmentation fault occurred was at https://cgit.kde.org/libkscreen.git/commit/?id=069bb65af0d0059391323f9ffd2edd9162d57588 https://phabricator.kde.org/D27625 The patch was listed in the 5.18.2 Changelog for libkscreen "Fix: handle when backend fails to load/initialize. Commit. Phabricator Code review D27625" https://kde.org/announcements/plasma-5.18.1-5.18.2-changelog.php The if block checking for null backend added by the commit doesn't appear in /usr/src/debug/libkscreen-qt5-5.18.2-1.fc32.x86_64/src/getconfigoperation.cpp at line 174. That patch might need to be added to fix these crashes.
Comment 3 Matt Fagnani 2020-02-29 19:58:36 UTC
This crash was fixed in Fedora in libkscreen-qt5-5.18.2-2.fc32 https://koji.fedoraproject.org/koji/buildinfo?buildID=1472018 which added the commit https://cgit.kde.org/libkscreen.git/commit/?id=069bb65af0d0059391323f9ffd2edd9162d57588 That patch was added to the libkscreen 5.18 branch after the 5.18.2 tag was created https://cgit.kde.org/libkscreen.git/log/?h=Plasma/5.18 I'm reassigning this report to kscreen. Thanks.
Comment 4 Roman Gilg 2020-02-29 21:43:57 UTC
Closed as fixed as reported
Comment 5 David Edmundson 2020-02-29 21:58:28 UTC
>The patch was listed in the 5.18.2 Changelog Heh weird. According to git: git tag --contains ff98585 git branch --contains=ff98585 * Plasma/5.18 So the fact that it isn't in the release tarball isn't surprising, the changelog is broken.