annotate src/lib-storage/index/dbox-single/sdbox-sync.c @ 12925:654b5a2fc926

sdbox: Added more error message logging.
author Timo Sirainen <tss@iki.fi>
date Tue, 05 Apr 2011 22:43:54 +0300
parents 44d0474a451e
children edf79127fccd
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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 dbox_sync_file_expunge(struct sdbox_sync_context *ctx,
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
27 struct dbox_file *file, uint32_t seq)
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
28 {
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
29 struct sdbox_file *sfile = (struct sdbox_file *)file;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
30 struct mailbox *box = &ctx->mbox->box;
12312
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
31 int ret;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
32
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
33 if (mail_index_transaction_is_expunged(ctx->trans, seq)) {
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
34 /* already expunged within this transaction */
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
35 return;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
36 }
12312
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
37
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
38 if (file->storage->attachment_dir != NULL)
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
39 ret = sdbox_file_unlink_with_attachments(sfile);
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
40 else
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
41 ret = dbox_file_unlink(file);
28eaaa23f2c6 lib-storage: Added support for saving mail attachments separately via filesystem API.
Timo Sirainen <tss@iki.fi>
parents: 11820
diff changeset
42 if (ret < 0) {
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
43 /* some non-ENOENT error trying to unlink the file */
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
44 return;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
45 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
46
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
47 /* file was either unlinked by us or someone else */
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
48 mail_index_expunge(ctx->trans, seq);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
49 if (box->v.sync_notify != NULL)
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
50 box->v.sync_notify(box, sfile->uid, MAILBOX_SYNC_TYPE_EXPUNGE);
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
51 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
52
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
53 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
54 uint32_t seq, uint32_t uid,
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
55 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
56 {
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
57 struct dbox_file *file;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
58
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
59 file = sdbox_file_init(ctx->mbox, uid);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
60 switch (type) {
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
61 case SDBOX_SYNC_ENTRY_TYPE_EXPUNGE:
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
62 dbox_sync_file_expunge(ctx, file, seq);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
63 break;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
64 case SDBOX_SYNC_ENTRY_TYPE_MOVE_FROM_ALT:
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
65 case SDBOX_SYNC_ENTRY_TYPE_MOVE_TO_ALT:
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
66 dbox_sync_file_move_if_needed(file, type);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
67 break;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
68 }
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
69 dbox_file_unref(&file);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
70 }
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
71
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
72 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
73 const struct mail_index_sync_rec *sync_rec)
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
74 {
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
75 uint32_t uid;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
76 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
77 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
78
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
79 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
80 /* we're interested */
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
81 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
82 } 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
83 /* we care only about alt flag changes */
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
84 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
85 type = SDBOX_SYNC_ENTRY_TYPE_MOVE_TO_ALT;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
86 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
87 type = SDBOX_SYNC_ENTRY_TYPE_MOVE_FROM_ALT;
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
88 else
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
89 return;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
90 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
91 /* not interested */
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
92 return;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
93 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
94
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
95 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
96 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
97 &seq1, &seq2)) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
98 /* already expunged everything. nothing to do. */
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
99 return;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
100 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
101
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
102 for (seq = seq1; seq <= seq2; seq++) {
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
103 mail_index_lookup_uid(ctx->sync_view, seq, &uid);
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
104 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
105 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
106 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
107
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
108 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
109 {
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
110 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
111 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
112 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
113 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
114
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
115 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
116 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
117 /* newly created index file */
12925
654b5a2fc926 sdbox: Added more error message logging.
Timo Sirainen <tss@iki.fi>
parents: 12822
diff changeset
118 mail_storage_set_critical(box->storage,
654b5a2fc926 sdbox: Added more error message logging.
Timo Sirainen <tss@iki.fi>
parents: 12822
diff changeset
119 "sdbox %s: Broken index: missing UIDVALIDITY",
654b5a2fc926 sdbox: Added more error message logging.
Timo Sirainen <tss@iki.fi>
parents: 12822
diff changeset
120 box->path);
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
121 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
122 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
123
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
124 /* 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
125 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
126 hdr->next_uid, &seq1, &seq2))
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
127 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
128
11657
4e5032891954 single-dbox: Simplified syncing code.
Timo Sirainen <tss@iki.fi>
parents: 11176
diff changeset
129 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
130 sdbox_sync_add(ctx, &sync_rec);
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
131
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
132 if (box->v.sync_notify != NULL)
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
133 box->v.sync_notify(box, 0, 0);
11658
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
134 return 1;
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
135 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
136
11024
d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors.
Timo Sirainen <tss@iki.fi>
parents: 10678
diff changeset
137 static int
d16bfa082e88 sdbox: Avoid logging multiple adjacent "Invalid dbox header size" errors.
Timo Sirainen <tss@iki.fi>
parents: 10678
diff changeset
138 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
139 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
140 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
141 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
142 int ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
143
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
144 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
145 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
146 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
147
11025
1e118fd825c7 sdbox: Removed unused code.
Timo Sirainen <tss@iki.fi>
parents: 11024
diff changeset
148 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
149 (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
150 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
151 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
152 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
153 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
154
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
155 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
156 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
157 {
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
158 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
159 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
160 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
161 unsigned int i;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
162 int ret;
11658
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
163 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
164
11658
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
165 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
166 rebuild = force_rebuild ||
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
167 mbox->corrupted_rebuild_count != 0 ||
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
168 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
169
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
170 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
171 ctx->mbox = mbox;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
172 ctx->flags = flags;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
173
10678
9740c4858a57 Mailbox deletion: Fixed race condition where a mailbox couldn't get deleted.
Timo Sirainen <tss@iki.fi>
parents: 10660
diff changeset
174 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
175 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
176 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
177 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
178 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
179 /* 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
180 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
181
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
182 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
183 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
184 &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
185 &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
186 sync_flags);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
187 if (ret <= 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
188 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
189 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
190 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
191 *ctx_r = NULL;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
192 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
193 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
194
11658
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
195 if (rebuild)
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
196 ret = 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
197 else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
198 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
199 break;
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
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
202 /* 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
203 rebuild. */
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
204 if (ret == 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
205 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
206 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
207 "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
208 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
209 ret = -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
210 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
211 /* 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
212 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
213 ctx->mbox->box.path);
11658
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
214 rebuild = FALSE;
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
215 ret = sdbox_sync_index_rebuild(mbox,
9d9be6aa3a55 single-dbox: Fixed broken mailbox handling.
Timo Sirainen <tss@iki.fi>
parents: 11657
diff changeset
216 force_rebuild);
9977
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
217 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
218 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
219 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
220 if (ret < 0) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
221 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
222 return -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
223 }
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 *ctx_r = ctx;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
227 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
228 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
229
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
230 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
231 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
232 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
233 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
234
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
235 *_ctx = NULL;
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 if (success) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
238 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
239 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
240 ret = -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 } else {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
243 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
244 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
245
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
246 i_free(ctx);
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
247 return ret;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
248 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
249
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
250 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
251 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
252 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
253
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
254 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
255 return -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
256
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
257 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
258 return 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
259 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
260 }
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 mailbox_sync_context *
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
263 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
264 {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
265 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
266 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
267 int ret = 0;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
268
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
269 if (!box->opened) {
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
270 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
271 ret = -1;
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
272 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
273
11025
1e118fd825c7 sdbox: Removed unused code.
Timo Sirainen <tss@iki.fi>
parents: 11024
diff changeset
274 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
275 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
276 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
277 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
278 }
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
279
0bb321c347ae Split dbox (single-dbox) and mdbox (multi-dbox) into separate storage backends.
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
280 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
281 }