Summary: | KDED crashes when KScreen::Config::current() returns null | ||
---|---|---|---|
Product: | [Plasma] KScreen | Reporter: | Achim Bohnet <ach> |
Component: | kded | Assignee: | Alex Fiestas <afiestas> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | dvratil |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kscreen/c0935935046cd9fe8b10acb88bbddbf70767c8c7 | Version Fixed In: | |
Attachments: |
xrandr log
kscreen console log |
Description
Achim Bohnet
2013-01-07 18:32:53 UTC
Created attachment 76286 [details]
xrandr log
Created attachment 76287 [details]
kscreen console log
@#$^^. My fault. My pkgs missed to install the KSC_* plugins of libkscreen. Now kcm plugin starts and kded4 no longer crashes. It's maybe a good idea to check if the plugin could be loaded/is loaded sucessfully, but on a properly installed system, the crash would not happen. Sorry, for the false alarm. Achim Reopening. Crashing when no module is install/fails to load is a bug. We should handle it and exit gracefully with an error message. Git commit c0935935046cd9fe8b10acb88bbddbf70767c8c7 by Dan Vrátil. Committed on 07/01/2013 at 22:21. Pushed by dvratil into branch 'master'. Be more careful about null pointers Check for null pointers before dereferencing (mostly in KDED). M +4 -0 kcm/src/displayconfiguration.cpp M +7 -0 kded/daemon.cpp M +17 -0 kded/generator.cpp M +14 -1 kded/serializer.cpp http://commits.kde.org/kscreen/c0935935046cd9fe8b10acb88bbddbf70767c8c7 |