[dovecot-cvs] dovecot/src/lib-storage/index/mbox mbox-sync.c, 1.50, 1.51

[cras at procontrol.fi]

Update of /home/cvs/dovecot/src/lib-storage/index/mbox
In directory talvi:/tmp/cvs-serv23823/lib-storage/index/mbox

Modified Files:
	mbox-sync.c 
Log Message:
Complain about reappearing UIDs, but reassign new UIDs to them anyway. Don't
try to update \recent mails if we aren't really going to make them
non-recent.



Index: mbox-sync.c
===================================================================
RCS file: /home/cvs/dovecot/src/lib-storage/index/mbox/mbox-sync.c,v
retrieving revision 1.50
retrieving revision 1.51
diff -u -d -r1.50 -r1.51
--- mbox-sync.c	20 Jun 2004 20:46:11 -0000	1.50
+++ mbox-sync.c	23 Jun 2004 03:31:52 -0000	1.51
@@ -223,8 +223,10 @@
 	/* save the offset permanently with recent flag state */
 	mail_ctx->mail.from_offset = mail_ctx->from_offset;
 	if ((mail_ctx->mail.flags & MBOX_NONRECENT) == 0) {
-		/* need to add 'O' flag to Status-header */
-		mail_ctx->need_rewrite = TRUE;
+		if (!sync_ctx->ibox->keep_recent) {
+			/* need to add 'O' flag to Status-header */
+			mail_ctx->need_rewrite = TRUE;
+		}
 		// FIXME: save it somewhere
 	}
 	return 1;
@@ -340,13 +342,21 @@
 		rec = NULL;
 	}
 
-	if (rec != NULL && rec->uid != uid) {
+	if (ret == 0 && uid < sync_ctx->hdr->next_uid) {
+		/* this UID was already in index and it was expunged */
+		mail_storage_set_critical(sync_ctx->ibox->box.storage,
+			"mbox sync: Expunged message reappeared in mailbox %s "
+			"(UID %u < %u)", sync_ctx->ibox->path, uid,
+			sync_ctx->hdr->next_uid);
+		ret = 0;
+	} else if (rec != NULL && rec->uid != uid) {
 		/* new UID in the middle of the mailbox - shouldn't happen */
 		mail_storage_set_critical(sync_ctx->ibox->box.storage,
 			"mbox sync: UID inserted in the middle of mailbox %s "
 			"(%u > %u)", sync_ctx->ibox->path, rec->uid, uid);
-		mail_index_mark_corrupted(sync_ctx->ibox->index);
-		return -1;
+		ret = 0;
+	} else {
+		ret = 1;
 	}
 
 	*rec_r = rec;
@@ -747,12 +757,8 @@
 			ret = mbox_sync_read_index_rec(sync_ctx, uid, &rec);
 			if (ret < 0)
 				return -1;
-			if (ret == 0 && uid < sync_ctx->hdr->next_uid) {
-				/* this UID was already in index and it was
-				   expunged */
+			if (ret == 0)
 				uid = 0;
-				rec = NULL;
-			}
 		}
 		if (uid == 0) {
 			/* missing/broken X-UID. all the rest of the mails