Bug 1321119

Summary: [abrt] Add thread safety around ECalBackendEws::priv::cancellable
Product: [Fedora] Fedora Reporter: Kenneth <kennethohman1>
Component: evolution-ewsAssignee: Milan Crha <mcrha>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: mbarnes, mcrha
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
URL: https://retrace.fedoraproject.org/faf/reports/bthash/a5ce02ff40efc5b603f7b68713a8850ed9a06f5e
Whiteboard: abrt_hash:eb09088a143ea78e50cd8de630e27ab79b7fb726;
Fixed In Version: evolution-ews-3.20.1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-03-30 12:24:09 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
File: backtrace
none
File: cgroup
none
File: core_backtrace
none
File: dso_list
none
File: environ
none
File: exploitable
none
File: limits
none
File: maps
none
File: mountinfo
none
File: namespaces
none
File: open_fds
none
File: proc_pid_status
none
File: var_log_messages none

Description Kenneth 2016-03-24 17:27:40 UTC 
Version-Release number of selected component:
evolution-data-server-3.18.5-1.fc23

Additional info:
reporter:       libreport-2.6.4
backtrace_rating: 4
cmdline:        /usr/libexec/evolution-calendar-factory-subprocess --factory ews --bus-name org.gnome.evolution.dataserver.Subprocess.Backend.Calendarx4627x5 --own-path /org/gnome/evolution/dataserver/Subprocess/Backend/Calendar/4627/5
crash_function: g_type_check_instance_is_fundamentally_a
executable:     /usr/libexec/evolution-calendar-factory-subprocess
global_pid:     4779
kernel:         4.4.5-300.fc23.x86_64
runlevel:       N 5
type:           CCpp
uid:            1000

Truncated backtrace:
Thread no. 1 (7 frames)
 #0 g_type_check_instance_is_fundamentally_a at gtype.c:4032
 #1 g_object_ref at gobject.c:3045
 #2 e_ews_connection_queue_request at e-ews-connection.c:758
 #3 e_ews_connection_sync_folder_items at e-ews-connection.c:3787
 #4 e_ews_connection_sync_folder_items_sync at e-ews-connection.c:3852
 #5 ews_start_sync_thread at e-cal-backend-ews.c:3718
 #6 g_thread_proxy at gthread.c:778
Comment 1 Kenneth 2016-03-24 17:27:45 UTC 
Created attachment 1140063 [details]
File: backtrace
Comment 2 Kenneth 2016-03-24 17:27:46 UTC 
Created attachment 1140064 [details]
File: cgroup
Comment 3 Kenneth 2016-03-24 17:27:48 UTC 
Created attachment 1140065 [details]
File: core_backtrace
Comment 4 Kenneth 2016-03-24 17:27:50 UTC 
Created attachment 1140066 [details]
File: dso_list
Comment 5 Kenneth 2016-03-24 17:27:51 UTC 
Created attachment 1140067 [details]
File: environ
Comment 6 Kenneth 2016-03-24 17:27:53 UTC 
Created attachment 1140068 [details]
File: exploitable
Comment 7 Kenneth 2016-03-24 17:27:54 UTC 
Created attachment 1140069 [details]
File: limits
Comment 8 Kenneth 2016-03-24 17:27:56 UTC 
Created attachment 1140070 [details]
File: maps
Comment 9 Kenneth 2016-03-24 17:27:57 UTC 
Created attachment 1140071 [details]
File: mountinfo
Comment 10 Kenneth 2016-03-24 17:27:59 UTC 
Created attachment 1140072 [details]
File: namespaces
Comment 11 Kenneth 2016-03-24 17:28:00 UTC 
Created attachment 1140073 [details]
File: open_fds
Comment 12 Kenneth 2016-03-24 17:28:02 UTC 
Created attachment 1140074 [details]
File: proc_pid_status
Comment 13 Kenneth 2016-03-24 17:28:03 UTC 
Created attachment 1140075 [details]
File: var_log_messages
Comment 14 Milan Crha 2016-03-30 12:24:09 UTC 
Thanks for a bug report. It looks like the ECalBackendEws::priv::cancellable needs a thread safety guard, which I added upstream with the below change. It will be available in the Fedora 24 once the release will be done.

Created commit 7d902cb in ews master (3.21.1+) [1]
Created commit dc44e46 in ews gnome-3-20 (3.20.1+)

[1] https://git.gnome.org/browse/evolution-ews/commit/?id=7d902cb