diff options
| author | Milan Crha <mcrha@redhat.com> | 2008-10-20 01:08:50 +0800 |
|---|---|---|
| committer | Bharath Acharya <abharath@src.gnome.org> | 2008-10-20 01:08:50 +0800 |
| commit | 3acdc755d63edcc65ef58adefbced984d37e7c50 (patch) | |
| tree | 2ee90877dfcbe97835623e3c75e3cc242a87ce75 /mail/message-list.c | |
| parent | 8b854f7f9448be3e2f351e8a37a9d620a17cd976 (diff) | |
| download | gsoc2013-evolution-3acdc755d63edcc65ef58adefbced984d37e7c50.tar.gz gsoc2013-evolution-3acdc755d63edcc65ef58adefbced984d37e7c50.tar.zst gsoc2013-evolution-3acdc755d63edcc65ef58adefbced984d37e7c50.zip | |
** Fix for bug #438155
2008-10-19 Milan Crha <mcrha@redhat.com>
** Fix for bug #438155
* message-list.h: (struct _MessageList): message-list.c: (message_list_init),
(message_list_finalise), (regen_list_done), (regen_list_free), (ml_regen_timeout),
(mail_regen_cancel): Guard access to the 'regen' list with a lock.
svn path=/trunk/; revision=36652
Diffstat (limited to 'mail/message-list.c')
| -rw-r--r-- | mail/message-list.c | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/mail/message-list.c b/mail/message-list.c index d8d92af49d..7333467fd1 100644 --- a/mail/message-list.c +++ b/mail/message-list.c @@ -2274,6 +2274,8 @@ message_list_init (MessageList *message_list) message_list->cursor_uid = NULL; message_list->last_sel_single = FALSE; + message_list->regen_lock = g_mutex_new (); + /* TODO: Should this only get the selection if we're realised? */ p = message_list->priv = g_malloc0(sizeof(*message_list->priv)); p->invisible = gtk_invisible_new(); @@ -2375,6 +2377,7 @@ message_list_finalise (GObject *object) g_free(message_list->cursor_uid); g_mutex_free(message_list->hide_lock); + g_mutex_free (message_list->regen_lock); g_free(message_list->folder_uri); message_list->folder_uri = NULL; @@ -4256,7 +4259,9 @@ regen_list_done (struct _regen_list_msg *m) m->ml->search = m->search; m->search = NULL; + g_mutex_lock (m->ml->regen_lock); m->ml->regen = g_list_remove(m->ml->regen, m); + g_mutex_unlock (m->ml->regen_lock); if (m->ml->regen == NULL && m->ml->pending_select_uid) { char *uid = m->ml->pending_select_uid; @@ -4319,7 +4324,9 @@ regen_list_free (struct _regen_list_msg *m) camel_folder_change_info_free (m->changes); /* we have to poke this here as well since we might've been cancelled and regened wont get called */ + g_mutex_lock (m->ml->regen_lock); m->ml->regen = g_list_remove(m->ml->regen, m); + g_mutex_unlock (m->ml->regen_lock); if (m->expand_state) xmlFreeDoc (m->expand_state); @@ -4340,7 +4347,9 @@ ml_regen_timeout(struct _regen_list_msg *m) { e_profile_event_emit("list.regenerate", m->folder->full_name, 0); + g_mutex_lock (m->ml->regen_lock); m->ml->regen = g_list_prepend(m->ml->regen, m); + g_mutex_unlock (m->ml->regen_lock); /* TODO: we should manage our own thread stuff, would make cancelling outstanding stuff easier */ mail_msg_fast_ordered_push (m); @@ -4355,8 +4364,11 @@ mail_regen_cancel(MessageList *ml) { /* cancel any outstanding regeneration requests, not we don't clear, they clear themselves */ if (ml->regen) { - GList *l = ml->regen; + GList *l; + + g_mutex_lock (ml->regen_lock); + l = ml->regen; while (l) { MailMsg *mm = l->data; @@ -4364,6 +4376,8 @@ mail_regen_cancel(MessageList *ml) camel_operation_cancel(mm->cancel); l = l->next; } + + g_mutex_unlock (ml->regen_lock); } /* including unqueued ones */ |