From: Timo Sirainen <tss@iki.fi>
Date: Mon, 10 Aug 2009 17:17:21 +0000 (-0400)
Subject: maildir: Don't assert-crash if we're changing UIDs without saving mails.
X-Git-Tag: 2.0.alpha1~294
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ee2b11b42fe529dc6855b9178d78ad79bd6c7401;p=thirdparty%2Fdovecot%2Fcore.git

maildir: Don't assert-crash if we're changing UIDs without saving mails.

--HG--
branch : HEAD
---

diff --git a/src/lib-storage/index/maildir/maildir-save.c b/src/lib-storage/index/maildir/maildir-save.c
index f3454b2e71..89471dc8f7 100644
--- a/src/lib-storage/index/maildir/maildir-save.c
+++ b/src/lib-storage/index/maildir/maildir-save.c
@@ -712,14 +712,16 @@ maildir_save_set_recent_flags(struct maildir_save_context *ctx)
 	unsigned int i, count;
 	uint32_t uid;
 
+	uids = array_get(&saved_sorted_uids, &count);
+	if (count == 0)
+		return 0;
+
 	t_array_init(&saved_sorted_uids,
 		     array_count(&ctx->ctx.transaction->changes->saved_uids));
 	array_append_array(&saved_sorted_uids,
 			   &ctx->ctx.transaction->changes->saved_uids);
 	array_sort(&saved_sorted_uids, seq_range_cmp);
 
-	uids = array_get(&saved_sorted_uids, &count);
-	i_assert(count > 0);
 	for (i = 0; i < count; i++) {
 		for (uid = uids[i].seq1; uid <= uids[i].seq2; uid++)
 			index_mailbox_set_recent_uid(&mbox->ibox, uid);