Bug 1251123

Summary: [abrt] evolution: _g_log_abort(): evolution killed by SIGTRAP
Product: [Fedora] Fedora Reporter: Erik van Pienbroek <erik-fedora>
Component: evolutionAssignee: Milan Crha <mcrha>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 23CC: lucilanga, mbarnes, mcrha, tpopela
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
URL: https://retrace.fedoraproject.org/faf/reports/bthash/87e6c26af5cb2df2a201cb0f86fb2c1d5259b347
Whiteboard: abrt_hash:45d3c1bbe0c524ac725cf03346dc7c3271dc0e72
Fixed In Version: evolution-3.17.90 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-07 04:37:14 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: dso_list
none
File: environ
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
Proposed patch none

Description Erik van Pienbroek 2015-08-06 13:34:39 UTC 
Description of problem:
The segfault can be reproduced easily by composing a new HTML-based email (using an EWS account) and pressing the 'Send' button

Version-Release number of selected component:
evolution-3.17.4-2.fc23

Additional info:
reporter:       libreport-2.6.2
backtrace_rating: 4
cmdline:        evolution
crash_function: _g_log_abort
executable:     /usr/bin/evolution
global_pid:     8112
kernel:         4.1.0+
runlevel:       N 5
type:           CCpp
uid:            500

Truncated backtrace:
Thread no. 1 (10 frames)
 #0 _g_log_abort at gmessages.c:316
 #3 g_settings_set_property at gsettings.c:628
 #4 object_set_property at gobject.c:1415
 #5 g_object_new_internal at gobject.c:1808
 #6 g_object_new_valist at gobject.c:2033
 #8 g_settings_new at gsettings.c:1000
 #9 e_util_ref_settings at e-misc-utils.c:3059
 #10 e_util_prompt_user at e-misc-utils.c:3120
 #11 ask_confirm_for_unwanted_html_mail at em-composer-utils.c:132
 #12 composer_presend_check_unwanted_html at em-composer-utils.c:489
Comment 1 Erik van Pienbroek 2015-08-06 13:34:44 UTC 
Created attachment 1059943 [details]
File: backtrace
Comment 2 Erik van Pienbroek 2015-08-06 13:34:45 UTC 
Created attachment 1059944 [details]
File: cgroup
Comment 3 Erik van Pienbroek 2015-08-06 13:34:47 UTC 
Created attachment 1059945 [details]
File: core_backtrace
Comment 4 Erik van Pienbroek 2015-08-06 13:34:49 UTC 
Created attachment 1059946 [details]
File: dso_list
Comment 5 Erik van Pienbroek 2015-08-06 13:34:50 UTC 
Created attachment 1059947 [details]
File: environ
Comment 6 Erik van Pienbroek 2015-08-06 13:34:51 UTC 
Created attachment 1059948 [details]
File: limits
Comment 7 Erik van Pienbroek 2015-08-06 13:34:54 UTC 
Created attachment 1059949 [details]
File: maps
Comment 8 Erik van Pienbroek 2015-08-06 13:34:55 UTC 
Created attachment 1059950 [details]
File: mountinfo
Comment 9 Erik van Pienbroek 2015-08-06 13:34:57 UTC 
Created attachment 1059951 [details]
File: namespaces
Comment 10 Erik van Pienbroek 2015-08-06 13:34:58 UTC 
Created attachment 1059952 [details]
File: open_fds
Comment 11 Erik van Pienbroek 2015-08-06 13:34:59 UTC 
Created attachment 1059953 [details]
File: proc_pid_status
Comment 12 Erik van Pienbroek 2015-08-06 13:35:00 UTC 
Created attachment 1059954 [details]
File: var_log_messages
Comment 13 Erik van Pienbroek 2015-08-06 13:48:16 UTC 
Evolution itself shows this output when the crash occurs:

(evolution:8112): GLib-GIO-ERROR **: Settings schema 'prompt-on-unwanted-html' is not installed

The gsettings key where the evolution output references to does actually exist:

$ gsettings get org.gnome.evolution.mail prompt-on-unwanted-html
true
Comment 14 Erik van Pienbroek 2015-08-06 14:06:30 UTC 
I guess that the schema name and key are accidently swapped in the first part of https://git.gnome.org/browse/evolution/commit/mail/em-composer-utils.c?id=2b9ac3f621d1fe8fe49cb1d2d5217d10baefdcb3

The e_util_prompt_user function expects the schema name to be mentioned before the key name (as is also done for the rest of the mentioned commit)
Comment 15 Erik van Pienbroek 2015-08-06 14:43:10 UTC 
Created attachment 1059999 [details]
Proposed patch

Attached patch resolves the issue for me. It indeed was the swapped function arguments
Comment 16 Milan Crha 2015-08-07 04:37:14 UTC 
Thanks for a bug report and patch. That's a good catch.

Created commit 4658676 in evo master (3.17.90+) [1]

[1] https://git.gnome.org/browse/evolution/commit/?id=4658676