Make EAttachment a little more thread-safe.

EAttachment is now used from worker threads by EMailFormatterAttachment,
so add some thread-safe accessor functions to eliminate potential races.

Added thread-safe functions:

    e_attachment_dup_disposition()
    e_attachment_ref_file()
    e_attachment_ref_file_info()
    e_attachment_ref_icon()
    e_attachment_ref_mime_part()
    e_attachment_dup_description()
    e_attachment_dup_thumbnail_path()

Renamed functions:

    e_attachment_get_mime_type() -> e_attachment_dup_mime_type()

Removed non-thread-safe functions:

    e_attachment_get_file()
    e_attachment_get_file_info()
    e_attachment_get_icon()
    e_attachment_get_mime_part()
    e_attachment_get_description()
    e_attachment_get_thumbnail_path()

7 files changed, 228 insertions, 93 deletions

diff --git a/e-util/e-attachment-button.c b/e-util/e-attachment-button.c
index 7778bb106f..701b20eae1 100644
--- a/e-util/e-attachment-button.c
+++ b/e-util/e-attachment-button.c
@@ -277,7 +277,7 @@ attachment_button_expand_drag_data_get_cb (EAttachmentButton *button,
 	attachment = e_attachment_button_get_attachment (button);
 
 	if (attachment != NULL)
-		mime_type = e_attachment_get_mime_type (attachment);
+		mime_type = e_attachment_dup_mime_type (attachment);
 
 	if (mime_type != NULL) {
 		gboolean processed = FALSE;
@@ -290,7 +290,7 @@ attachment_button_expand_drag_data_get_cb (EAttachmentButton *button,
 		if (g_strcmp0 (atom_name, mime_type) == 0) {
 			CamelMimePart *mime_part;
 
-			mime_part = e_attachment_get_mime_part (attachment);
+			mime_part = e_attachment_ref_mime_part (attachment);
 
 			if (mime_part != NULL) {
 				CamelDataWrapper *wrapper;
@@ -314,6 +314,8 @@ attachment_button_expand_drag_data_get_cb (EAttachmentButton *button,
 				processed = TRUE;
 
 				g_byte_array_free (buffer, TRUE);
+
+				g_object_unref (mime_part);
 			}
 		}
 
@@ -789,7 +791,7 @@ e_attachment_button_set_attachment (EAttachmentButton *button,
 	if (attachment != NULL) {
 		gchar *simple_type;
 
-		simple_type = e_attachment_get_mime_type (attachment);
+		simple_type = e_attachment_dup_mime_type (attachment);
 		if (simple_type != NULL) {
 			GtkTargetEntry attach_entry[] = { { NULL, 0, 2 } };
 
diff --git a/e-util/e-attachment-dialog.c b/e-util/e-attachment-dialog.c
index 9a9a1e7942..91021953f9 100644
--- a/e-util/e-attachment-dialog.c
+++ b/e-util/e-attachment-dialog.c
@@ -57,8 +57,8 @@ attachment_dialog_update (EAttachmentDialog *dialog)
 	GtkWidget *widget;
 	const gchar *content_type;
 	const gchar *display_name;
-	const gchar *description;
-	const gchar *disposition;
+	gchar *description;
+	gchar *disposition;
 	gchar *type_description = NULL;
 	gboolean sensitive;
 	gboolean active;
@@ -66,9 +66,9 @@ attachment_dialog_update (EAttachmentDialog *dialog)
 	attachment = e_attachment_dialog_get_attachment (dialog);
 
 	if (attachment != NULL) {
-		file_info = e_attachment_get_file_info (attachment);
-		description = e_attachment_get_description (attachment);
-		disposition = e_attachment_get_disposition (attachment);
+		file_info = e_attachment_ref_file_info (attachment);
+		description = e_attachment_dup_description (attachment);
+		disposition = e_attachment_dup_disposition (attachment);
 	} else {
 		file_info = NULL;
 		description = NULL;
@@ -120,7 +120,11 @@ attachment_dialog_update (EAttachmentDialog *dialog)
 	gtk_widget_set_sensitive (widget, sensitive);
 	gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (widget), active);
 
+	g_free (description);
+	g_free (disposition);
 	g_free (type_description);
+
+	g_clear_object (&file_info);
 }
 
 static void
@@ -231,10 +235,10 @@ attachment_dialog_response (GtkDialog *dialog,
 	g_return_if_fail (E_IS_ATTACHMENT (priv->attachment));
 	attachment = priv->attachment;
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 	g_return_if_fail (G_IS_FILE_INFO (file_info));
 
-	mime_part = e_attachment_get_mime_part (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 
 	attribute = G_FILE_ATTRIBUTE_STANDARD_DISPLAY_NAME;
 	text = gtk_entry_get_text (GTK_ENTRY (priv->display_name_entry));
@@ -258,6 +262,9 @@ attachment_dialog_response (GtkDialog *dialog,
 	if (mime_part != NULL)
 		camel_mime_part_set_disposition (mime_part, text);
 
+	g_clear_object (&file_info);
+	g_clear_object (&mime_part);
+
 	g_object_notify (G_OBJECT (attachment), "file-info");
 }
 
diff --git a/e-util/e-attachment-handler-image.c b/e-util/e-attachment-handler-image.c
index 602f001ba6..f4b35e354b 100644
--- a/e-util/e-attachment-handler-image.c
+++ b/e-util/e-attachment-handler-image.c
@@ -173,7 +173,7 @@ attachment_handler_image_update_actions_cb (EAttachmentView *view,
 	if (e_attachment_get_saving (attachment))
 		goto exit;
 
-	mime_type = e_attachment_get_mime_type (attachment);
+	mime_type = e_attachment_dup_mime_type (attachment);
 	visible =
 		(mime_type != NULL) &&
 		(g_ascii_strncasecmp (mime_type, "image/", 6) == 0);
diff --git a/e-util/e-attachment-store.c b/e-util/e-attachment-store.c
index 8ea2f8965b..08d7dc49e8 100644
--- a/e-util/e-attachment-store.c
+++ b/e-util/e-attachment-store.c
@@ -405,9 +405,11 @@ e_attachment_store_get_total_size (EAttachmentStore *store)
 		EAttachment *attachment = iter->data;
 		GFileInfo *file_info;
 
-		file_info = e_attachment_get_file_info (attachment);
-		if (file_info != NULL)
+		file_info = e_attachment_ref_file_info (attachment);
+		if (file_info != NULL) {
 			total_size += g_file_info_get_size (file_info);
+			g_object_unref (file_info);
+		}
 	}
 
 	g_list_foreach (list, (GFunc) g_object_unref, NULL);
@@ -556,13 +558,18 @@ e_attachment_store_run_save_dialog (EAttachmentStore *store,
 		const gchar *name = NULL;
 
 		attachment = attachment_list->data;
-		file_info = e_attachment_get_file_info (attachment);
+		file_info = e_attachment_ref_file_info (attachment);
+
 		if (file_info != NULL)
 			name = g_file_info_get_display_name (file_info);
+
 		if (name == NULL)
 			/* Translators: Default attachment filename. */
 			name = _("attachment.dat");
+
 		gtk_file_chooser_set_current_name (file_chooser, name);
+
+		g_clear_object (&file_info);
 	}
 
 	response = gtk_dialog_run (GTK_DIALOG (dialog));
@@ -728,7 +735,7 @@ e_attachment_store_get_uris_async (EAttachmentStore *store,
 		EAttachment *attachment = iter->data;
 		GFile *file;
 
-		file = e_attachment_get_file (attachment);
+		file = e_attachment_ref_file (attachment);
 		if (file != NULL) {
 			gchar *uri;
 
@@ -738,6 +745,8 @@ e_attachment_store_get_uris_async (EAttachmentStore *store,
 			/* Mark the list node for deletion. */
 			trash = g_list_prepend (trash, iter);
 			g_object_unref (attachment);
+
+			g_object_unref (file);
 		}
 	}
 
diff --git a/e-util/e-attachment-view.c b/e-util/e-attachment-view.c
index d078bd4b26..1fb868ed8b 100644
--- a/e-util/e-attachment-view.c
+++ b/e-util/e-attachment-view.c
@@ -174,7 +174,7 @@ action_open_with_cb (GtkAction *action,
 		E_ATTACHMENT_STORE_COLUMN_ATTACHMENT, &attachment, -1);
 	g_return_if_fail (E_IS_ATTACHMENT (attachment));
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 	g_return_if_fail (file_info != NULL);
 
 	content_type = g_file_info_get_content_type (file_info);
@@ -192,6 +192,8 @@ action_open_with_cb (GtkAction *action,
 		g_object_unref (app_info);
 	}
 
+	g_object_unref (file_info);
+
 	g_list_foreach (list, (GFunc) gtk_tree_path_free, NULL);
 	g_list_free (list);
 }
@@ -1560,7 +1562,7 @@ e_attachment_view_drag_begin (EAttachmentView *view,
 		gint width, height;
 
 		attachment = E_ATTACHMENT (priv->selected->data);
-		icon = e_attachment_get_icon (attachment);
+		icon = e_attachment_ref_icon (attachment);
 		g_return_if_fail (icon != NULL);
 
 		icon_theme = gtk_icon_theme_get_default ();
@@ -1587,6 +1589,8 @@ e_attachment_view_drag_begin (EAttachmentView *view,
 
 			gtk_icon_info_free (icon_info);
 		}
+
+		g_object_unref (icon);
 	}
 }
 
diff --git a/e-util/e-attachment.c b/e-util/e-attachment.c
index 9f758c510c..10f251379e 100644
--- a/e-util/e-attachment.c
+++ b/e-util/e-attachment.c
@@ -57,6 +57,8 @@
 #define ATTACHMENT_QUERY "standard::*,preview::*,thumbnail::*"
 
 struct _EAttachmentPrivate {
+	GMutex property_lock;
+
 	GFile *file;
 	GIcon *icon;
 	GFileInfo *file_info;
@@ -117,7 +119,7 @@ create_system_thumbnail (EAttachment *attachment,
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), FALSE);
 	g_return_val_if_fail (icon != NULL, FALSE);
 
-	file = e_attachment_get_file (attachment);
+	file = e_attachment_ref_file (attachment);
 	if (file != NULL)
 		file_path = g_file_get_path (file);
 
@@ -142,12 +144,14 @@ create_system_thumbnail (EAttachment *attachment,
 		GFileInfo *file_info;
 		const gchar *attribute;
 
-		file_info = e_attachment_get_file_info (attachment);
+		file_info = e_attachment_ref_file_info (attachment);
 		attribute = G_FILE_ATTRIBUTE_THUMBNAIL_PATH;
 
-		if (file_info != NULL)
+		if (file_info != NULL) {
 			g_file_info_set_attribute_byte_string (
 				file_info, attribute, thumbnail);
+			g_object_unref (file_info);
+		}
 	}
 
 	g_free (thumbnail);
@@ -155,6 +159,8 @@ create_system_thumbnail (EAttachment *attachment,
 	success = TRUE;
 
 exit:
+	g_clear_object (&file);
+
 	return success;
 }
 
@@ -198,10 +204,10 @@ attachment_update_file_info_columns (EAttachment *attachment)
 	GtkTreeIter iter;
 	GFileInfo *file_info;
 	const gchar *content_type;
-	const gchar *description;
 	const gchar *display_name;
 	gchar *content_desc;
 	gchar *display_size;
+	gchar *description;
 	gchar *caption;
 	goffset size;
 
@@ -209,7 +215,7 @@ attachment_update_file_info_columns (EAttachment *attachment)
 	if (!gtk_tree_row_reference_valid (reference))
 		return;
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 	if (file_info == NULL)
 		return;
 
@@ -225,9 +231,11 @@ attachment_update_file_info_columns (EAttachment *attachment)
 	content_desc = g_content_type_get_description (content_type);
 	display_size = g_format_size (size);
 
-	description = e_attachment_get_description (attachment);
-	if (description == NULL || *description == '\0')
-		description = display_name;
+	description = e_attachment_dup_description (attachment);
+	if (description == NULL || *description == '\0') {
+		g_free (description);
+		description = g_strdup (display_name);
+	}
 
 	if (size > 0)
 		caption = g_strdup_printf (
@@ -245,7 +253,10 @@ attachment_update_file_info_columns (EAttachment *attachment)
 
 	g_free (content_desc);
 	g_free (display_size);
+	g_free (description);
 	g_free (caption);
+
+	g_clear_object (&file_info);
 }
 
 static void
@@ -271,7 +282,7 @@ attachment_update_icon_column (EAttachment *attachment)
 	gtk_tree_path_free (path);
 
 	cancellable = attachment->priv->cancellable;
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 
 	if (file_info != NULL) {
 		icon = g_file_info_get_icon (file_info);
@@ -374,6 +385,8 @@ attachment_update_icon_column (EAttachment *attachment)
 		g_object_unref (attachment->priv->icon);
 	attachment->priv->icon = icon;
 	g_object_notify (G_OBJECT (attachment), "icon");
+
+	g_clear_object (&file_info);
 }
 
 static void
@@ -587,23 +600,23 @@ attachment_get_property (GObject *object,
 			return;
 
 		case PROP_FILE:
-			g_value_set_object (
+			g_value_take_object (
 				value,
-				e_attachment_get_file (
+				e_attachment_ref_file (
 				E_ATTACHMENT (object)));
 			return;
 
 		case PROP_FILE_INFO:
-			g_value_set_object (
+			g_value_take_object (
 				value,
-				e_attachment_get_file_info (
+				e_attachment_ref_file_info (
 				E_ATTACHMENT (object)));
 			return;
 
 		case PROP_ICON:
-			g_value_set_object (
+			g_value_take_object (
 				value,
-				e_attachment_get_icon (
+				e_attachment_ref_icon (
 				E_ATTACHMENT (object)));
 			return;
 
@@ -622,9 +635,9 @@ attachment_get_property (GObject *object,
 			return;
 
 		case PROP_MIME_PART:
-			g_value_set_object (
+			g_value_take_object (
 				value,
-				e_attachment_get_mime_part (
+				e_attachment_ref_mime_part (
 				E_ATTACHMENT (object)));
 			return;
 
@@ -693,6 +706,8 @@ attachment_finalize (GObject *object)
 
 	priv = E_ATTACHMENT_GET_PRIVATE (object);
 
+	g_mutex_clear (&priv->property_lock);
+
 	g_free (priv->disposition);
 
 	/* Chain up to parent's finalize() method. */
@@ -865,6 +880,8 @@ e_attachment_init (EAttachment *attachment)
 	attachment->priv->encrypted = CAMEL_CIPHER_VALIDITY_ENCRYPT_NONE;
 	attachment->priv->signed_ = CAMEL_CIPHER_VALIDITY_SIGN_NONE;
 
+	g_mutex_init (&attachment->priv->property_lock);
+
 	g_signal_connect (
 		attachment, "notify::encrypted",
 		G_CALLBACK (attachment_update_icon_column), NULL);
@@ -1005,7 +1022,7 @@ e_attachment_add_to_multipart (EAttachment *attachment,
 	g_return_if_fail (CAMEL_IS_MULTIPART (multipart));
 
 	/* Still loading?  Too bad. */
-	mime_part = e_attachment_get_mime_part (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 	if (mime_part == NULL)
 		return;
 
@@ -1076,6 +1093,8 @@ e_attachment_add_to_multipart (EAttachment *attachment,
 
 exit:
 	camel_multipart_add_part (multipart, mime_part);
+
+	g_clear_object (&mime_part);
 }
 
 void
@@ -1113,24 +1132,55 @@ e_attachment_get_disposition (EAttachment *attachment)
 	return attachment->priv->disposition;
 }
 
+gchar *
+e_attachment_dup_disposition (EAttachment *attachment)
+{
+	const gchar *protected;
+	gchar *duplicate;
+
+	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
+
+	g_mutex_lock (&attachment->priv->property_lock);
+
+	protected = e_attachment_get_disposition (attachment);
+	duplicate = g_strdup (protected);
+
+	g_mutex_unlock (&attachment->priv->property_lock);
+
+	return duplicate;
+}
+
 void
 e_attachment_set_disposition (EAttachment *attachment,
                               const gchar *disposition)
 {
 	g_return_if_fail (E_IS_ATTACHMENT (attachment));
 
+	g_mutex_lock (&attachment->priv->property_lock);
+
 	g_free (attachment->priv->disposition);
 	attachment->priv->disposition = g_strdup (disposition);
 
+	g_mutex_unlock (&attachment->priv->property_lock);
+
 	g_object_notify (G_OBJECT (attachment), "disposition");
 }
 
 GFile *
-e_attachment_get_file (EAttachment *attachment)
+e_attachment_ref_file (EAttachment *attachment)
 {
+	GFile *file = NULL;
+
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	return attachment->priv->file;
+	g_mutex_lock (&attachment->priv->property_lock);
+
+	if (attachment->priv->file != NULL)
+		file = g_object_ref (attachment->priv->file);
+
+	g_mutex_unlock (&attachment->priv->property_lock);
+
+	return file;
 }
 
 void
@@ -1144,20 +1194,31 @@ e_attachment_set_file (EAttachment *attachment,
 		g_object_ref (file);
 	}
 
-	if (attachment->priv->file != NULL)
-		g_object_unref (attachment->priv->file);
+	g_mutex_lock (&attachment->priv->property_lock);
 
+	g_clear_object (&attachment->priv->file);
 	attachment->priv->file = file;
 
+	g_mutex_unlock (&attachment->priv->property_lock);
+
 	g_object_notify (G_OBJECT (attachment), "file");
 }
 
 GFileInfo *
-e_attachment_get_file_info (EAttachment *attachment)
+e_attachment_ref_file_info (EAttachment *attachment)
 {
+	GFileInfo *file_info = NULL;
+
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	return attachment->priv->file_info;
+	g_mutex_lock (&attachment->priv->property_lock);
+
+	if (attachment->priv->file_info != NULL)
+		file_info = g_object_ref (attachment->priv->file_info);
+
+	g_mutex_unlock (&attachment->priv->property_lock);
+
+	return file_info;
 }
 
 void
@@ -1195,7 +1256,7 @@ e_attachment_set_file_info (EAttachment *attachment,
 }
 
 /**
- * e_attachment_get_mime_type:
+ * e_attachment_dup_mime_type:
  * @attachment: an #EAttachment
  *
  * Returns the MIME type of @attachment according to its #GFileInfo.
@@ -1205,37 +1266,44 @@ e_attachment_set_file_info (EAttachment *attachment,
  * Returns: a newly-allocated MIME type string, or %NULL
  **/
 gchar *
-e_attachment_get_mime_type (EAttachment *attachment)
+e_attachment_dup_mime_type (EAttachment *attachment)
 {
 	GFileInfo *file_info;
-	const gchar *content_type;
-	gchar *mime_type;
+	const gchar *content_type = NULL;
+	gchar *mime_type = NULL;
 
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	file_info = e_attachment_get_file_info (attachment);
-	if (file_info == NULL)
-		return NULL;
+	file_info = e_attachment_ref_file_info (attachment);
+	if (file_info != NULL)
+		content_type = g_file_info_get_content_type (file_info);
 
-	content_type = g_file_info_get_content_type (file_info);
-	if (content_type == NULL)
-		return NULL;
+	if (content_type != NULL)
+		mime_type = g_content_type_get_mime_type (content_type);
 
-	mime_type = g_content_type_get_mime_type (content_type);
-	if (!mime_type)
-		return NULL;
+	if (mime_type != NULL)
+		camel_strdown (mime_type);
 
-	camel_strdown (mime_type);
+	g_clear_object (&file_info);
 
 	return mime_type;
 }
 
 GIcon *
-e_attachment_get_icon (EAttachment *attachment)
+e_attachment_ref_icon (EAttachment *attachment)
 {
+	GIcon *icon = NULL;
+
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	return attachment->priv->icon;
+	g_mutex_lock (&attachment->priv->property_lock);
+
+	if (attachment->priv->icon != NULL)
+		icon = g_object_ref (attachment->priv->icon);
+
+	g_mutex_unlock (&attachment->priv->property_lock);
+
+	return icon;
 }
 
 gboolean
@@ -1247,11 +1315,20 @@ e_attachment_get_loading (EAttachment *attachment)
 }
 
 CamelMimePart *
-e_attachment_get_mime_part (EAttachment *attachment)
+e_attachment_ref_mime_part (EAttachment *attachment)
 {
+	CamelMimePart *mime_part = NULL;
+
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	return attachment->priv->mime_part;
+	g_mutex_lock (&attachment->priv->property_lock);
+
+	if (attachment->priv->mime_part != NULL)
+		mime_part = g_object_ref (attachment->priv->mime_part);
+
+	g_mutex_unlock (&attachment->priv->property_lock);
+
+	return mime_part;
 }
 
 void
@@ -1265,11 +1342,13 @@ e_attachment_set_mime_part (EAttachment *attachment,
 		g_object_ref (mime_part);
 	}
 
-	if (attachment->priv->mime_part != NULL)
-		g_object_unref (attachment->priv->mime_part);
+	g_mutex_lock (&attachment->priv->property_lock);
 
+	g_clear_object (&attachment->priv->mime_part);
 	attachment->priv->mime_part = mime_part;
 
+	g_mutex_unlock (&attachment->priv->property_lock);
+
 	g_object_notify (G_OBJECT (attachment), "mime-part");
 }
 
@@ -1373,38 +1452,50 @@ e_attachment_set_signed (EAttachment *attachment,
 	g_object_notify (G_OBJECT (attachment), "signed");
 }
 
-const gchar *
-e_attachment_get_description (EAttachment *attachment)
+gchar *
+e_attachment_dup_description (EAttachment *attachment)
 {
 	GFileInfo *file_info;
 	const gchar *attribute;
+	const gchar *protected;
+	gchar *duplicate;
 
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	attribute = G_FILE_ATTRIBUTE_STANDARD_DESCRIPTION;
-	file_info = e_attachment_get_file_info (attachment);
-
+	file_info = e_attachment_ref_file_info (attachment);
 	if (file_info == NULL)
 		return NULL;
 
-	return g_file_info_get_attribute_string (file_info, attribute);
+	attribute = G_FILE_ATTRIBUTE_STANDARD_DESCRIPTION;
+	protected = g_file_info_get_attribute_string (file_info, attribute);
+	duplicate = g_strdup (protected);
+
+	g_object_unref (file_info);
+
+	return duplicate;
 }
 
-const gchar *
-e_attachment_get_thumbnail_path (EAttachment *attachment)
+gchar *
+e_attachment_dup_thumbnail_path (EAttachment *attachment)
 {
 	GFileInfo *file_info;
 	const gchar *attribute;
+	const gchar *protected;
+	gchar *duplicate;
 
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	attribute = G_FILE_ATTRIBUTE_THUMBNAIL_PATH;
-	file_info = e_attachment_get_file_info (attachment);
-
+	file_info = e_attachment_ref_file_info (attachment);
 	if (file_info == NULL)
 		return NULL;
 
-	return g_file_info_get_attribute_byte_string (file_info, attribute);
+	attribute = G_FILE_ATTRIBUTE_THUMBNAIL_PATH;
+	protected = g_file_info_get_attribute_string (file_info, attribute);
+	duplicate = g_strdup (protected);
+
+	g_object_unref (file_info);
+
+	return duplicate;
 }
 
 gboolean
@@ -1415,7 +1506,7 @@ e_attachment_is_rfc822 (EAttachment *attachment)
 
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), FALSE);
 
-	mime_type = e_attachment_get_mime_type (attachment);
+	mime_type = e_attachment_dup_mime_type (attachment);
 	is_rfc822 =
 		(mime_type != NULL) &&
 		(g_ascii_strcasecmp (mime_type, "message/rfc822") == 0);
@@ -1437,7 +1528,7 @@ e_attachment_list_apps (EAttachment *attachment)
 
 	g_return_val_if_fail (E_IS_ATTACHMENT (attachment), NULL);
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 	if (file_info == NULL)
 		return NULL;
 
@@ -1460,6 +1551,8 @@ e_attachment_list_apps (EAttachment *attachment)
 	g_free (allocated);
 
 exit:
+	g_clear_object (&file_info);
+
 	return app_info_list;
 }
 
@@ -1798,7 +1891,7 @@ attachment_load_from_mime_part_thread (GSimpleAsyncResult *simple,
 	g_object_set_data (G_OBJECT (simple), ATTACHMENT_LOAD_CONTEXT, NULL);
 
 	attachment = load_context->attachment;
-	mime_part = e_attachment_get_mime_part (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 
 	file_info = g_file_info_new ();
 	load_context->file_info = file_info;
@@ -1902,6 +1995,8 @@ attachment_load_from_mime_part_thread (GSimpleAsyncResult *simple,
 	g_simple_async_result_set_op_res_gpointer (
 		simple, load_context,
 		(GDestroyNotify) attachment_load_context_free);
+
+	g_clear_object (&mime_part);
 }
 
 void
@@ -1932,8 +2027,8 @@ e_attachment_load_async (EAttachment *attachment,
 		return;
 	}
 
-	file = e_attachment_get_file (attachment);
-	mime_part = e_attachment_get_mime_part (attachment);
+	file = e_attachment_ref_file (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 	g_return_if_fail (file != NULL || mime_part != NULL);
 
 	load_context = attachment_load_context_new (
@@ -1960,6 +2055,9 @@ e_attachment_load_async (EAttachment *attachment,
 			G_PRIORITY_DEFAULT,
 			cancellable);
 	}
+
+	g_clear_object (&file);
+	g_clear_object (&mime_part);
 }
 
 gboolean
@@ -2033,7 +2131,7 @@ e_attachment_load_handle_error (EAttachment *attachment,
 		return;
 	}
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 
 	if (file_info != NULL)
 		display_name = g_file_info_get_display_name (file_info);
@@ -2047,6 +2145,8 @@ e_attachment_load_handle_error (EAttachment *attachment,
 		primary_text = g_strdup_printf (
 			_("Could not load the attachment"));
 
+	g_clear_object (&file_info);
+
 	dialog = gtk_message_dialog_new_with_markup (
 		parent, GTK_DIALOG_DESTROY_WITH_PARENT,
 		GTK_MESSAGE_ERROR, GTK_BUTTONS_OK,
@@ -2274,8 +2374,8 @@ e_attachment_open_async (EAttachment *attachment,
 
 	g_return_if_fail (E_IS_ATTACHMENT (attachment));
 
-	file = e_attachment_get_file (attachment);
-	mime_part = e_attachment_get_mime_part (attachment);
+	file = e_attachment_ref_file (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 	g_return_if_fail (file != NULL || mime_part != NULL);
 
 	open_context = attachment_open_context_new (
@@ -2292,6 +2392,9 @@ e_attachment_open_async (EAttachment *attachment,
 
 	} else if (mime_part != NULL)
 		attachment_open_save_temporary (open_context);
+
+	g_clear_object (&file);
+	g_clear_object (&mime_part);
 }
 
 gboolean
@@ -2334,7 +2437,7 @@ e_attachment_open_handle_error (EAttachment *attachment,
 	if (g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
 		return;
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 
 	if (file_info != NULL)
 		display_name = g_file_info_get_display_name (file_info);
@@ -2348,6 +2451,8 @@ e_attachment_open_handle_error (EAttachment *attachment,
 		primary_text = g_strdup_printf (
 			_("Could not open the attachment"));
 
+	g_clear_object (&file_info);
+
 	dialog = gtk_message_dialog_new_with_markup (
 		parent, GTK_DIALOG_DESTROY_WITH_PARENT,
 		GTK_MESSAGE_ERROR, GTK_BUTTONS_OK,
@@ -2480,7 +2585,7 @@ attachment_save_new_candidate (SaveContext *save_context)
 	gchar *basename;
 
 	attachment = save_context->attachment;
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 
 	if (file_info != NULL)
 		display_name = g_file_info_get_display_name (file_info);
@@ -2516,6 +2621,8 @@ attachment_save_new_candidate (SaveContext *save_context)
 
 	g_free (basename);
 
+	g_clear_object (&file_info);
+
 	return candidate;
 }
 
@@ -2630,7 +2737,7 @@ attachment_save_got_output_stream (SaveContext *save_context)
 
 	attachment = save_context->attachment;
 	cancellable = attachment->priv->cancellable;
-	mime_part = e_attachment_get_mime_part (attachment);
+	mime_part = e_attachment_ref_mime_part (attachment);
 
 	/* Decode the MIME part to an in-memory buffer.  We have to do
 	 * this because CamelStream is synchronous-only, and using threads
@@ -2661,6 +2768,8 @@ attachment_save_got_output_stream (SaveContext *save_context)
 		G_PRIORITY_DEFAULT, cancellable,
 		(GAsyncReadyCallback) attachment_save_read_cb,
 		save_context);
+
+	g_clear_object (&mime_part);
 }
 
 static void
@@ -2800,7 +2909,8 @@ e_attachment_save_async (EAttachment *attachment,
 		return;
 	}
 
-	if (e_attachment_get_mime_part (attachment) == NULL) {
+	/* Just peek, don't reference. */
+	if (attachment->priv->mime_part == NULL) {
 		g_simple_async_report_error_in_idle (
 			G_OBJECT (attachment), callback, user_data,
 			G_IO_ERROR, G_IO_ERROR_INVALID_ARGUMENT,
@@ -2871,7 +2981,7 @@ e_attachment_save_handle_error (EAttachment *attachment,
 	if (g_error_matches (error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
 		return;
 
-	file_info = e_attachment_get_file_info (attachment);
+	file_info = e_attachment_ref_file_info (attachment);
 
 	if (file_info != NULL)
 		display_name = g_file_info_get_display_name (file_info);
@@ -2885,6 +2995,8 @@ e_attachment_save_handle_error (EAttachment *attachment,
 		primary_text = g_strdup_printf (
 			_("Could not save the attachment"));
 
+	g_clear_object (&file_info);
+
 	dialog = gtk_message_dialog_new_with_markup (
 		parent, GTK_DIALOG_DESTROY_WITH_PARENT,
 		GTK_MESSAGE_ERROR, GTK_BUTTONS_OK,
diff --git a/e-util/e-attachment.h b/e-util/e-attachment.h
index 0da2c9d883..b21a0f2899 100644
--- a/e-util/e-attachment.h
+++ b/e-util/e-attachment.h
@@ -76,18 +76,19 @@ gboolean	e_attachment_get_can_show	(EAttachment *attachment);
 void		e_attachment_set_can_show	(EAttachment *attachment,
 						 gboolean can_show);
 const gchar *	e_attachment_get_disposition	(EAttachment *attachment);
+gchar *		e_attachment_dup_disposition	(EAttachment *attachment);
 void		e_attachment_set_disposition	(EAttachment *attachment,
 						 const gchar *disposition);
-GFile *		e_attachment_get_file		(EAttachment *attachment);
+GFile *		e_attachment_ref_file		(EAttachment *attachment);
 void		e_attachment_set_file		(EAttachment *attachment,
 						 GFile *file);
-GFileInfo *	e_attachment_get_file_info	(EAttachment *attachment);
+GFileInfo *	e_attachment_ref_file_info	(EAttachment *attachment);
 void		e_attachment_set_file_info	(EAttachment *attachment,
 						 GFileInfo *file_info);
-gchar *		e_attachment_get_mime_type	(EAttachment *attachment);
-GIcon *		e_attachment_get_icon		(EAttachment *attachment);
+gchar *		e_attachment_dup_mime_type	(EAttachment *attachment);
+GIcon *		e_attachment_ref_icon		(EAttachment *attachment);
 gboolean	e_attachment_get_loading	(EAttachment *attachment);
-CamelMimePart *	e_attachment_get_mime_part	(EAttachment *attachment);
+CamelMimePart *	e_attachment_ref_mime_part	(EAttachment *attachment);
 void		e_attachment_set_mime_part	(EAttachment *attachment,
 						 CamelMimePart *mime_part);
 gint		e_attachment_get_percent	(EAttachment *attachment);
@@ -107,8 +108,8 @@ camel_cipher_validity_sign_t
 		e_attachment_get_signed		(EAttachment *attachment);
 void		e_attachment_set_signed		(EAttachment *attachment,
 						 camel_cipher_validity_sign_t signed_);
-const gchar *	e_attachment_get_description	(EAttachment *attachment);
-const gchar *	e_attachment_get_thumbnail_path	(EAttachment *attachment);
+gchar *		e_attachment_dup_description	(EAttachment *attachment);
+gchar *		e_attachment_dup_thumbnail_path	(EAttachment *attachment);
 gboolean	e_attachment_is_rfc822		(EAttachment *attachment);
 GList *		e_attachment_list_apps		(EAttachment *attachment);