# HG changeset patch
# User Timo Sirainen <timo.sirainen@dovecot.fi>
# Date 1522011686 -10800
# Node ID 59c3c804bffae1bf1b6d447938153ba3516b3dca
# Parent  c905428ab98825f88a50a82d2cb60337a5b01f4b
imapc: Ignore fetch-msn-workarounds/no-msn-updates during initial FETCH

Otherwise duplicate UIDs aren't detected and it causes an assert-crash:

Panic: file imapc-mailbox.c: line 533 (imapc_untagged_fetch): assertion failed: (lseq == mbox->sync_next_lseq)

diff -r c905428ab988 -r 59c3c804bffa src/lib-storage/index/imapc/imapc-mailbox.c
--- a/src/lib-storage/index/imapc/imapc-mailbox.c	Wed Mar 21 11:29:26 2018 +0200
+++ b/src/lib-storage/index/imapc/imapc-mailbox.c	Mon Mar 26 00:01:26 2018 +0300
@@ -357,11 +357,14 @@
 
 	msgmap = imapc_client_mailbox_get_msgmap(mbox->client_box);
 	msg_count = imapc_msgmap_count(msgmap);
-	if (fetch_uid != 0 &&
+	if (fetch_uid != 0 && mbox->state_fetched_success &&
 	    (IMAPC_BOX_HAS_FEATURE(mbox, IMAPC_FEATURE_FETCH_MSN_WORKAROUNDS) ||
 	     IMAPC_BOX_HAS_FEATURE(mbox, IMAPC_FEATURE_NO_MSN_UPDATES))) {
 		/* if we know the UID, use own own generated rseq instead of
-		   the potentially broken rseq that the server sent. */
+		   the potentially broken rseq that the server sent.
+		   Skip this during the initial FETCH 1:* (UID ..) handling,
+		   or we can't detect duplicate UIDs and will instead
+		   assert-crash later on. */
 		uint32_t fixed_rseq;
 
 		if (imapc_msgmap_uid_to_rseq(msgmap, fetch_uid, &fixed_rseq))