Bug 641108 – Crash on new folder move when unsubscribing

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 641108 - Crash on new folder move when unsubscribing


Summary:	Crash on new folder move when unsubscribing


Status:	RESOLVED FIXED

Product:	evolution-mapi
Classification:	Applications
Component:	Mail
Version:	2.91.x
Hardware:	Other Linux

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

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2011-02-01 05:48 UTC by Akhil Laddha
Modified:	2011-02-08 18:37 UTC

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

Attachments
ema patch (3.13 KB, patch) 2011-02-08 18:34 UTC, Milan Crha	committed	Details \| Review

Description Akhil Laddha 2011-02-01 05:48:11 UTC

evolution 2.91.6
gtk+ master
glib master

1. Created a folder 'A'
2. Created a folder 'C'
3. Selected folder 'C', right click, chose 'move to folder'
4. Selected folder 'A' in move to dialog
5. Click Move
6. evolution crashed

Program received signal SIGSEGV, Segmentation fault.

+ Trace 225787

Thread 1 (Thread 0xb5fe3850 (LWP 30214))

#0 gtk_tree_store_get_value
at gtktreestore.c line 656
#1 gtk_tree_model_get_value
at gtktreemodel.c line 1353
#2 gtk_tree_model_get_valist
at gtktreemodel.c line 1723
#3 gtk_tree_model_get
at gtktreemodel.c line 1687
#4 em_folder_tree_model_remove_folders
at em-folder-tree-model.c line 1166
#5 folder_renamed_cb
at em-folder-tree-model.c line 999
#6 camel_marshal_VOID__STRING_POINTER
at camel-marshal.c line 121
#7 g_closure_invoke
at gclosure.c line 767
#8 signal_emit_unlocked_R
at gsignal.c line 3252
#9 g_signal_emit_valist
at gsignal.c line 2983
#10 g_signal_emit
at gsignal.c line 3040
#11 store_emit_folder_renamed_cb
at camel-store.c line 158
#12 g_idle_dispatch
at gmain.c line 4536
#13 g_main_dispatch
at gmain.c line 2440
#14 g_main_context_dispatch
at gmain.c line 3013
#15 g_main_context_iterate
at gmain.c line 3091
#16 g_main_loop_run
at gmain.c line 3299
#17 gtk_main
at gtkmain.c line 1338
#18 main
at main.c line 734

Comment 1 Milan Crha 2011-02-08 18:34:27 UTC

Created attachment 180407 [details] [review]
ema patch

for evolution-mapi;

It's a "correct" crash, the folder wasn't in the cache hence the 'si' was NULL, so dereferencing it is causing the crash.

Comment 2 Milan Crha 2011-02-08 18:36:35 UTC

Created commit c922117 in ema master (2.91.90+)