Mercurial > dovecot > core-2.2

changeset 10748:a792d411e73e HEAD

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
lib-storage: Fixed error handling in mailbox_copy().
author Timo Sirainen <tss@iki.fi>
date Thu, 18 Feb 2010 07:32:13 +0200
parents c0c4898fc803
children 9c188cfba679
files src/lib-storage/mail-copy.c
diffstat 1 files changed, 16 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-storage/mail-copy.c	Thu Feb 18 07:31:55 2010 +0200
+++ b/src/lib-storage/mail-copy.c	Thu Feb 18 07:32:13 2010 +0200
@@ -5,7 +5,8 @@
 #include "mail-storage-private.h"
 #include "mail-copy.h"
 
-int mail_storage_copy(struct mail_save_context *ctx, struct mail *mail)
+static int
+mail_storage_try_copy(struct mail_save_context *ctx, struct mail *mail)
 {
 	struct mail_private *pmail = (struct mail_private *)mail;
 	struct istream *input;
@@ -48,18 +49,26 @@
 			break;
 	} while (i_stream_read(input) != -1);
 
+	if (input->stream_errno != 0) {
+		mail_storage_set_critical(ctx->transaction->box->storage,
+					  "copy: i_stream_read() failed: %m");
+		return -1;
+	}
+	return 0;
+}
+
+int mail_storage_copy(struct mail_save_context *ctx, struct mail *mail)
+{
 	if (ctx->keywords != NULL) {
-		/* keywords gets unreferenced twice, because we call
-		   mailbox_save_cancel/finish */
+		/* keywords gets unreferenced twice: first in
+		   mailbox_save_cancel()/_finish() and second time in
+		   mailbox_copy(). */
 		mailbox_keywords_ref(ctx->transaction->box, ctx->keywords);
 	}
 
-	if (input->stream_errno != 0) {
-		mail_storage_set_critical(ctx->transaction->box->storage,
-					  "copy: i_stream_read() failed: %m");
+	if (mail_storage_try_copy(ctx, mail) < 0) {
 		mailbox_save_cancel(&ctx);
 		return -1;
 	}
-
 	return mailbox_save_finish(&ctx);
 }