diff options
| author | Milan Crha <mcrha@redhat.com> | 2011-01-11 20:57:35 +0800 |
|---|---|---|
| committer | Milan Crha <mcrha@redhat.com> | 2011-01-11 20:58:40 +0800 |
| commit | 098a7abf652e5a1596801c466f0de0fb656b69c9 (patch) | |
| tree | 97e0d45d96e4230d1fc661b8eae2cb705f36bdf3 | |
| parent | 886b3ac072e2d7dde15b4ad3d0abf879b82f849d (diff) | |
| download | gsoc2013-evolution-098a7abf652e5a1596801c466f0de0fb656b69c9.tar.gz gsoc2013-evolution-098a7abf652e5a1596801c466f0de0fb656b69c9.tar.zst gsoc2013-evolution-098a7abf652e5a1596801c466f0de0fb656b69c9.zip | |
Bug #634305 - Crash on move of contacts between books
| -rw-r--r-- | addressbook/gui/widgets/e-addressbook-selector.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/addressbook/gui/widgets/e-addressbook-selector.c b/addressbook/gui/widgets/e-addressbook-selector.c index 402fcf9295..110343ce80 100644 --- a/addressbook/gui/widgets/e-addressbook-selector.c +++ b/addressbook/gui/widgets/e-addressbook-selector.c @@ -42,6 +42,7 @@ struct _MergeContext { EContact *current_contact; GList *remaining_contacts; guint pending_removals; + gboolean pending_adds; gint remove_from_source : 1; gint copy_done : 1; @@ -63,6 +64,10 @@ merge_context_next (MergeContext *merge_context) { GList *list; + merge_context->current_contact = NULL; + if (!merge_context->remaining_contacts) + return; + list = merge_context->remaining_contacts; merge_context->current_contact = list->data; list = g_list_delete_link (list, list); @@ -104,7 +109,7 @@ addressbook_selector_removed_cb (EBook *book, { merge_context->pending_removals--; - if (merge_context->remaining_contacts != NULL) + if (merge_context->pending_adds) return; if (merge_context->pending_removals > 0) @@ -139,8 +144,10 @@ addressbook_selector_merge_next_cb (EBook *book, (EBookIdAsyncCallback) addressbook_selector_merge_next_cb, merge_context); - } else if (merge_context->pending_removals == 0) + } else if (merge_context->pending_removals == 0) { merge_context_free (merge_context); + } else + merge_context->pending_adds = FALSE; } static void @@ -275,6 +282,7 @@ addressbook_selector_data_dropped (ESourceSelector *selector, merge_context = merge_context_new (source_book, target_book, list); merge_context->remove_from_source = remove_from_source; + merge_context->pending_adds = TRUE; eab_merging_book_add_contact ( target_book, merge_context->current_contact, |