Mercurial > dovecot > core-2.2
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 |
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 | 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 | 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 } |