Bug 651709 – Crash on deselect in new calendar folder tree

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 651709 - Crash on deselect in new calendar folder tree


Summary:	Crash on deselect in new calendar folder tree


Status:	RESOLVED FIXED

Product:	evolution-mapi
Classification:	Applications
Component:	Calendar
Version:	0.32.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	evolution-mapi-maint
QA Contact:	evolution-mapi-maint

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2011-06-02 14:09 UTC by Pedro Villavicencio
Modified:	2011-08-19 11:04 UTC

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

Attachments
ema patch (1.12 KB, patch) 2011-08-19 11:02 UTC, Milan Crha	committed	Details \| Review

Description Pedro Villavicencio 2011-06-02 14:09:30 UTC

this report has been filed here:

https://bugs.launchpad.net/ubuntu/+source/evolution-mapi/+bug/787627

"Tried to add an additional MAPI calendar."

".

+ Trace 227350

Thread 1 (Thread 8974)

#0 gtk_tree_model_get_valist
from /usr/lib/libgtk-x11-2.0.so.0
#1 gtk_tree_model_get
from /usr/lib/libgtk-x11-2.0.so.0
#2 exchange_mapi_cursor_change
at exchange-mapi-account-setup.c line 604
#3 g_closure_invoke
at /build/buildd/glib2.0-2.28.6/./gobject/gclosure.c line 767
#4 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3252
#5 g_signal_emit_valist
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 2983
#6 g_signal_emit
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3040
#7 ??
from /usr/lib/libgtk-x11-2.0.so.0
#8 ??
from /usr/lib/libgtk-x11-2.0.so.0
#9 g_closure_invoke
at /build/buildd/glib2.0-2.28.6/./gobject/gclosure.c line 767
#10 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3290
#11 g_signal_emit_valist
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 2983
#12 g_signal_emit
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3040
#13 gtk_widget_grab_focus
from /usr/lib/libgtk-x11-2.0.so.0
#14 ??
from /usr/lib/libgtk-x11-2.0.so.0
#15 ??
from /usr/lib/libgtk-x11-2.0.so.0
#16 ??
from /usr/lib/libgtk-x11-2.0.so.0
#17 g_closure_invoke
at /build/buildd/glib2.0-2.28.6/./gobject/gclosure.c line 767
#18 signal_emit_unlocked_R
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3290
#19 g_signal_emit_valist
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 2993
#20 g_signal_emit
at /build/buildd/glib2.0-2.28.6/./gobject/gsignal.c line 3040
#21 ??
from /usr/lib/libgtk-x11-2.0.so.0
#22 gtk_propagate_event
from /usr/lib/libgtk-x11-2.0.so.0
#23 gtk_main_do_event
from /usr/lib/libgtk-x11-2.0.so.0
#24 ??
from /usr/lib/libgdk-x11-2.0.so.0
#25 g_main_dispatch
at /build/buildd/glib2.0-2.28.6/./glib/gmain.c line 2440
#26 g_main_context_dispatch
at /build/buildd/glib2.0-2.28.6/./glib/gmain.c line 3013
#27 g_main_context_iterate
at /build/buildd/glib2.0-2.28.6/./glib/gmain.c line 3091
#28 g_main_loop_run
at /build/buildd/glib2.0-2.28.6/./glib/gmain.c line 3299
#29 gtk_main
from /usr/lib/libgtk-x11-2.0.so.0
#30 ??
#31 __libc_start_main
from /lib/x86_64-linux-gnu/libc.so.6
#32 ??
#33 ??
#34 ??
#35 ??
#36 ??
#37 ??

Comment 1 Milan Crha 2011-08-19 11:01:17 UTC

Confirming, I can reproduce it too after little fidling with selected/unselected nodes in the source tree and shift+clicking in that tree.

Comment 2 Milan Crha 2011-08-19 11:02:46 UTC

Created attachment 194216 [details] [review]
ema patch

for evolution-mapi;

The thing is that the selection returned FALSE, which means that there is nothing selected, thus the iterator was invalid and the tree store was accessing an invalid memory.

Comment 3 Milan Crha 2011-08-19 11:04:46 UTC

Created commit bc49d37 in ema master (3.1.90+)