dovecot/core-2.2: src/lib-storage/index/dbox-single/sdbox-sync.c annotate

annotate src/lib-storage/index/dbox-single/sdbox-sync.c @ 12943:edf79127fccd

sdbox: Commit expunges to index first before unlinking the files.

author	Timo Sirainen <tss@iki.fi>
date	Tue, 12 Apr 2011 18:38:00 +0300
parents	654b5a2fc926
children	80eef14e9e15

rev	line source
12822 44d0474a451e Updated copyright notices to include year 2011. Timo Sirainen <tss@iki.fi> parents: 12332 diff changeset	1 /* Copyright (c) 2007-2011 Dovecot authors, see the included COPYING file */
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	2
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	3 #include "lib.h"
12312 28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API. Timo Sirainen <tss@iki.fi> parents: 11820 diff changeset	4 #include "dbox-attachment.h"
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	5 #include "sdbox-storage.h"
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	6 #include "sdbox-file.h"
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	7 #include "sdbox-sync.h"
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	8
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	9 #define SDBOX_REBUILD_COUNT 3
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	10
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	11 static void
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	12 dbox_sync_file_move_if_needed(struct dbox_file *file,
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	13 enum sdbox_sync_entry_type type)
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	14 {
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	15 bool move_to_alt = type == SDBOX_SYNC_ENTRY_TYPE_MOVE_TO_ALT;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	16 bool deleted;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	17
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	18 if (move_to_alt != dbox_file_is_in_alt(file)) {
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	19 /* move the file. if it fails, nothing broke so
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	20 don't worry about it. */
12332 9e400f6963eb dbox: Reverted the "check if altpath is mounted" change for now. Timo Sirainen <tss@iki.fi> parents: 12327 diff changeset	21 if (dbox_file_open(file, &deleted) > 0 && !deleted)
11764 8ec35eb6d7b3 dbox: dbox_file_move() was used by only single-dbox, so moved it there. Timo Sirainen <tss@iki.fi> parents: 11658 diff changeset	22 (void)sdbox_file_move(file, move_to_alt);
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	23 }
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	24 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	25
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	26 static void sdbox_sync_file(struct sdbox_sync_context *ctx,
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	27 uint32_t seq, uint32_t uid,
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	28 enum sdbox_sync_entry_type type)
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	29 {
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	30 struct dbox_file *file;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	31
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	32 switch (type) {
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	33 case SDBOX_SYNC_ENTRY_TYPE_EXPUNGE:
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	34 if (!mail_index_transaction_is_expunged(ctx->trans, seq)) {
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	35 mail_index_expunge(ctx->trans, seq);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	36 array_append(&ctx->expunged_uids, &uid, 1);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	37 }
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	38 break;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	39 case SDBOX_SYNC_ENTRY_TYPE_MOVE_FROM_ALT:
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	40 case SDBOX_SYNC_ENTRY_TYPE_MOVE_TO_ALT:
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	41 file = sdbox_file_init(ctx->mbox, uid);
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	42 dbox_sync_file_move_if_needed(file, type);
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	43 dbox_file_unref(&file);
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	44 break;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	45 }
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	46 }
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	47
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	48 static void sdbox_sync_add(struct sdbox_sync_context *ctx,
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	49 const struct mail_index_sync_rec *sync_rec)
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	50 {
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	51 uint32_t uid;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	52 enum sdbox_sync_entry_type type;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	53 uint32_t seq, seq1, seq2;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	54
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	55 if (sync_rec->type == MAIL_INDEX_SYNC_TYPE_EXPUNGE) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	56 /* we're interested */
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	57 type = SDBOX_SYNC_ENTRY_TYPE_EXPUNGE;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	58 } else if (sync_rec->type == MAIL_INDEX_SYNC_TYPE_FLAGS) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	59 /* we care only about alt flag changes */
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	60 if ((sync_rec->add_flags & DBOX_INDEX_FLAG_ALT) != 0)
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	61 type = SDBOX_SYNC_ENTRY_TYPE_MOVE_TO_ALT;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	62 else if ((sync_rec->remove_flags & DBOX_INDEX_FLAG_ALT) != 0)
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	63 type = SDBOX_SYNC_ENTRY_TYPE_MOVE_FROM_ALT;
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	64 else
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	65 return;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	66 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	67 /* not interested */
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	68 return;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	69 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	70
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	71 if (!mail_index_lookup_seq_range(ctx->sync_view,
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	72 sync_rec->uid1, sync_rec->uid2,
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	73 &seq1, &seq2)) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	74 /* already expunged everything. nothing to do. */
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	75 return;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	76 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	77
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	78 for (seq = seq1; seq <= seq2; seq++) {
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	79 mail_index_lookup_uid(ctx->sync_view, seq, &uid);
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	80 sdbox_sync_file(ctx, seq, uid, type);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	81 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	82 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	83
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	84 static int sdbox_sync_index(struct sdbox_sync_context *ctx)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	85 {
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	86 struct mailbox *box = &ctx->mbox->box;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	87 const struct mail_index_header *hdr;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	88 struct mail_index_sync_rec sync_rec;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	89 uint32_t seq1, seq2;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	90
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	91 hdr = mail_index_get_header(ctx->sync_view);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	92 if (hdr->uid_validity == 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	93 /* newly created index file */
12925 654b5a2fc926 sdbox: Added more error message logging. Timo Sirainen <tss@iki.fi> parents: 12822 diff changeset	94 mail_storage_set_critical(box->storage,
654b5a2fc926 sdbox: Added more error message logging. Timo Sirainen <tss@iki.fi> parents: 12822 diff changeset	95 "sdbox %s: Broken index: missing UIDVALIDITY",
654b5a2fc926 sdbox: Added more error message logging. Timo Sirainen <tss@iki.fi> parents: 12822 diff changeset	96 box->path);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	97 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	98 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	99
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	100 /* mark the newly seen messages as recent */
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	101 if (mail_index_lookup_seq_range(ctx->sync_view, hdr->first_recent_uid,
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	102 hdr->next_uid, &seq1, &seq2))
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	103 index_mailbox_set_recent_seq(box, ctx->sync_view, seq1, seq2);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	104
11657 4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	105 while (mail_index_sync_next(ctx->index_sync_ctx, &sync_rec))
4e5032891954 single-dbox: Simplified syncing code. Timo Sirainen <tss@iki.fi> parents: 11176 diff changeset	106 sdbox_sync_add(ctx, &sync_rec);
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	107 return 1;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	108 }
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	109
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	110 static void dbox_sync_file_expunge(struct sdbox_sync_context *ctx,
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	111 uint32_t uid)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	112 {
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	113 struct mailbox *box = &ctx->mbox->box;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	114 struct dbox_file *file;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	115 struct sdbox_file *sfile;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	116 int ret;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	117
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	118 file = sdbox_file_init(ctx->mbox, uid);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	119 sfile = (struct sdbox_file *)file;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	120 if (file->storage->attachment_dir != NULL)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	121 ret = sdbox_file_unlink_with_attachments(sfile);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	122 else
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	123 ret = dbox_file_unlink(file);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	124
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	125 /* do sync_notify only when the file was unlinked by us */
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	126 if (ret > 0 && box->v.sync_notify != NULL)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	127 box->v.sync_notify(box, uid, MAILBOX_SYNC_TYPE_EXPUNGE);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	128 dbox_file_unref(&file);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	129 }
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	130
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	131 static void dbox_sync_expunge_files(struct sdbox_sync_context *ctx)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	132 {
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	133 const uint32_t *uidp;
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	134
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	135 /* NOTE: Index is no longer locked. Multiple processes may be unlinking
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	136 the files at the same time. */
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	137 array_foreach(&ctx->expunged_uids, uidp)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	138 dbox_sync_file_expunge(ctx, *uidp);
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	139 if (ctx->mbox->box.v.sync_notify != NULL)
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	140 ctx->mbox->box.v.sync_notify(&ctx->mbox->box, 0, 0);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	141 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	142
11024 d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors. Timo Sirainen <tss@iki.fi> parents: 10678 diff changeset	143 static int
d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors. Timo Sirainen <tss@iki.fi> parents: 10678 diff changeset	144 sdbox_refresh_header(struct sdbox_mailbox *mbox, bool retry, bool log_error)
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	145 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	146 struct mail_index_view *view;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	147 struct sdbox_index_header hdr;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	148 int ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	149
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	150 view = mail_index_view_open(mbox->box.index);
11024 d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors. Timo Sirainen <tss@iki.fi> parents: 10678 diff changeset	151 ret = sdbox_read_header(mbox, &hdr, log_error);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	152 mail_index_view_close(&view);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	153
11025 1e118fd825c7 sdbox: Removed unused code. Timo Sirainen <tss@iki.fi> parents: 11024 diff changeset	154 if (ret < 0 && retry) {
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	155 (void)mail_index_refresh(mbox->box.index);
11024 d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors. Timo Sirainen <tss@iki.fi> parents: 10678 diff changeset	156 return sdbox_refresh_header(mbox, FALSE, log_error);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	157 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	158 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	159 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	160
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	161 int sdbox_sync_begin(struct sdbox_mailbox *mbox, enum sdbox_sync_flags flags,
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	162 struct sdbox_sync_context **ctx_r)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	163 {
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	164 struct mail_storage *storage = mbox->box.storage;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	165 struct sdbox_sync_context *ctx;
10678 9740c4858a57 Mailbox deletion: Fixed race condition where a mailbox couldn't get deleted. Timo Sirainen <tss@iki.fi> parents: 10660 diff changeset	166 enum mail_index_sync_flags sync_flags;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	167 unsigned int i;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	168 int ret;
11658 9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	169 bool rebuild, force_rebuild;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	170
11658 9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	171 force_rebuild = (flags & SDBOX_SYNC_FLAG_FORCE_REBUILD) != 0;
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	172 rebuild = force_rebuild \|\|
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	173 mbox->corrupted_rebuild_count != 0 \|\|
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	174 sdbox_refresh_header(mbox, TRUE, FALSE) < 0;
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	175
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	176 ctx = i_new(struct sdbox_sync_context, 1);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	177 ctx->mbox = mbox;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	178 ctx->flags = flags;
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	179 i_array_init(&ctx->expunged_uids, 32);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	180
10678 9740c4858a57 Mailbox deletion: Fixed race condition where a mailbox couldn't get deleted. Timo Sirainen <tss@iki.fi> parents: 10660 diff changeset	181 sync_flags = index_storage_get_sync_flags(&mbox->box);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	182 if (!rebuild && (flags & SDBOX_SYNC_FLAG_FORCE) == 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	183 sync_flags \|= MAIL_INDEX_SYNC_FLAG_REQUIRE_CHANGES;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	184 if ((flags & SDBOX_SYNC_FLAG_FSYNC) != 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	185 sync_flags \|= MAIL_INDEX_SYNC_FLAG_FSYNC;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	186 /* don't write unnecessary dirty flag updates */
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	187 sync_flags \|= MAIL_INDEX_SYNC_FLAG_AVOID_FLAG_UPDATES;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	188
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	189 for (i = 0;; i++) {
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	190 ret = mail_index_sync_begin(mbox->box.index,
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	191 &ctx->index_sync_ctx,
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	192 &ctx->sync_view, &ctx->trans,
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	193 sync_flags);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	194 if (ret <= 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	195 if (ret < 0)
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	196 mail_storage_set_index_error(&mbox->box);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	197 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	198 *ctx_r = NULL;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	199 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	200 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	201
11658 9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	202 if (rebuild)
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	203 ret = 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	204 else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	205 if ((ret = sdbox_sync_index(ctx)) > 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	206 break;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	207 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	208
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	209 /* failure. keep the index locked while we're doing a
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	210 rebuild. */
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	211 if (ret == 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	212 if (i >= SDBOX_REBUILD_COUNT) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	213 mail_storage_set_critical(storage,
11820 76ee1fa16012 Renamed "dbox" to "sdbox". Keep also dbox as an alias for it. Timo Sirainen <tss@iki.fi> parents: 11764 diff changeset	214 "sdbox %s: Index keeps breaking",
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	215 ctx->mbox->box.path);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	216 ret = -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	217 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	218 /* do a full resync and try again. */
11820 76ee1fa16012 Renamed "dbox" to "sdbox". Keep also dbox as an alias for it. Timo Sirainen <tss@iki.fi> parents: 11764 diff changeset	219 i_warning("sdbox %s: Rebuilding index",
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	220 ctx->mbox->box.path);
11658 9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	221 rebuild = FALSE;
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	222 ret = sdbox_sync_index_rebuild(mbox,
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling. Timo Sirainen <tss@iki.fi> parents: 11657 diff changeset	223 force_rebuild);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	224 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	225 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	226 mail_index_sync_rollback(&ctx->index_sync_ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	227 if (ret < 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	228 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	229 return -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	230 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	231 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	232
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	233 *ctx_r = ctx;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	234 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	235 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	236
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	237 int sdbox_sync_finish(struct sdbox_sync_context **_ctx, bool success)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	238 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	239 struct sdbox_sync_context ctx = _ctx;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	240 int ret = success ? 0 : -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	241
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	242 *_ctx = NULL;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	243
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	244 if (success) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	245 if (mail_index_sync_commit(&ctx->index_sync_ctx) < 0) {
10660 56b1d4dd9c7d lib-storage: _mailboxes don't descend from index_mailbox anymore, it's now a context. Timo Sirainen <tss@iki.fi>* parents: 10657 diff changeset	246 mail_storage_set_index_error(&ctx->mbox->box);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	247 ret = -1;
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	248 } else {
edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	249 dbox_sync_expunge_files(ctx);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	250 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	251 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	252 mail_index_sync_rollback(&ctx->index_sync_ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	253 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	254
12943 edf79127fccd sdbox: Commit expunges to index first before unlinking the files. Timo Sirainen <tss@iki.fi> parents: 12925 diff changeset	255 array_free(&ctx->expunged_uids);
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	256 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	257 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	258 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	259
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	260 int sdbox_sync(struct sdbox_mailbox *mbox, enum sdbox_sync_flags flags)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	261 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	262 struct sdbox_sync_context *sync_ctx;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	263
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	264 if (sdbox_sync_begin(mbox, flags, &sync_ctx) < 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	265 return -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	266
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	267 if (sync_ctx == NULL)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	268 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	269 return sdbox_sync_finish(&sync_ctx, TRUE);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	270 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	271
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	272 struct mailbox_sync_context *
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	273 sdbox_storage_sync_init(struct mailbox *box, enum mailbox_sync_flags flags)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	274 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	275 struct sdbox_mailbox mbox = (struct sdbox_mailbox )box;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	276 enum sdbox_sync_flags sdbox_sync_flags = 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	277 int ret = 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	278
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	279 if (!box->opened) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	280 if (mailbox_open(box) < 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	281 ret = -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	282 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	283
11025 1e118fd825c7 sdbox: Removed unused code. Timo Sirainen <tss@iki.fi> parents: 11024 diff changeset	284 if (ret == 0 && index_mailbox_want_full_sync(&mbox->box, flags)) {
9977 0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	285 if ((flags & MAILBOX_SYNC_FLAG_FORCE_RESYNC) != 0)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	286 sdbox_sync_flags \|= SDBOX_SYNC_FLAG_FORCE_REBUILD;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	287 ret = sdbox_sync(mbox, sdbox_sync_flags);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	288 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	289
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	290 return index_mailbox_sync_init(box, flags, ret < 0);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends. Timo Sirainen <tss@iki.fi> parents: diff changeset	291 }

Mercurial > dovecot > core-2.2

annotate src/lib-storage/index/dbox-single/sdbox-sync.c @ 12943:edf79127fccd