Bug 1635465

Summary: [abrt] Make sure message_list_folder_changed() is called from the main/UI thread
Product: [Fedora] Fedora Reporter: krinkodot22
Component: evolutionAssignee: Milan Crha <mcrha>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 28CC: alexl, caillon+fedoraproject, lucilanga, mcrha, rhughes, rstrode, sandmann
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
URL: https://retrace.fedoraproject.org/faf/reports/bthash/7186fbc00eedafd686533d96a122fedd4ed33e63
Whiteboard: abrt_hash:90ab854aab57fd12f1301d55bd4d1b8d2c93a625;VARIANT_ID=workstation;
Fixed In Version: evolution-3.30.2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-10-03 15:04:00 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
File: backtrace
none
File: cgroup
none
File: core_backtrace
none
File: cpuinfo
none
File: dso_list
none
File: environ
none
File: exploitable
none
File: limits
none
File: maps
none
File: mountinfo
none
File: open_fds
none
File: proc_pid_status
none
File: var_log_messages none

Description krinkodot22 2018-10-03 02:06:40 UTC 
Description of problem:
-Sent an email with an attachment, from a MS Exchange account
-Waited a moment for it to get sent
-Opened my Sent Items folder to confirm that it was sent

As soon as the Sent Items folder was selected, no messages appeared in the main area, despite there being many sent items from before.
After waiting a few seconds, Evolution automatically closed itself.

After restarting Evolution, everything worked as intended: the Sent Items folder was populated, and included the email I had sent earlier.

Version-Release number of selected component:
evolution-3.28.5-1.fc28

Additional info:
reporter:       libreport-2.9.5
backtrace_rating: 3
cmdline:        evolution
executable:     /usr/bin/evolution
journald_cursor: s=7b59ff08fa4540e69cb175086488a1c7;i=28592;b=2a6a27090e6149fc9f60f0c226561b7d;m=7dbfc1bd;t=577482e623bad;x=d5b925ec1cb6424d
kernel:         4.18.9-200.fc28.x86_64
rootdir:        /
runlevel:       N 5
type:           CCpp
uid:            1000

Truncated backtrace:
Thread no. 1 (10 frames)
 #0 ??
 #1 gtk_css_node_ensure_style at gtkcssnode.c:1007
 #3 gtk_css_node_get_style at gtkcssnode.c:1033
 #4 gtk_style_context_lookup_style at gtkstylecontext.c:2671
 #5 gtk_style_context_get_padding at gtkstylecontext.c:2671
 #6 get_button_padding at /usr/src/debug/evolution-3.28.5-1.fc28.x86_64/src/e-util/e-table-header-utils.c:48
 #7 e_table_header_compute_height at /usr/src/debug/evolution-3.28.5-1.fc28.x86_64/src/e-util/e-table-header-utils.c:74
 #8 e_table_header_item_get_height at /usr/src/debug/evolution-3.28.5-1.fc28.x86_64/src/e-util/e-table-header-item.c:164
 #9 ethi_add_table_header at /usr/src/debug/evolution-3.28.5-1.fc28.x86_64/src/e-util/e-table-header-item.c:272
 #10 ethi_set_property at /usr/src/debug/evolution-3.28.5-1.fc28.x86_64/src/e-util/e-table-header-item.c:306
Comment 1 krinkodot22 2018-10-03 02:06:47 UTC 
Created attachment 1489768 [details]
File: backtrace
Comment 2 krinkodot22 2018-10-03 02:06:48 UTC 
Created attachment 1489769 [details]
File: cgroup
Comment 3 krinkodot22 2018-10-03 02:06:50 UTC 
Created attachment 1489770 [details]
File: core_backtrace
Comment 4 krinkodot22 2018-10-03 02:06:51 UTC 
Created attachment 1489771 [details]
File: cpuinfo
Comment 5 krinkodot22 2018-10-03 02:06:52 UTC 
Created attachment 1489772 [details]
File: dso_list
Comment 6 krinkodot22 2018-10-03 02:06:53 UTC 
Created attachment 1489773 [details]
File: environ
Comment 7 krinkodot22 2018-10-03 02:06:54 UTC 
Created attachment 1489774 [details]
File: exploitable
Comment 8 krinkodot22 2018-10-03 02:06:55 UTC 
Created attachment 1489775 [details]
File: limits
Comment 9 krinkodot22 2018-10-03 02:06:57 UTC 
Created attachment 1489776 [details]
File: maps
Comment 10 krinkodot22 2018-10-03 02:06:59 UTC 
Created attachment 1489777 [details]
File: mountinfo
Comment 11 krinkodot22 2018-10-03 02:07:00 UTC 
Created attachment 1489778 [details]
File: open_fds
Comment 12 krinkodot22 2018-10-03 02:07:01 UTC 
Created attachment 1489779 [details]
File: proc_pid_status
Comment 13 krinkodot22 2018-10-03 02:07:02 UTC 
Created attachment 1489780 [details]
File: var_log_messages
Comment 14 Milan Crha 2018-10-03 07:20:43 UTC 
Thanks for a bug report and detailed steps. I see from the backtrace that evolution does some UI (gtk+) calls in a dedicated thread, not in the UI thread, which can cause random misbehaviour. I'll investigate this further and let you know when I know more.
Comment 15 Milan Crha 2018-10-03 15:04:00 UTC 
I made some changes in the upstream code to ensure the message_list_folder_changed() is called from the main/UI thread, which will avoid the crash. This will be available in the next upstream release.

Created commit [1] in evo master (3.31.1+)
Created commit [2] in evo gnome-3-30 (3.30.2+)

[1] https://gitlab.gnome.org/GNOME/evolution/commit/cc222b59f6
[2] https://gitlab.gnome.org/GNOME/evolution/commit/aca99a14fe