Bug 680153 – [abrt] Crash under message_info_from_db()

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 680153 - [abrt] Crash under message_info_from_db()


Summary:	[abrt] Crash under message_info_from_db()


Status:	RESOLVED FIXED

Product:	evolution-ews
Classification:	Other
Component:	Mail
Version:	3.4.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	Evolution EWS maintainer(s)
QA Contact:	Evolution EWS maintainer(s)

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2012-07-18 10:50 UTC by Milan Crha
Modified:	2012-07-18 17:04 UTC

See Also:
GNOME target:	---
GNOME version:	---

Attachments
ews patch (848 bytes, patch) 2012-07-18 16:58 UTC, Milan Crha	committed	Details \| Review
ews patch ][ (890 bytes, patch) 2012-07-18 17:02 UTC, Milan Crha	committed	Details \| Review

Description Milan Crha 2012-07-18 10:50:18 UTC

Moving this from a downstream bug report:
https://bugzilla.redhat.com/show_bug.cgi?id=840892

[abrt] evolution-3.4.3-2.fc17: __GI_____strtol_l_internal: Process /usr/bin/evolution was killed by signal 11 (SIGSEGV)

libreport version: 2.0.10
abrt_version:   2.0.10
backtrace_rating: 3
cmdline:        evolution
crash_function: __GI_____strtol_l_internal
executable:     /usr/bin/evolution
kernel:         3.4.4-3.fc17.x86_64
time:           Tue 17 Jul 2012 14:51:58 BST

+ Trace 230530

Thread 1 (Thread 0x7f8e9a7fc700 (LWP 32464))

#0 __GI_____strtol_l_internal
at ../stdlib/strtol_l.c line 298
#1 message_info_from_db
at camel-ews-summary.c line 203
#2 camel_read_mir_callback
at camel-folder-summary.c line 2376
#3 sqlite3_exec
at sqlite3.c line 90907
#4 cdb_sql_exec
at camel-db.c line 414
#5 camel_db_select
at camel-db.c line 988
#6 camel_db_read_message_info_records
at camel-db.c line 1958
#7 cfs_reload_from_db
at camel-folder-summary.c line 2142
#8 camel_folder_summary_prepare_fetch_all
at camel-folder-summary.c line 2201
#9 camel_folder_thread_messages_new
at camel-folder-thread.c line 633
#10 regen_list_exec
at message-list.c line 4756
#11 mail_msg_proxy
at mail-mt.c line 423
#12 ??
from /lib64/libglib-2.0.so.0
#13 g_thread_error_quark
from /lib64/libglib-2.0.so.0
#14 ??

Comment 1 Milan Crha 2012-07-18 16:58:54 UTC

Created attachment 219140 [details] [review]
ews patch

for evolution-ews;

There doesn't seem to be done much here, the summary record is broken, probably due to some folders.db corruption caused by sudden crash of evolution. Avoid crash-caused-by-previous-crash by this way seems to be good to do.

Comment 2 Milan Crha 2012-07-18 17:02:43 UTC

Created attachment 219141 [details] [review]
ews patch ][

for evolution-ews;

Oops, the previous commit could cause free of uninitialized memory, which this patch fixes.

Comment 3 Milan Crha 2012-07-18 17:04:48 UTC

Created commits de8e452 and b5dc492 in ews master (3.5.5+)
Created commit dac3c3e in ews gnome-3-4 (3.4.4+)