| Summary: | Contact list crashed after unlocking screen | ||
|---|---|---|---|
| Product: | [Unmaintained] telepathy | Reporter: | Elias Probst <mail> |
| Component: | contactlist | Assignee: | Telepathy Bugs <kde-telepathy-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | crash | CC: | ddomenichelli, kde, kde, mail, mklapetek, wengxt |
| Priority: | NOR | ||
| Version: | 0.4.0 | ||
| Target Milestone: | Future | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/telepathy-contact-list/acc91df99a169e8528dc1ecabcc8239d3f30880b | Version Fixed In: | 0.6.2 |
| Sentry Crash Report: | |||
| Attachments: | New crash information added by DrKonqi | ||
|
Description
Elias Probst
2012-07-26 11:49:54 UTC
Thanks, great report. My theory is that the custom auto-away presence message is inserted as the 'temp presence' to the combobox, then when it's removed before you're switched to available/the-presence-before, it automatically moves down in the list, which is now playing. Then something strange happens and it goes down. Hopefully I'll be able to fix it before 0.5. Git commit 67d235c46e492db89de25fc185201128d5be7bdc by Martin Klapetek. Committed on 31/07/2012 at 15:22. Pushed by mklapetek into branch 'master'. Set the new presence first and only then remove the temp presence Removing the temp presence first causes setting new selected index (as the currently selected one was just deleted) which is set to the next in the list, which in case of the temp presence is the 'Now playing...' one, which prompts the user for enabling it if it is disabled. So now we first change the presence (and current index) and only then remove the temp presence, then it won't cause another unwanted index re-setting. Reviewed-by: David Edmundson FIXED-IN: 0.5 M +3 -2 global-presence-chooser.cpp http://commits.kde.org/telepathy-contact-list/67d235c46e492db89de25fc185201128d5be7bdc Created attachment 74873 [details]
New crash information added by DrKonqi
ktp-contactlist (0.5.1) on KDE Platform 4.9.2 using Qt 4.8.3
- What I was doing when the application crashed:
This happened right now again…
I restarted NetworkManager and once NetworkManager was down (it wasn't up again yet) contactlist showed the dialog:
This plugin is currently disabled. Do you want to enable it and use as your presence?
Once I responded with "No" contactlist crashed with the attached backtrace.
-- Backtrace (Reduced):
#6 0x000000000043f531 in parent (this=0x6978b0) at /usr/include/qt4/QtCore/qabstractitemmodel.h:393
#7 PresenceModelExtended::data (this=0x881680, index=..., role=0) at /var/tmp/portage/net-im/ktp-contact-list-9999/work/ktp-contact-list-9999/global-presence-chooser.cpp:95
#8 0x00007f2bb0dd5c40 in QComboBoxPrivate::itemText (this=0x8208b0, index=...) at widgets/qcombobox.cpp:1287
#9 0x00007f2bb0dd9e3a in QComboBoxPrivate::_q_emitCurrentIndexChanged (this=<optimized out>, index=...) at widgets/qcombobox.cpp:1279
#10 0x00007f2bb0dda10a in QComboBoxPrivate::setCurrentIndex (this=0x8208b0, mi=<optimized out>) at widgets/qcombobox.cpp:2046
I can reproduce this crash every time now. I have the 'Now playing…' presence disabled in KTP settings. *** Bug 306743 has been marked as a duplicate of this bug. *** Elias, can you still reproduce this issue? And if so, are you using custom presence message for autoaway? Git commit 3315e817fb1ce8fd802b9ff7e9aa665d352573ef by David Edmundson. Committed on 15/01/2013 at 18:00. Pushed by davidedmundson into branch 'master'. Merge big model refactor. Replace all the models used to power KTp Subclass of Tp::Contact (KTp::Contact) to power the model Create a list of contacts ContactListModel Which can be grouped into AccountsTreeProxyModel or GroupsTreeProxyModel Related: bug 307899, bug 282218, bug 279023 REVIEW: 107377 REVIEW: 107629 DIGEST: M +5 -0 KTp/CMakeLists.txt http://commits.kde.org/telepathy-common-internals/3315e817fb1ce8fd802b9ff7e9aa665d352573ef I don't think this bug was fixed changing models. This is with the presence chooser. Is this bug still valid? Version Fixed in reports "0.5" Git commit acc91df99a169e8528dc1ecabcc8239d3f30880b by David Edmundson. Committed on 29/04/2013 at 03:54. Pushed by davidedmundson into branch 'kde-telepathy-0.6'. Remove pointless call to model index parent to count top level items REVIEW: 110236 M +3 -3 global-presence-chooser.cpp http://commits.kde.org/telepathy-contact-list/acc91df99a169e8528dc1ecabcc8239d3f30880b *** Bug 319525 has been marked as a duplicate of this bug. *** |