Bug 552491 – evolution crashed with SIGSEGV in e_source_group_uid_from_xmldoc()

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 552491 - evolution crashed with SIGSEGV in e_source_group_uid_from_xmldoc()


Summary:	evolution crashed with SIGSEGV in e_source_group_uid_from_xmldoc()


Status:	RESOLVED FIXED

Product:	evolution-data-server
Classification:	Platform
Component:	Contacts
Version:	2.24.x (obsolete)
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	evolution-addressbook-maintainers
QA Contact:	Evolution QA team

URL:
Whiteboard:

Duplicates:	457525 473994 556108 578791 (view as bug list)
Depends on:
Blocks:

Reported:	2008-09-16 12:30 UTC by Pedro Villavicencio
Modified:	2011-01-27 12:07 UTC

See Also:
GNOME target:	---
GNOME version:	2.23/2.24

Attachments
Check for NULL output from xmlParseDoc before continuing. (498 bytes, patch) 2008-09-21 16:58 UTC, Philip Withnall	committed	Details \| Review

Description Pedro Villavicencio 2008-09-16 12:30:35 UTC

this report has been filed here:

https://bugs.edge.launchpad.net/ubuntu/+source/evolution-data-server/+bug/269832

".

+ Trace 206837

Thread 1 (process 16154)

#0 e_source_group_uid_from_xmldoc
at e-source-group.c line 525
#1 e_source_list_is_gconf_updated
at e-source-list.c line 631
#2 e_source_list_sync
at e-source-list.c line 592
#3 sync_idle_callback
at e-source-list.c line 198
#4 IA__g_main_context_dispatch
at /build/buildd/glib2.0-2.18.0/glib/gmain.c line 2142
#5 g_main_context_iterate
at /build/buildd/glib2.0-2.18.0/glib/gmain.c line 2775
#6 IA__g_main_loop_run
at /build/buildd/glib2.0-2.18.0/glib/gmain.c line 2983
#7 bonobo_main
at bonobo-main.c line 311
#8 main
at main.c line 689

Comment 1 Philip Withnall 2008-09-21 16:58:06 UTC

Created attachment 119115 [details] [review]
Check for NULL output from xmlParseDoc before continuing.

Add a missing NULL check after parsing the XML file. Should fix the crash.

Comment 2 Srinivasa Ragavan 2008-09-30 17:21:54 UTC

Commit to stable/trunk

Comment 3 Philip Withnall 2008-09-30 18:19:57 UTC

2008-09-30  Philip Withnall  <philip@tecnocode.co.uk>

	** Fix for bug #552491

	* libedataserver/e-source-list.c (e_source_list_is_gconf_updated):
	Check for NULL output from xmlParseDoc before continuing.

Comment 4 André Klapper 2008-10-15 20:01:38 UTC

Hey kidz,
arr bug 552491 end bug 473994 doopz of tis one?

Comment 5 Philip Withnall 2008-10-15 20:35:58 UTC

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

Comment 6 Philip Withnall 2008-10-15 20:37:11 UTC

Well, bug #552491 isn't a dupe of itself, but the other one is a dupe (and now marked as such). :-)

Comment 7 André Klapper 2008-10-15 20:48:47 UTC

Gnarf! I meant bug 556108.

Comment 8 Philip Withnall 2008-10-15 20:56:00 UTC

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

Comment 9 palfrey 2009-04-20 10:23:06 UTC

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

Comment 10 Milan Crha 2011-01-27 12:07:26 UTC

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