After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 793566 - Crash in geary_imap_engine_account_synchronizer_revoke_all
Crash in geary_imap_engine_account_synchronizer_revoke_all
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: engine
0.12.x
Other Linux
: Normal normal
: 0.13.0
Assigned To: Geary Maintainers
Geary Maintainers
: 713967 746506 (view as bug list)
Depends on: 793438
Blocks:
 
 
Reported: 2018-02-18 16:17 UTC by Michael Catanzaro
Modified: 2018-04-13 13:32 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Michael Catanzaro 2018-02-18 16:17:26 UTC
Crash in geary_imap_engine_account_synchronizer_revoke_all:

Truncated backtrace:
Thread no. 1 (10 frames)
 #0 gee_priority_queue_type1_node_remove at priorityqueue.c:3319
 #1 _gee_priority_queue_remove_type1_node at priorityqueue.c:2529
 #2 _gee_priority_queue_re_insert at priorityqueue.c:1900
 #3 _gee_priority_queue_decrease_key at priorityqueue.c:2051
 #4 _gee_priority_queue_delete at priorityqueue.c:1992
 #5 gee_priority_queue_iterator_real_remove at priorityqueue.c:3743
 #6 gee_priority_queue_real_remove at priorityqueue.c:1186
 #7 geary_imap_engine_account_synchronizer_revoke_all at /usr/src/debug/geary-0.12.0-1.fc27.x86_64/src/engine/imap-engine/imap-engine-account-synchronizer.vala:174
 #8 geary_imap_engine_account_synchronizer_on_folders_available_unavailable at /usr/src/debug/geary-0.12.0-1.fc27.x86_64/src/engine/imap-engine/imap-engine-account-synchronizer.vala:107
 #9 _geary_imap_engine_account_synchronizer_on_folders_available_unavailable_geary_account_folders_available_unavailable at /usr/src/debug/geary-0.12.0-1.fc27.x86_64/src/engine/imap-engine/imap-engine-account-synchronizer.vala:40

There is a better backtrace in https://bugzilla.redhat.com/show_bug.cgi?id=1546551.
Comment 1 Michael Catanzaro 2018-02-18 16:18:27 UTC
https://bugzilla.redhat.com/show_bug.cgi?id=1149919 was reported in 2014, so this has been around for a while.
Comment 2 Michael Catanzaro 2018-02-18 16:19:48 UTC
Original bug report says: "geary crashed when I removed an account while it was indexing it"

But I definitely did not remove any accounts when I hit this today.
Comment 3 Michael Gratton 2018-02-18 23:51:53 UTC
I just discovered this in Fedora bz last week, seems to be a genuine crash in libgee: Bug 793438

A workaround for this would be a somewhat intrusive change in the engine code, and an equivalent is already on master, but I'll look into it for 0.12.
Comment 4 Michael Gratton 2018-04-13 01:25:50 UTC
*** Bug 746506 has been marked as a duplicate of this bug. ***
Comment 5 Michael Gratton 2018-04-13 01:51:23 UTC
*** Bug 713967 has been marked as a duplicate of this bug. ***
Comment 6 Michael Gratton 2018-04-13 13:32:41 UTC
This is petty annoying, but it is crash-on-exit in one scenario, being low impact, and otherwise an uncommon crash when removing accounts, or when bulk mailbox removal is occurring. So aside from stack traces in disto infrastructure it's not a huge problem.

Since the problematic code has already gone away on master, and it seems like it will be a lot of trouble to backport the same work to geary-0.12, I'm going to call this fixed for 0.13.