[evolution] Bug #563795 - Crashed while importing an mbox file in local folder
- From: Milan Crha <mcrha src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [evolution] Bug #563795 - Crashed while importing an mbox file in local folder
- Date: Wed, 5 Aug 2009 17:19:30 +0000 (UTC)
commit af049df0c5451734101ff53c6dd1f6e11716bf58
Author: Milan Crha <mcrha redhat com>
Date: Wed Aug 5 19:17:58 2009 +0200
Bug #563795 - Crashed while importing an mbox file in local folder
shell/e-shell-importer.c | 25 ++++++++++++++++++++++---
1 files changed, 22 insertions(+), 3 deletions(-)
---
diff --git a/shell/e-shell-importer.c b/shell/e-shell-importer.c
index 9ad741a..b24eca1 100644
--- a/shell/e-shell-importer.c
+++ b/shell/e-shell-importer.c
@@ -439,7 +439,10 @@ static void
import_druid_cancel (GnomeDruid *druid,
ImportData *data)
{
- gtk_widget_destroy (GTK_WIDGET (data->dialog));
+ if (data->import_dialog)
+ gtk_dialog_response (GTK_DIALOG (data->import_dialog), GTK_RESPONSE_CANCEL);
+ else
+ gtk_widget_destroy (GTK_WIDGET (data->dialog));
}
static gboolean
@@ -460,6 +463,15 @@ import_druid_weak_notify (gpointer blah,
{
ImportData *data = (ImportData *) blah;
+ if (data->import_dialog && (GObject *)data->import_dialog != where_the_object_was) {
+ /* postpone freeing of 'data' after the 'import_dialog' will stop,
+ but also indicate that the 'dialog' gone already */
+ data->dialog = NULL;
+ g_object_weak_ref ((GObject *)data->import_dialog, import_druid_weak_notify, data);
+ gtk_dialog_response (GTK_DIALOG (data->import_dialog), GTK_RESPONSE_CANCEL);
+ return;
+ }
+
if (data->importerpage->target)
e_import_target_free(data->import, data->importerpage->target);
g_slist_free(data->importerpage->importers);
@@ -493,9 +505,16 @@ static void
import_done(EImport *ei, gpointer d)
{
ImportData *data = d;
+ gboolean have_dialog = data->dialog != NULL;
+
+ gtk_widget_destroy (data->import_dialog);
- gtk_widget_destroy(data->import_dialog);
- gtk_widget_destroy(data->dialog);
+ /* if doesn't have dialog, then the 'data' pointer is freed
+ on the above destroy call */
+ if (have_dialog) {
+ data->import_dialog = NULL;
+ gtk_widget_destroy (data->dialog);
+ }
}
static void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]