From: Timo Sirainen <tss@iki.fi>
Date: Fri, 27 Mar 2015 13:12:50 +0000 (+0200)
Subject: imapc: Don't assert-crash when trying to sync a mailbox with MAILBOX_FLAG_SAVEONLY.
X-Git-Tag: 2.2.17.rc1~232
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=603cfcf51ee9a6956d9c1fd4649aea88aefdf0bb;p=thirdparty%2Fdovecot%2Fcore.git

imapc: Don't assert-crash when trying to sync a mailbox with MAILBOX_FLAG_SAVEONLY.
We can just skip the sync, because we don't do anything except save mails.
---

diff --git a/src/lib-storage/index/imapc/imapc-sync.c b/src/lib-storage/index/imapc/imapc-sync.c
index 63b69f7e87..c26c79ea75 100644
--- a/src/lib-storage/index/imapc/imapc-sync.c
+++ b/src/lib-storage/index/imapc/imapc-sync.c
@@ -488,6 +488,11 @@ static int imapc_sync(struct imapc_mailbox *mbox)
 	struct imapc_sync_context *sync_ctx;
 	bool force = mbox->sync_fetch_first_uid != 0;
 
+	if ((mbox->box.flags & MAILBOX_FLAG_SAVEONLY) != 0) {
+		/* we're only saving mails here - no syncing actually wanted */
+		return 0;
+	}
+
 	if (imapc_sync_begin(mbox, &sync_ctx, force) < 0)
 		return -1;
 	if (sync_ctx == NULL)