| Summary: | Plasma Taskbar Crash | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Armando <arodlinux> |
| Component: | Application Menu (Kicker) | Assignee: | Eike Hein <hein> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | bhush94, darkeyenick, kde, ngompa13, plasma-bugs |
| Priority: | NOR | Keywords: | drkonqi |
| Version: | 5.4.2 | ||
| Target Milestone: | 1.0 | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/plasma-desktop/3dbbddcc75e6e99c2967b3be460511c833021c71 | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
Armando
2015-10-31 23:27:54 UTC
From what I can tell:
Qt tries to run the focusOut called from
QMetaObject::invokeMethod(this,
"focusOut",
Qt::QueuedConnection,
Q_ARG(QQuickWindow*, qobject_cast<QQuickWindow *>(watched)));
but "watched" has been deleted by the QML engine in the meantime, so it can't go back into a QML context as an arg to the signal.
Obvious "fix" is simply to remove the arg as it's not being used by anyone. Eike, thoughts?
Not the biggest fan of that since from an API POV it means a client can no longer tell which window the focusOut is for. It's true that this information currently isn't used by Kicker, but it's really crummy API without it. I think I'd be more comfortable with making it synchronous instead. Curious how this wound up being called while moving a panel though. Git commit 3dbbddcc75e6e99c2967b3be460511c833021c71 by Eike Hein. Committed on 12/11/2015 at 10:41. Pushed by hein into branch 'master'. Avoid crash by making emit synchronous. M +1 -4 applets/kicker/plugin/windowsystem.cpp http://commits.kde.org/plasma-desktop/3dbbddcc75e6e99c2967b3be460511c833021c71 *** Bug 355751 has been marked as a duplicate of this bug. *** *** Bug 355750 has been marked as a duplicate of this bug. *** |