Bug 674599 – [abrt] Crash on contact view open

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 674599 - [abrt] Crash on contact view open


Summary:	[abrt] Crash on contact view open


Status:	RESOLVED FIXED

Product:	evolution-ews
Classification:	Other
Component:	Contacts (personal addressbooks)
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-04-23 06:38 UTC by Milan Crha
Modified:	2012-06-06 11:10 UTC

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

Attachments
ews patch (6.01 KB, patch) 2012-06-06 11:00 UTC, Milan Crha	committed	Details \| Review

Description Milan Crha 2012-04-23 06:38:40 UTC

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

[abrt] evolution-data-server-3.4.1-1.fc17: Process /usr/libexec/evolution-addressbook-factory was killed by signal 11 (SIGSEGV)

libreport version: 2.0.10
abrt_version:   2.0.10
backtrace_rating: 4
cmdline:        /usr/libexec/evolution-addressbook-factory
comment:        Opened contacts view in Evolution using EWS
crash_function: e_book_backend_sqlitedb_get_is_populated
executable:     /usr/libexec/evolution-addressbook-factory
kernel:         3.3.2-1.fc17.x86_64
time:           Fri 20 Apr 2012 01:30:29 PM EDT

Core was generated by `/usr/libexec/evolution-addressbook-factory'.
Program terminated with signal 11, Segmentation fault.

+ Trace 230120

Thread 1 (Thread 0x7fde5ffff700 (LWP 18908))

#0 e_book_backend_sqlitedb_get_is_populated
at e-book-backend-sqlitedb.c line 1820
#1 e_book_backend_ews_get_contact_list
#2 operation_thread
at e-data-book.c line 165
#3 g_thread_pool_thread_proxy
at gthreadpool.c line 309
#4 g_thread_proxy
at gthread.c line 801
#5 start_thread
at pthread_create.c line 309
#6 clone
at ../sysdeps/unix/sysv/linux/x86_64/clone.S line 115

Comment 1 Milan Crha 2012-06-06 10:57:48 UTC

I was able to reproduce this for couple times, but then it suddenly stopped crashing for me. After looking around the code I would add couple sanity tests, just in case.

Comment 2 Milan Crha 2012-06-06 11:00:00 UTC

Created attachment 215730 [details] [review]
ews patch

for evolution-ews;

Let's add these sanity tests. It also fixes one memory leak and makes sure the GAL books are marked always as read-only.

Comment 3 Milan Crha 2012-06-06 11:10:00 UTC

Created commit c1089c0 in ews master (3.5.3+)
Created commit 294101a in ews gnome-3-4 (3.4.3+)