Bug 644532 – Crash when opening ESource with local:/system URI

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 644532 - Crash when opening ESource with local:/system URI


Summary:	Crash when opening ESource with local:/system URI


Status:	RESOLVED FIXED

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

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

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2011-03-11 19:41 UTC by Bill Nottingham
Modified:	2013-09-14 16:53 UTC

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

Attachments
eds patch (2.36 KB, patch) 2011-03-14 10:20 UTC, Milan Crha	committed	Details \| Review

Description Bill Nottingham 2011-03-11 19:41:59 UTC

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

+ Trace 226276

Thread 1 (Thread 0x7f56ab7807e0 (LWP 513))

#0 g_str_hash
at gstring.c line 142
#1 g_hash_table_lookup_node
at ghash.c line 313
#2 g_hash_table_lookup
at ghash.c line 902
#3 impl_BookFactory_getBook
at e-data-book-factory.c line 331
#4 _e_gdbus_gdbus_cclosure_marshaller_BOOLEAN__OBJECT_STRING
at e-gdbus-marshallers.c line 84
#5 g_closure_invoke
at gclosure.c line 767
#6 signal_emit_unlocked_R
at gsignal.c line 3252
#7 g_signal_emit_valist
at gsignal.c line 2993
#8 g_signal_emit
at gsignal.c line 3040
#9 handle_method_call
at e-gdbus-egdbusbookfactory.c line 385
#10 call_in_idle_cb
at gdbusconnection.c line 4427
#11 g_main_dispatch
at gmain.c line 2440
#12 g_main_context_dispatch
at gmain.c line 3013
#13 g_main_context_iterate
at gmain.c line 3091
#14 g_main_loop_run
at gmain.c line 3299
#15 main
at e-data-book-factory.c line 675


evolution-data-server-2.91.91-1.fc15.x86_64

Comment 1 Bill Nottingham 2011-03-11 19:42:26 UTC

Triggered by nautilus-sendto (see bug 644531).

Comment 2 Milan Crha 2011-03-14 10:07:10 UTC

Confirming, I can reproduce it too. And it's my fault, by the way.

Comment 3 Milan Crha 2011-03-14 10:20:42 UTC

Created attachment 183323 [details] [review]
eds patch

for evolution-data-server;

I added there this behaviour couple weeks ago. The intention was to remove any absolute uri of a form "local:/system", but I was removing it, not replacing with a correct one "local:system" (the slash should not be there).

The e-contact-entry.c in nautilus-sendto is creating that "local:/system" uri, so it invoked this code on the book factory side, which requires this absolute uri, and as it returned NULL, with a warning on the console about asking for an absolute uri on a source without it set, and later wasn't checked whether it is or is not NULL, then it crashed afterwards in GLib function.

Comment 4 Milan Crha 2011-03-14 10:22:19 UTC

Created commit 1443076 in eds master (2.91.92+)