changeset 21659:0f6a1fee4c7f

imapc: Error handling wasn't setting storage error everywhere. This would have used the previous error, which might not have been correct.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Tue, 21 Feb 2017 15:44:09 +0200
parents bd1d7007cb01
children ed928407244b
files src/lib-storage/index/imapc/imapc-mail-fetch.c
diffstat 1 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-storage/index/imapc/imapc-mail-fetch.c	Mon Feb 20 15:47:33 2017 +0200
+++ b/src/lib-storage/index/imapc/imapc-mail-fetch.c	Tue Feb 21 15:44:09 2017 +0200
@@ -213,8 +213,10 @@
 	uint32_t seq;
 	unsigned int i;
 
-	if (_mail->lookup_abort != MAIL_LOOKUP_ABORT_NEVER)
+	if (_mail->lookup_abort != MAIL_LOOKUP_ABORT_NEVER) {
+		mail_set_aborted(_mail);
 		return -1;
+	}
 	_mail->mail_stream_opened = TRUE;
 
 	/* drop any fields that we may already be fetching currently */
@@ -444,8 +446,10 @@
 		!imail->header_list_fetched)) {
 		imapc_mailbox_run_nofetch(mbox);
 	}
-	if (imail->fetch_failed)
+	if (imail->fetch_failed) {
+		mail_storage_set_internal_error(&mbox->storage->storage);
 		return -1;
+	}
 	return 0;
 }