Mercurial > dovecot > original-hg > dovecot-1.2

changeset 2702:d6f25d0e104a HEAD

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Don't complain about corruption when there is none.
author Timo Sirainen <tss@iki.fi>
date Fri, 08 Oct 2004 00:19:05 +0300
parents 1e7b3a1db07f
children 28693bf81196
files src/lib-index/mail-transaction-log-view.c
diffstat 1 files changed, 2 insertions(+), 28 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-index/mail-transaction-log-view.c	Fri Oct 08 00:11:19 2004 +0300
+++ b/src/lib-index/mail-transaction-log-view.c	Fri Oct 08 00:19:05 2004 +0300
@@ -22,8 +22,6 @@
 	uint32_t prev_file_seq;
 	uoff_t prev_file_offset;
 
-	uint32_t max_ext_id;
-
 	unsigned int broken:1;
 };
 
@@ -239,7 +237,7 @@
 static int
 log_view_get_next(struct mail_transaction_log_view *view,
 		  const struct mail_transaction_header **hdr_r,
-		  const void **data_r, enum mail_transaction_type type_mask)
+		  const void **data_r)
 {
 	const struct mail_transaction_header *hdr;
 	struct mail_transaction_log_file *file;
@@ -344,8 +342,6 @@
 			}
 
 			intro = CONST_PTR_OFFSET(data, i);
-			if (intro->ext_id > view->max_ext_id)
-				view->max_ext_id = intro->ext_id;
 			if (intro->name_size >
 			    hdr_size - sizeof(*hdr) - sizeof(*intro)) {
 				mail_transaction_log_file_set_corrupted(file,
@@ -355,27 +351,6 @@
 
 			i += sizeof(*intro) + intro->name_size;
 		}
-	} else if (hdr->type == MAIL_TRANSACTION_EXT_REC_UPDATE ||
-		   hdr->type == MAIL_TRANSACTION_EXT_HDR_UPDATE ||
-		   hdr->type == MAIL_TRANSACTION_EXT_RESET) {
-		const uint32_t *ext_id = data;
-		uint32_t max_ext_id;
-
-		max_ext_id = view->log->index->map->extensions == NULL ? 0 :
-			(view->log->index->map->extensions->used /
-			 sizeof(struct mail_index_ext));
-		if (view->max_ext_id > max_ext_id)
-			max_ext_id = view->max_ext_id;
-
-		/* don't check this error if we're just skipping over this.
-		   we could have skipped over the ext_intro just before this
-		   which is the reason we don't yet know it. */
-		if (*ext_id >= max_ext_id && (type_mask & hdr->type) != 0) {
-			mail_transaction_log_file_set_corrupted(file,
-				"extension update out of range (%u >= %u)",
-				*ext_id, max_ext_id);
-			return -1;
-		}
 	}
 
 	*hdr_r = hdr;
@@ -397,8 +372,7 @@
 	if (view->broken)
 		return -1;
 
-	while ((ret = log_view_get_next(view, &hdr, &data,
-					view->type_mask)) > 0) {
+	while ((ret = log_view_get_next(view, &hdr, &data)) > 0) {
 		if ((view->type_mask & hdr->type) != 0)
 			break;