Bug 338203 – EDS Crash @ Startup

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 338203 - EDS Crash @ Startup


Summary:	EDS Crash @ Startup


Status:	RESOLVED FIXED

Product:	evolution-data-server
Classification:	Platform
Component:	Calendar
Version:	1.6.x (obsolete)
Hardware:	Other All

Importance:	High critical
Target Milestone:	---
Assigned To:	Chenthill P
QA Contact:	Evolution QA team

URL:
Whiteboard:

Duplicates:	337374 344210 (view as bug list)
Depends on:	343979
Blocks:

Reported:	2006-04-12 11:03 UTC by David Richards
Modified:	2006-07-29 08:37 UTC

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

Attachments
Fixes the bug (1.45 KB, patch) 2006-07-26 19:10 UTC, Chenthill P	reviewed	Details \| Review

Description David Richards 2006-04-12 11:03:57 UTC

Steps to reproduce:
Often reported issue of getting a crash at startup, finally I got one myself:

Stack trace:
Backtrace was generated from '/opt/gnome/libexec/evolution-data-server-1.6'

Using host libthread_db library "/lib/tls/libthread_db.so.1".
[Thread debugging using libthread_db enabled]
[New Thread 1085556288 (LWP 25479)]
[New Thread 1094069168 (LWP 25542)]
[Thread debugging using libthread_db enabled]
[New Thread 1085556288 (LWP 25479)]
[New Thread 1094069168 (LWP 25542)]
[Thread debugging using libthread_db enabled]
[New Thread 1085556288 (LWP 25479)]
[New Thread 1094069168 (LWP 25542)]
[New Thread 1096170416 (LWP 25541)]
[New Thread 1103461296 (LWP 25519)]
[New Thread 1091963824 (LWP 25486)]
0xffffe410 in ?? ()

+ Trace 67586

Thread 3 (Thread 1096170416 (LWP 25541))

#0 ??
#1 ??
#2 ??
#3 ??
#4 __lll_mutex_lock_wait
from /lib/tls/libpthread.so.0
#5 _L_mutex_lock_33
from /lib/tls/libpthread.so.0
#6 ??
#7 ??
#8 ??
from /lib/tls/libc.so.6
#9 ??
#10 __libc_ptyname1
from /lib/tls/libc.so.6
#11 ??
#12 gnome_segv_handler
at server.c line 97
#13 gnome_segv_handler
at server.c line 97
#14 <signal handler called>
#15 getenv
from /lib/tls/libc.so.6
#16 tzset_internal
from /lib/tls/libc.so.6
#17 tzset
from /lib/tls/libc.so.6
#18 strftime_l
from /lib/tls/libc.so.6
#19 strftime
from /lib/tls/libc.so.6
#20 isodate_from_time_t
at e-cal-time-util.c line 577
#21 e_cal_component_gen_uid
at e-cal-component.c line 432
#22 e_cal_component_alarm_new
at e-cal-component.c line 5295
#23 e_gw_item_to_cal_component
at e-cal-backend-groupwise-utils.c line 1108
#24 populate_cache
at e-cal-backend-groupwise.c line 178
#25 cache_init
at e-cal-backend-groupwise.c line 649
#26 g_thread_create_full
from /opt/gnome/lib/libglib-2.0.so.0
#27 start_thread
from /lib/tls/libpthread.so.0
#28 clone
from /lib/tls/libc.so.6



Other information:

Comment 1 Christian Kirbach 2006-04-12 13:16:38 UTC

Thanks for the bug report. Unfortunately, that stack trace is not very useful in determining the cause of the crash. Can you get us one with debugging symbols? Please see http://live.gnome.org/GettingTraces for more information on how to do so.

Comment 2 Chenthill P 2006-04-17 08:17:22 UTC

This has the debugging traces.

Comment 3 Chenthill P 2006-05-02 16:32:08 UTC

*** Bug 337374 has been marked as a duplicate of this bug. ***

Comment 4 André Klapper 2006-05-10 01:29:30 UTC

could be a dup of bug 334065 according to stacktrace

Comment 5 Elijah Newren 2006-06-07 21:14:47 UTC

*** Bug 344210 has been marked as a duplicate of this bug. ***

Comment 6 Chenthill P 2006-07-26 19:10:13 UTC

Created attachment 69683 [details] [review]
Fixes the bug

Comment 7 David Richards 2006-07-26 19:54:45 UTC

chen:
   Is there a way to check for all places where this threading problem might happen and change all of them?  Looking forward to testing these EDS patches.

Comment 8 Chenthill P 2006-07-27 05:35:54 UTC

Yes, we have already replaced some of the non-thread safe functions with thread safe implementations. There are some pending, which is being delt now. All the above traces show the crash at the same point. We are looking into other crashes also and harish is grouping the related ones so that they can be solved at a higher level.

Comment 9 Harish Krishnaswamy 2006-07-27 06:18:49 UTC

Chen, I prefer the use of g_snprintf - the rest of the patch looks fine.
Also, 
s/
	size_t size = 17;
/
#define ISODATE_LENGTH 17 
/* 4+2+2+1+2+2+2+1 + 1 */
...

        size_t size = ISODATE_LENGTH;
/

would look more elegant.

Comment 10 David Richards 2006-07-27 13:04:45 UTC

Excellent that you are looking at these issues as part of the big picture.  I would bet the percentage of people running Evolution on a single processor is very high.  Our is running a quad-hyper-threaded machine, and shows up as 8 CPUs.  I think it creates a very different runtime environment for Evo.

Very much looking forward to testing these patches.

Comment 11 Chenthill P 2006-07-29 08:37:16 UTC

Made the suggested changes and committed the patch to cvs HEAD and gnome-2-14 branch.