Re: question about tny_camel_folder_add_msg_async_default

["Dirk-Jan C. Binnema" <dirk-jan binnema nokia com>]

Hi Philip,

On Thu, 10 Jan 2008, ext Philip Van Hoof wrote:

> 
> On Thu, 2008-01-10 at 18:25 +0200, Dirk-Jan Binnema nokia com wrote:
> 
> > Did some more valgrinding; I found the following, when saving
> > messages to draft. It seems this issue happens for all of the async
> > funtions in tny_camel_folder.
> 
> What valgrind errors triggered you into investigating this? Perhaps a
> real memory mistake in Tinymail?
> 
> Such mistakes must of course be fixed :)

Sergio showed that part of this was some misunderstanding on my part; at
least it seems tinymail does not behave the way I thought (Sergio
explained this is the ML).


Anyway, my valgrind jihad brought the following:


==32492== Invalid read of size 4
==32492==    at 0x4B7EDF8: g_type_check_instance_cast (gtype.c:3143)
==32492==    by 0x808C06A: on_save_to_drafts_cb
(modest-ui-actions.c:2240)
==32492==    by 0x806FF09:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1029)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==  Address 0x539EB90 is 0 bytes inside a block of size 424
free'd
==32492==    at 0x4021258: free (vg_replace_malloc.c:233)
==32492==    by 0x4BC3D10: g_free (gmem.c:187)
==32492==    by 0x4BD3B40: g_slice_free1 (gslice.c:886)
==32492==    by 0x4B7B559: g_type_free_instance (gtype.c:1448)
==32492==    by 0x4B62B78: g_object_unref (gobject.c:1803)
==32492==    by 0x46DD04F: gtk_main_do_event (gtkmain.c:1594)
==32492==    by 0x4967550: gdk_event_dispatch (gdkevents-x11.c:2318)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)


==32492== Invalid read of size 4
==32492==    at 0x80BCF18: modest_msg_edit_window_set_draft
(modest-msg-edit-window.c:3148)
==32492==    by 0x808C0EA: on_save_to_drafts_cb
(modest-ui-actions.c:2251)
==32492==    by 0x806FF09:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1029)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==  Address 0x539EB90 is 0 bytes inside a block of size 424
free'd
==32492==    at 0x4021258: free (vg_replace_malloc.c:233)
==32492==    by 0x4BC3D10: g_free (gmem.c:187)
==32492==    by 0x4BD3B40: g_slice_free1 (gslice.c:886)
==32492==    by 0x4B7B559: g_type_free_instance (gtype.c:1448)
==32492==    by 0x4B62B78: g_object_unref (gobject.c:1803)
==32492==    by 0x46DD04F: gtk_main_do_event (gtkmain.c:1594)
==32492==    by 0x4967550: gdk_event_dispatch (gdkevents-x11.c:2318)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)


==32492== Invalid read of size 4
==32492==    at 0x4B7DA08: g_type_check_instance_is_a (gtype.c:3110)
==32492==    by 0x80BCF43: modest_msg_edit_window_set_draft
(modest-msg-edit-window.c:3148)
==32492==    by 0x808C0EA: on_save_to_drafts_cb
(modest-ui-actions.c:2251)
==32492==    by 0x806FF09:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1029)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==  Address 0x539EB90 is 0 bytes inside a block of size 424
free'd
==32492==    at 0x4021258: free (vg_replace_malloc.c:233)
==32492==    by 0x4BC3D10: g_free (gmem.c:187)
==32492==    by 0x4BD3B40: g_slice_free1 (gslice.c:886)
==32492==    by 0x4B7B559: g_type_free_instance (gtype.c:1448)
==32492==    by 0x4B62B78: g_object_unref (gobject.c:1803)
==32492==    by 0x46DD04F: gtk_main_do_event (gtkmain.c:1594)
==32492==    by 0x4967550: gdk_event_dispatch (gdkevents-x11.c:2318)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)

==32492== Invalid read of size 4
==32492==    at 0x806FFBA:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1041)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==  Address 0x5358820 is 24 bytes inside a block of size 28
free'd
==32492==    at 0x4021258: free (vg_replace_malloc.c:233)
==32492==    by 0x4BC3D10: g_free (gmem.c:187)
==32492==    by 0x4BD3B40: g_slice_free1 (gslice.c:886)
==32492==    by 0x806FFB6:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1039)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492== 

==32492== Invalid read of size 4
==32492==    at 0x806FFC8:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1042)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==  Address 0x5358820 is 24 bytes inside a block of size 28
free'd
==32492==    at 0x4021258: free (vg_replace_malloc.c:233)
==32492==    by 0x4BC3D10: g_free (gmem.c:187)
==32492==    by 0x4BD3B40: g_slice_free1 (gslice.c:886)
==32492==    by 0x806FFB6:
modest_mail_operation_save_to_drafts_add_msg_cb
(modest-mail-operation.c:1039)
==32492==    by 0x41A1A7F: tny_camel_folder_add_msg_async_callback
(tny-camel-folder.c:841)
==32492==    by 0x41BCD26: perform_callback (tny-camel-queue.c:104)
==32492==    by 0x4BBF4D0: g_idle_dispatch (gmain.c:3928)
==32492==    by 0x4BBC1FB: g_main_context_dispatch (gmain.c:2045)
==32492==    by 0x4BBDBB4: g_main_context_iterate (gmain.c:2677)
==32492==    by 0x4BBDED9: g_main_loop_run (gmain.c:2881)
==32492==    by 0x46DC172: gtk_main (gtkmain.c:1155)
==32492==    by 0x80747A9: main (modest-main.c:157)
==32492==