dovecot/core-2.2: src/lib-index/mail-index.c annotate

annotate src/lib-index/mail-index.c @ 22361:73479718bc0b

lib-index: Mark index deleted if index log file is missing This fixes all kinds of errors caused by mailbox being deleted by another process.

author	Aki Tuomi <aki.tuomi@dovecot.fi>
date	Tue, 18 Jul 2017 14:37:53 +0300
parents	2e2563132d5f
children	5e665ab8487c

rev	line source
21390 2e2563132d5f Updated copyright notices to include the year 2017. Stephan Bosch <stephan.bosch@dovecot.fi> parents: 21389 diff changeset	1 /* Copyright (c) 2003-2017 Dovecot authors, see the included COPYING file */
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	2
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	3 #include "lib.h"
5819 4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	4 #include "ioloop.h"
3470 346a494c2feb Moved array declaration to array-decl.h and include it in lib.h. So array.h Timo Sirainen <tss@iki.fi> parents: 3445 diff changeset	5 #include "array.h"
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	6 #include "buffer.h"
9535 bbe4c24faf21 Use eacces_get_error() in more places for handling EACCES errors. Timo Sirainen <tss@iki.fi> parents: 9421 diff changeset	7 #include "eacces-error.h"
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	8 #include "hash.h"
6384 2c8b1d487728 Make sure the extension name doesn't contain control characters. It most Timo Sirainen <tss@iki.fi> parents: 6378 diff changeset	9 #include "str-sanitize.h"
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	10 #include "mmap-util.h"
4071 cd3d26cf124a Renamed safe-open.* to nfs-workarounds., safe_open() to nfs_safe_open() and Timo Sirainen <tss@iki.fi>* parents: 4067 diff changeset	11 #include "nfs-workarounds.h"
1917 68938dccbc45 Forced locking to be right with mprotect()ing index file. Support for Timo Sirainen <tss@iki.fi> parents: 1915 diff changeset	12 #include "read-full.h"
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	13 #include "write-full.h"
10656 9d3b5cbef222 Moved mail_index lookup cache from lib-storage to lib-index. Timo Sirainen <tss@iki.fi> parents: 10655 diff changeset	14 #include "mail-index-alloc-cache.h"
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	15 #include "mail-index-private.h"
5272 82b4a055e109 mmap_disable=yes crashes with recent changes. Timo Sirainen <tss@iki.fi> parents: 5247 diff changeset	16 #include "mail-index-view-private.h"
2885 1880f191278a Avoid re-reading index file with mmap_disabled=yes. Instead sync it by Timo Sirainen <tss@iki.fi> parents: 2884 diff changeset	17 #include "mail-index-sync-private.h"
7620 4b8c1c164d8f Initial CONDSTORE support. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	18 #include "mail-index-modseq.h"
21205 5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	19 #include "mail-transaction-log-private.h"
21184 e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	20 #include "mail-transaction-log-view-private.h"
2200 97bb7b127617 Beginnings of getting cache file working again. Easy to break currently, but Timo Sirainen <tss@iki.fi> parents: 2193 diff changeset	21 #include "mail-cache.h"
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	22
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	23 #include <stdio.h>
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	24 #include <stddef.h>
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	25 #include <time.h>
1917 68938dccbc45 Forced locking to be right with mprotect()ing index file. Support for Timo Sirainen <tss@iki.fi> parents: 1915 diff changeset	26 #include <sys/stat.h>
68938dccbc45 Forced locking to be right with mprotect()ing index file. Support for Timo Sirainen <tss@iki.fi> parents: 1915 diff changeset	27
5453 a0a1432bb4e8 Better type safety to module_contexts arrays. Already fixed some bugs. Timo Sirainen <tss@iki.fi> parents: 5430 diff changeset	28 struct mail_index_module_register mail_index_module_register = { 0 };
4453 85fcdb478608 Beginnings of joining mail-storage API more closely to mail-index, so that Timo Sirainen <timo.sirainen@movial.fi> parents: 4451 diff changeset	29
17579 092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	30 static void mail_index_close_nonopened(struct mail_index *index);
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	31
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	32 struct mail_index mail_index_alloc(const char dir, const char *prefix)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	33 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	34 struct mail_index *index;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	35
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	36 index = i_new(struct mail_index, 1);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	37 index->dir = i_strdup(dir);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	38 index->prefix = i_strdup(prefix);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	39 index->fd = -1;
2116 fbb4da13ee3b Fixes for extra_records Timo Sirainen <tss@iki.fi> parents: 2115 diff changeset	40
5028 774c965ac79a Memory pool size changes Timo Sirainen <tss@iki.fi> parents: 5024 diff changeset	41 index->extension_pool =
774c965ac79a Memory pool size changes Timo Sirainen <tss@iki.fi> parents: 5024 diff changeset	42 pool_alloconly_create(MEMPOOL_GROWING"index extension", 1024);
4596 bf4e98a0de3f Replaced ARRAY_CREATE() macro with [ipt]_array_init() macros. The macro Timo Sirainen <tss@iki.fi> parents: 4594 diff changeset	43 p_array_init(&index->extensions, index->extension_pool, 5);
bf4e98a0de3f Replaced ARRAY_CREATE() macro with [ipt]_array_init() macros. The macro Timo Sirainen <tss@iki.fi> parents: 4594 diff changeset	44 i_array_init(&index->sync_lost_handlers, 4);
5453 a0a1432bb4e8 Better type safety to module_contexts arrays. Already fixed some bugs. Timo Sirainen <tss@iki.fi> parents: 5430 diff changeset	45 i_array_init(&index->module_contexts,
a0a1432bb4e8 Better type safety to module_contexts arrays. Already fixed some bugs. Timo Sirainen <tss@iki.fi> parents: 5430 diff changeset	46 I_MIN(5, mail_index_module_register.id));
4453 85fcdb478608 Beginnings of joining mail-storage API more closely to mail-index, so that Timo Sirainen <timo.sirainen@movial.fi> parents: 4451 diff changeset	47
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	48 index->mode = 0600;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	49 index->gid = (gid_t)-1;
11623 c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	50 index->lock_method = FILE_LOCK_METHOD_FCNTL;
15904 d3cf06639864 Replaced all -1U and (unsigned int)-1 with UINT_MAX. Timo Sirainen <tss@iki.fi> parents: 15715 diff changeset	51 index->max_lock_timeout_secs = UINT_MAX;
3016 61c8d205d887 Initial support for keywords. Syncing to mbox/maildir doesn't work yet. Timo Sirainen <tss@iki.fi> parents: 2956 diff changeset	52
21205 5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	53 index->log_rotate_min_size =
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	54 MAIL_TRANSACTION_LOG_ROTATE_DEFAULT_MIN_SIZE;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	55 index->log_rotate_max_size =
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	56 MAIL_TRANSACTION_LOG_ROTATE_DEFAULT_MAX_SIZE;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	57 index->log_rotate_min_created_ago_secs =
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	58 MAIL_TRANSACTION_LOG_ROTATE_DEFAULT_TIME;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	59 index->log_rotate_log2_stale_secs =
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	60 MAIL_TRANSACTION_LOG2_DEFAULT_STALE_SECS;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	61
3016 61c8d205d887 Initial support for keywords. Syncing to mbox/maildir doesn't work yet. Timo Sirainen <tss@iki.fi> parents: 2956 diff changeset	62 index->keywords_ext_id =
8547 545cb188f7ab Replaced literal "keywords" strings with a macro. Timo Sirainen <tss@iki.fi> parents: 8146 diff changeset	63 mail_index_ext_register(index, MAIL_INDEX_EXT_KEYWORDS,
545cb188f7ab Replaced literal "keywords" strings with a macro. Timo Sirainen <tss@iki.fi> parents: 8146 diff changeset	64 128, 2, 1);
3086 66c44404f9a1 Keyword fixes. Timo Sirainen <tss@iki.fi> parents: 3016 diff changeset	65 index->keywords_pool = pool_alloconly_create("keywords", 512);
4596 bf4e98a0de3f Replaced ARRAY_CREATE() macro with [ipt]_array_init() macros. The macro Timo Sirainen <tss@iki.fi> parents: 4594 diff changeset	66 i_array_init(&index->keywords, 16);
14918 8eae4e205c82 Hash table API is now (mostly) type safe. Timo Sirainen <tss@iki.fi> parents: 14917 diff changeset	67 hash_table_create(&index->keywords_hash, index->keywords_pool, 0,
8eae4e205c82 Hash table API is now (mostly) type safe. Timo Sirainen <tss@iki.fi> parents: 14917 diff changeset	68 strcase_hash, strcasecmp);
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	69 index->log = mail_transaction_log_alloc(index);
7620 4b8c1c164d8f Initial CONDSTORE support. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	70 mail_index_modseq_init(index);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	71 return index;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	72 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	73
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	74 void mail_index_free(struct mail_index **_index)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	75 {
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	76 struct mail_index index = _index;
928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	77
928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	78 *_index = NULL;
10677 93fe3aa23bdb lib-index: Don't close index files if they're still being used by another mailbox. Timo Sirainen <tss@iki.fi> parents: 10674 diff changeset	79
93fe3aa23bdb lib-index: Don't close index files if they're still being used by another mailbox. Timo Sirainen <tss@iki.fi> parents: 10674 diff changeset	80 i_assert(index->open_count == 0);
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	81
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	82 mail_transaction_log_free(&index->log);
8573 f9166a09423a Renamed hash_() to hash_table_() to avoid conflicts with OSX's strhash.h Timo Sirainen <tss@iki.fi> parents: 8572 diff changeset	83 hash_table_destroy(&index->keywords_hash);
6428 7cad076906eb pool_unref() now takes ** pointer. Timo Sirainen <tss@iki.fi> parents: 6417 diff changeset	84 pool_unref(&index->extension_pool);
7cad076906eb pool_unref() now takes ** pointer. Timo Sirainen <tss@iki.fi> parents: 6417 diff changeset	85 pool_unref(&index->keywords_pool);
1967 b6d900688c55 memory leak fixes Timo Sirainen <tss@iki.fi> parents: 1965 diff changeset	86
3191 0df3f5d71958 Changed many buffers to arrays. Cleans up the code a lot. Timo Sirainen <tss@iki.fi> parents: 3179 diff changeset	87 array_free(&index->sync_lost_handlers);
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	88 array_free(&index->keywords);
5453 a0a1432bb4e8 Better type safety to module_contexts arrays. Already fixed some bugs. Timo Sirainen <tss@iki.fi> parents: 5430 diff changeset	89 array_free(&index->module_contexts);
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	90
14079 2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	91 i_free(index->ext_hdr_init_data);
9537 b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	92 i_free(index->gid_origin);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	93 i_free(index->error);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	94 i_free(index->dir);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	95 i_free(index->prefix);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	96 i_free(index);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	97 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	98
11634 7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	99 void mail_index_set_fsync_mode(struct mail_index *index,
7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	100 enum fsync_mode mode,
15063 0efc12f3eb2a lib-index: Changed mail_index_set_fsync_mode() to use a separate enum for the mask. Timo Sirainen <tss@iki.fi> parents: 14923 diff changeset	101 enum mail_index_fsync_mask mask)
6736 3e189cbb7e7a fsync transaction commits only if the transaction contains change types Timo Sirainen <tss@iki.fi> parents: 6686 diff changeset	102 {
11634 7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	103 index->fsync_mode = mode;
7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	104 index->fsync_mask = mask;
6736 3e189cbb7e7a fsync transaction commits only if the transaction contains change types Timo Sirainen <tss@iki.fi> parents: 6686 diff changeset	105 }
3e189cbb7e7a fsync transaction commits only if the transaction contains change types Timo Sirainen <tss@iki.fi> parents: 6686 diff changeset	106
2650 9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	107 void mail_index_set_permissions(struct mail_index *index,
9537 b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	108 mode_t mode, gid_t gid, const char *gid_origin)
2650 9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	109 {
9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	110 index->mode = mode & 0666;
9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	111 index->gid = gid;
9537 b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	112
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	113 i_free(index->gid_origin);
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	114 index->gid_origin = i_strdup(gid_origin);
2650 9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	115 }
9b78674428f5 Get index file mode and GID from dovecot-shared file. Timo Sirainen <tss@iki.fi> parents: 2618 diff changeset	116
11623 c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	117 void mail_index_set_lock_method(struct mail_index *index,
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	118 enum file_lock_method lock_method,
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	119 unsigned int max_timeout_secs)
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	120 {
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	121 index->lock_method = lock_method;
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	122 index->max_lock_timeout_secs = max_timeout_secs;
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	123 }
c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	124
21205 5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	125 void mail_index_set_log_rotation(struct mail_index *index,
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	126 uoff_t min_size, uoff_t max_size,
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	127 unsigned int min_created_ago_secs,
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	128 unsigned int log2_stale_secs)
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	129 {
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	130 index->log_rotate_min_size = min_size;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	131 index->log_rotate_max_size = max_size;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	132 index->log_rotate_min_created_ago_secs = min_created_ago_secs;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	133 index->log_rotate_log2_stale_secs = log2_stale_secs;
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	134 }
5d905618c4b1 lib-index: Add mail_index_set_log_rotation() Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21184 diff changeset	135
14079 2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	136 void mail_index_set_ext_init_data(struct mail_index *index, uint32_t ext_id,
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	137 const void *data, size_t size)
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	138 {
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	139 const struct mail_index_registered_ext *rext;
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	140
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	141 i_assert(index->ext_hdr_init_data == NULL \|\|
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	142 index->ext_hdr_init_id == ext_id);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	143
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	144 rext = array_idx(&index->extensions, ext_id);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	145 i_assert(rext->hdr_size == size);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	146
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	147 index->ext_hdr_init_id = ext_id;
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	148 i_free(index->ext_hdr_init_data);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	149 index->ext_hdr_init_data = i_malloc(size);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	150 memcpy(index->ext_hdr_init_data, data, size);
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	151 }
2ab26bb55346 lib-index: Added mail_index_set_ext_init_data() for adding data to index on creation. Timo Sirainen <tss@iki.fi> parents: 13384 diff changeset	152
2671 3b70ddb51771 Renamed "extra record info" and variations of it to "extension" or "ext" in Timo Sirainen <tss@iki.fi> parents: 2660 diff changeset	153 uint32_t mail_index_ext_register(struct mail_index index, const char name,
2683 d41c42a7bf1e Index extensions can now specify record alignment, and it's possible to Timo Sirainen <tss@iki.fi> parents: 2671 diff changeset	154 uint32_t default_hdr_size,
d41c42a7bf1e Index extensions can now specify record alignment, and it's possible to Timo Sirainen <tss@iki.fi> parents: 2671 diff changeset	155 uint16_t default_record_size,
d41c42a7bf1e Index extensions can now specify record alignment, and it's possible to Timo Sirainen <tss@iki.fi> parents: 2671 diff changeset	156 uint16_t default_record_align)
2114 4aedcb4c51cb Added support for per-index sized mail_index_record. Timo Sirainen <tss@iki.fi> parents: 2095 diff changeset	157 {
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	158 struct mail_index_registered_ext rext;
6679 90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	159 uint32_t ext_id;
2115 f720b3f15333 Modifying extra_records should work now. Timo Sirainen <tss@iki.fi> parents: 2114 diff changeset	160
6871 f7ec019fd3f6 mail_index_ext_register(): Panic if parameters aren't correct. Timo Sirainen <tss@iki.fi> parents: 6857 diff changeset	161 if (*name == '\0' \|\| strcmp(name, str_sanitize(name, -1)) != 0)
6384 2c8b1d487728 Make sure the extension name doesn't contain control characters. It most Timo Sirainen <tss@iki.fi> parents: 6378 diff changeset	162 i_panic("mail_index_ext_register(%s): Invalid name", name);
2c8b1d487728 Make sure the extension name doesn't contain control characters. It most Timo Sirainen <tss@iki.fi> parents: 6378 diff changeset	163
7361 11eb0bd76299 mail_index_ext_register(): Allow registering an extension without record or Timo Sirainen <tss@iki.fi> parents: 7086 diff changeset	164 if (default_record_size != 0 && default_record_align == 0) {
6871 f7ec019fd3f6 mail_index_ext_register(): Panic if parameters aren't correct. Timo Sirainen <tss@iki.fi> parents: 6857 diff changeset	165 i_panic("mail_index_ext_register(%s): "
7361 11eb0bd76299 mail_index_ext_register(): Allow registering an extension without record or Timo Sirainen <tss@iki.fi> parents: 7086 diff changeset	166 "Invalid record alignment", name);
6871 f7ec019fd3f6 mail_index_ext_register(): Panic if parameters aren't correct. Timo Sirainen <tss@iki.fi> parents: 6857 diff changeset	167 }
f7ec019fd3f6 mail_index_ext_register(): Panic if parameters aren't correct. Timo Sirainen <tss@iki.fi> parents: 6857 diff changeset	168
6679 90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	169 if (mail_index_ext_lookup(index, name, &ext_id))
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	170 return ext_id;
2116 fbb4da13ee3b Fixes for extra_records Timo Sirainen <tss@iki.fi> parents: 2115 diff changeset	171
21389 59437f8764c6 global: Replaced all instances of memset(p, 0, sizeof(p)) with the new i_zero() macro. Stephan Bosch <stephan.bosch@dovecot.fi>* parents: 21323 diff changeset	172 i_zero(&rext);
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	173 rext.name = p_strdup(index->extension_pool, name);
6679 90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	174 rext.index_idx = array_count(&index->extensions);
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	175 rext.hdr_size = default_hdr_size;
575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	176 rext.record_size = default_record_size;
575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	177 rext.record_align = default_record_align;
2563 5752e5ac88f0 Save extra record/header infos into index file permanently. Timo Sirainen <tss@iki.fi> parents: 2553 diff changeset	178
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	179 array_append(&index->extensions, &rext, 1);
6679 90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	180 return rext.index_idx;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	181 }
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	182
18849 9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	183 void mail_index_ext_register_resize_defaults(struct mail_index *index,
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	184 uint32_t ext_id,
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	185 uint32_t default_hdr_size,
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	186 uint16_t default_record_size,
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	187 uint16_t default_record_align)
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	188 {
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	189 struct mail_index_registered_ext *rext;
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	190
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	191 rext = array_idx_modifiable(&index->extensions, ext_id);
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	192 rext->hdr_size = default_hdr_size;
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	193 rext->record_size = default_record_size;
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	194 rext->record_align = default_record_align;
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	195 }
9c90eba19aee lib-index: Added mail_index_ext_register_resize_defaults() Timo Sirainen <tss@iki.fi> parents: 18137 diff changeset	196
6679 90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	197 bool mail_index_ext_lookup(struct mail_index index, const char name,
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	198 uint32_t *ext_id_r)
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	199 {
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	200 const struct mail_index_registered_ext *extensions;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	201 unsigned int i, count;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	202
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	203 extensions = array_get(&index->extensions, &count);
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	204 for (i = 0; i < count; i++) {
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	205 if (strcmp(extensions[i].name, name) == 0) {
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	206 *ext_id_r = i;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	207 return TRUE;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	208 }
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	209 }
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	210
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	211 *ext_id_r = (uint32_t)-1;
90196328bcce Added mail_index_ext_lookup() Timo Sirainen <tss@iki.fi> parents: 6429 diff changeset	212 return FALSE;
2563 5752e5ac88f0 Save extra record/header infos into index file permanently. Timo Sirainen <tss@iki.fi> parents: 2553 diff changeset	213 }
5752e5ac88f0 Save extra record/header infos into index file permanently. Timo Sirainen <tss@iki.fi> parents: 2553 diff changeset	214
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	215 void mail_index_register_expunge_handler(struct mail_index *index,
4903 204d7edc7cdc Added context parameter type safety checks for most callback APIs. Timo Sirainen <tss@iki.fi> parents: 4876 diff changeset	216 uint32_t ext_id, bool call_always,
4490 56d17f2e1d66 Added support for calling expunge handler for an extension even if the Timo Sirainen <tss@iki.fi> parents: 4453 diff changeset	217 mail_index_expunge_handler_t *cb,
4903 204d7edc7cdc Added context parameter type safety checks for most callback APIs. Timo Sirainen <tss@iki.fi> parents: 4876 diff changeset	218 void *context)
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	219 {
3817 dc560ef2654b Moved expunge handlers to mail_index_registered_ext. Timo Sirainen <tss@iki.fi> parents: 3816 diff changeset	220 struct mail_index_registered_ext *rext;
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	221
4451 1a35d53c18fc Array API redesigned to work using unions. It now provides type safety Timo Sirainen <tss@iki.fi> parents: 4396 diff changeset	222 rext = array_idx_modifiable(&index->extensions, ext_id);
4616 b0966236fda7 Memory leak fixes Timo Sirainen <tss@iki.fi> parents: 4596 diff changeset	223 i_assert(rext->expunge_handler == NULL \|\| rext->expunge_handler == cb);
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	224
3817 dc560ef2654b Moved expunge handlers to mail_index_registered_ext. Timo Sirainen <tss@iki.fi> parents: 3816 diff changeset	225 rext->expunge_handler = cb;
4490 56d17f2e1d66 Added support for calling expunge handler for an extension even if the Timo Sirainen <tss@iki.fi> parents: 4453 diff changeset	226 rext->expunge_context = context;
56d17f2e1d66 Added support for calling expunge handler for an extension even if the Timo Sirainen <tss@iki.fi> parents: 4453 diff changeset	227 rext->expunge_handler_call_always = call_always;
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	228 }
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	229
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	230 void mail_index_unregister_expunge_handler(struct mail_index *index,
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	231 uint32_t ext_id)
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	232 {
3817 dc560ef2654b Moved expunge handlers to mail_index_registered_ext. Timo Sirainen <tss@iki.fi> parents: 3816 diff changeset	233 struct mail_index_registered_ext *rext;
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	234
4451 1a35d53c18fc Array API redesigned to work using unions. It now provides type safety Timo Sirainen <tss@iki.fi> parents: 4396 diff changeset	235 rext = array_idx_modifiable(&index->extensions, ext_id);
3817 dc560ef2654b Moved expunge handlers to mail_index_registered_ext. Timo Sirainen <tss@iki.fi> parents: 3816 diff changeset	236 i_assert(rext->expunge_handler != NULL);
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	237
3817 dc560ef2654b Moved expunge handlers to mail_index_registered_ext. Timo Sirainen <tss@iki.fi> parents: 3816 diff changeset	238 rext->expunge_handler = NULL;
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	239 }
512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	240
512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	241 void mail_index_register_sync_handler(struct mail_index *index, uint32_t ext_id,
2869 d2daf90c79f1 Cache invalidating needed to hook into view syncing as well. Timo Sirainen <tss@iki.fi> parents: 2857 diff changeset	242 mail_index_sync_handler_t *cb,
d2daf90c79f1 Cache invalidating needed to hook into view syncing as well. Timo Sirainen <tss@iki.fi> parents: 2857 diff changeset	243 enum mail_index_sync_handler_type type)
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	244 {
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	245 struct mail_index_registered_ext *rext;
2869 d2daf90c79f1 Cache invalidating needed to hook into view syncing as well. Timo Sirainen <tss@iki.fi> parents: 2857 diff changeset	246
4451 1a35d53c18fc Array API redesigned to work using unions. It now provides type safety Timo Sirainen <tss@iki.fi> parents: 4396 diff changeset	247 rext = array_idx_modifiable(&index->extensions, ext_id);
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	248 i_assert(rext->sync_handler.callback == NULL);
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	249
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	250 rext->sync_handler.callback = cb;
575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	251 rext->sync_handler.type = type;
2853 512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	252 }
512dd7d76cdc Removed cache_offset from mail_index_record and changed it to use extension Timo Sirainen <tss@iki.fi> parents: 2825 diff changeset	253
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	254 void mail_index_unregister_sync_handler(struct mail_index *index,
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	255 uint32_t ext_id)
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	256 {
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	257 struct mail_index_registered_ext *rext;
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	258
4451 1a35d53c18fc Array API redesigned to work using unions. It now provides type safety Timo Sirainen <tss@iki.fi> parents: 4396 diff changeset	259 rext = array_idx_modifiable(&index->extensions, ext_id);
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	260 i_assert(rext->sync_handler.callback != NULL);
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	261
3816 575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	262 rext->sync_handler.callback = NULL;
575c79a0aabc Cleanup: Created mail_index_registered_ext which is used for Timo Sirainen <tss@iki.fi> parents: 3582 diff changeset	263 rext->sync_handler.type = 0;
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	264 }
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	265
2934 c3ae75597952 Added "sync lost" handlers which are called with mmap_disable=yes when index Timo Sirainen <tss@iki.fi> parents: 2926 diff changeset	266 void mail_index_register_sync_lost_handler(struct mail_index *index,
c3ae75597952 Added "sync lost" handlers which are called with mmap_disable=yes when index Timo Sirainen <tss@iki.fi> parents: 2926 diff changeset	267 mail_index_sync_lost_handler_t *cb)
c3ae75597952 Added "sync lost" handlers which are called with mmap_disable=yes when index Timo Sirainen <tss@iki.fi> parents: 2926 diff changeset	268 {
3191 0df3f5d71958 Changed many buffers to arrays. Cleans up the code a lot. Timo Sirainen <tss@iki.fi> parents: 3179 diff changeset	269 array_append(&index->sync_lost_handlers, &cb, 1);
2934 c3ae75597952 Added "sync lost" handlers which are called with mmap_disable=yes when index Timo Sirainen <tss@iki.fi> parents: 2926 diff changeset	270 }
c3ae75597952 Added "sync lost" handlers which are called with mmap_disable=yes when index Timo Sirainen <tss@iki.fi> parents: 2926 diff changeset	271
3224 f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	272 void mail_index_unregister_sync_lost_handler(struct mail_index *index,
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	273 mail_index_sync_lost_handler_t *cb)
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	274 {
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	275 mail_index_sync_lost_handler_t const handlers;
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	276 unsigned int i, count;
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	277
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	278 handlers = array_get(&index->sync_lost_handlers, &count);
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	279 for (i = 0; i < count; i++) {
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	280 if (handlers[i] == cb) {
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	281 array_delete(&index->sync_lost_handlers, i, 1);
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	282 break;
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	283 }
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	284 }
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	285 }
f6e0d1826645 Added sync/sync_lost/expunge handler unregistering, and also do that. Fixes Timo Sirainen <tss@iki.fi> parents: 3222 diff changeset	286
3863 55df57c028d4 Added "bool" type and changed all ints that were used as booleans to bool. Timo Sirainen <tss@iki.fi> parents: 3833 diff changeset	287 bool mail_index_keyword_lookup(struct mail_index *index,
6335 a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	288 const char keyword, unsigned int idx_r)
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	289 {
14923 96fd2c3bf932 Reverted "support for non-pointers" part of the hash table API changes. Timo Sirainen <tss@iki.fi> parents: 14918 diff changeset	290 char *key;
96fd2c3bf932 Reverted "support for non-pointers" part of the hash table API changes. Timo Sirainen <tss@iki.fi> parents: 14918 diff changeset	291 void *value;
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	292
3818 6758c0f3c318 Comment updates and minor cleanups Timo Sirainen <tss@iki.fi> parents: 3817 diff changeset	293 /* keywords_hash keeps a name => index mapping of keywords.
6758c0f3c318 Comment updates and minor cleanups Timo Sirainen <tss@iki.fi> parents: 3817 diff changeset	294 Keywords are never removed from it, so the index values are valid
6758c0f3c318 Comment updates and minor cleanups Timo Sirainen <tss@iki.fi> parents: 3817 diff changeset	295 for the lifetime of the mail_index. */
8573 f9166a09423a Renamed hash_() to hash_table_() to avoid conflicts with OSX's strhash.h Timo Sirainen <tss@iki.fi> parents: 8572 diff changeset	296 if (hash_table_lookup_full(index->keywords_hash, keyword,
14629 c93ca5e46a8a Marked functions parameters that are allowed to be NULL. Some APIs were also changed. Timo Sirainen <tss@iki.fi> parents: 14231 diff changeset	297 &key, &value)) {
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	298 *idx_r = POINTER_CAST_TO(value, unsigned int);
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	299 return TRUE;
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	300 }
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	301
15904 d3cf06639864 Replaced all -1U and (unsigned int)-1 with UINT_MAX. Timo Sirainen <tss@iki.fi> parents: 15715 diff changeset	302 *idx_r = UINT_MAX;
6335 a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	303 return FALSE;
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	304 }
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	305
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	306 void mail_index_keyword_lookup_or_create(struct mail_index *index,
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	307 const char *keyword,
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	308 unsigned int *idx_r)
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	309 {
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	310 char *keyword_dup;
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	311
6350 e7f61751822e Added assert against empty keyword. Timo Sirainen <tss@iki.fi> parents: 6349 diff changeset	312 i_assert(*keyword != '\0');
e7f61751822e Added assert against empty keyword. Timo Sirainen <tss@iki.fi> parents: 6349 diff changeset	313
6335 a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	314 if (mail_index_keyword_lookup(index, keyword, idx_r))
a1c587d3d633 Removed autocreate parameter from mail_index_keyword_lookup(). Added a new Timo Sirainen <tss@iki.fi> parents: 6330 diff changeset	315 return;
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	316
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	317 keyword = keyword_dup = p_strdup(index->keywords_pool, keyword);
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	318 *idx_r = array_count(&index->keywords);
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	319
14923 96fd2c3bf932 Reverted "support for non-pointers" part of the hash table API changes. Timo Sirainen <tss@iki.fi> parents: 14918 diff changeset	320 hash_table_insert(index->keywords_hash, keyword_dup,
96fd2c3bf932 Reverted "support for non-pointers" part of the hash table API changes. Timo Sirainen <tss@iki.fi> parents: 14918 diff changeset	321 POINTER_CAST(*idx_r));
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	322 array_append(&index->keywords, &keyword, 1);
13306 ba03935cc599 mail_index_get_keywords() / status.keywords weren't actually guaranteed to be NULL terminated Timo Sirainen <tss@iki.fi> parents: 12936 diff changeset	323
ba03935cc599 mail_index_get_keywords() / status.keywords weren't actually guaranteed to be NULL terminated Timo Sirainen <tss@iki.fi> parents: 12936 diff changeset	324 /* keep the array NULL-terminated, but the NULL itself invisible */
14686 9ff19c1d5f69 Added array_append_zero() to write a zero-filled record to an array. Timo Sirainen <tss@iki.fi> parents: 14682 diff changeset	325 array_append_zero(&index->keywords);
13306 ba03935cc599 mail_index_get_keywords() / status.keywords weren't actually guaranteed to be NULL terminated Timo Sirainen <tss@iki.fi> parents: 12936 diff changeset	326 array_delete(&index->keywords, array_count(&index->keywords)-1, 1);
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	327 }
a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	328
4451 1a35d53c18fc Array API redesigned to work using unions. It now provides type safety Timo Sirainen <tss@iki.fi> parents: 4396 diff changeset	329 const ARRAY_TYPE(keywords) mail_index_get_keywords(struct mail_index index)
3016 61c8d205d887 Initial support for keywords. Syncing to mbox/maildir doesn't work yet. Timo Sirainen <tss@iki.fi> parents: 2956 diff changeset	330 {
3254 a2943c050571 Keywords are now stored in X-Keywords headers in mbox. Did several related Timo Sirainen <tss@iki.fi> parents: 3243 diff changeset	331 return &index->keywords;
3016 61c8d205d887 Initial support for keywords. Syncing to mbox/maildir doesn't work yet. Timo Sirainen <tss@iki.fi> parents: 2956 diff changeset	332 }
61c8d205d887 Initial support for keywords. Syncing to mbox/maildir doesn't work yet. Timo Sirainen <tss@iki.fi> parents: 2956 diff changeset	333
9613 ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	334 struct mail_keywords *
9614 d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	335 mail_index_keywords_create(struct mail_index *index,
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	336 const char *const keywords[])
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	337 {
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	338 struct mail_keywords *k;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	339 unsigned int src, dest, i, count;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	340
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	341 count = str_array_length(keywords);
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	342 if (count == 0) {
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	343 k = i_new(struct mail_keywords, 1);
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	344 k->index = index;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	345 k->refcount = 1;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	346 return k;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	347 }
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	348
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	349 /* @UNSAFE */
21323 d223fad9767f global: Make sure _malloc() calculations won't cause integer overflows. Timo Sirainen <timo.sirainen@dovecot.fi>* parents: 21205 diff changeset	350 k = i_malloc(MALLOC_ADD(sizeof(struct mail_keywords),
d223fad9767f global: Make sure _malloc() calculations won't cause integer overflows. Timo Sirainen <timo.sirainen@dovecot.fi>* parents: 21205 diff changeset	351 MALLOC_MULTIPLY(sizeof(k->idx), (count-1))));
9614 d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	352 k->index = index;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	353 k->refcount = 1;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	354
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	355 /* look up the keywords from index. they're never removed from there
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	356 so we can permanently store indexes to them. */
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	357 for (src = dest = 0; src < count; src++) {
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	358 mail_index_keyword_lookup_or_create(index, keywords[src],
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	359 &k->idx[dest]);
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	360 /* ignore if this is a duplicate */
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	361 for (i = 0; i < src; i++) {
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	362 if (k->idx[i] == k->idx[dest])
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	363 break;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	364 }
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	365 if (i == src)
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	366 dest++;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	367 }
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	368 k->count = dest;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	369 return k;
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	370 }
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	371
d670b5d66302 Previous commit accidentally removed mail_index_keywords_create(). Timo Sirainen <tss@iki.fi> parents: 9613 diff changeset	372 struct mail_keywords *
9613 ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	373 mail_index_keywords_create_from_indexes(struct mail_index *index,
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	374 const ARRAY_TYPE(keyword_indexes)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	375 *keyword_indexes)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	376 {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	377 struct mail_keywords *k;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	378 const unsigned int *indexes;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	379 unsigned int src, dest, i, count;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	380
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	381 indexes = array_get(keyword_indexes, &count);
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	382 if (count == 0) {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	383 k = i_new(struct mail_keywords, 1);
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	384 k->index = index;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	385 k->refcount = 1;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	386 return k;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	387 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	388
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	389 /* @UNSAFE */
21323 d223fad9767f global: Make sure _malloc() calculations won't cause integer overflows. Timo Sirainen <timo.sirainen@dovecot.fi>* parents: 21205 diff changeset	390 k = i_malloc(MALLOC_ADD(sizeof(struct mail_keywords),
d223fad9767f global: Make sure _malloc() calculations won't cause integer overflows. Timo Sirainen <timo.sirainen@dovecot.fi>* parents: 21205 diff changeset	391 MALLOC_MULTIPLY(sizeof(k->idx), (count-1))));
9613 ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	392 k->index = index;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	393 k->refcount = 1;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	394
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	395 /* copy but skip duplicates */
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	396 for (src = dest = 0; src < count; src++) {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	397 for (i = 0; i < src; i++) {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	398 if (k->idx[i] == indexes[src])
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	399 break;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	400 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	401 if (i == src)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	402 k->idx[dest++] = indexes[src];
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	403 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	404 k->count = dest;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	405 return k;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	406 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	407
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	408 void mail_index_keywords_ref(struct mail_keywords *keywords)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	409 {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	410 keywords->refcount++;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	411 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	412
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	413 void mail_index_keywords_unref(struct mail_keywords **_keywords)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	414 {
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	415 struct mail_keywords keywords = _keywords;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	416
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	417 i_assert(keywords->refcount > 0);
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	418
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	419 *_keywords = NULL;
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	420 if (--keywords->refcount == 0)
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	421 i_free(keywords);
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	422 }
ed9b4bc28913 Moved mail_index_keywords_() to mail-index.c. Timo Sirainen <tss@iki.fi>* parents: 9537 diff changeset	423
5686 0d2a6a7f2a1b Moved mail index map related code to its own file. Timo Sirainen <tss@iki.fi> parents: 5666 diff changeset	424 int mail_index_try_open_only(struct mail_index *index)
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	425 {
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	426 i_assert(index->fd == -1);
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	427 i_assert(!MAIL_INDEX_IS_IN_MEMORY(index));
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	428
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	429 /* Note that our caller must close index->fd by itself. */
6686 610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	430 if (index->readonly)
610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	431 errno = EACCES;
610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	432 else {
610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	433 index->fd = nfs_safe_open(index->filepath, O_RDWR);
610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	434 index->readonly = FALSE;
610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	435 }
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	436
4023 b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	437 if (index->fd == -1 && errno == EACCES) {
b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	438 index->fd = open(index->filepath, O_RDONLY);
b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	439 index->readonly = TRUE;
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	440 }
4023 b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	441
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	442 if (index->fd == -1) {
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	443 if (errno != ENOENT) {
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	444 mail_index_set_syscall_error(index, "open()");
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	445 return -1;
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	446 }
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	447
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	448 /* have to create it */
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	449 return 0;
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	450 }
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	451 return 1;
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	452 }
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	453
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	454 static int
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	455 mail_index_try_open(struct mail_index *index)
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	456 {
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	457 int ret;
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	458
4023 b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	459 i_assert(index->fd == -1);
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	460
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	461 if (MAIL_INDEX_IS_IN_MEMORY(index))
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	462 return 0;
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	463
5854 eca7e29dce0d Rewrite index lock handling. Timo Sirainen <tss@iki.fi> parents: 5853 diff changeset	464 ret = mail_index_map(index, MAIL_INDEX_SYNC_HANDLER_HEAD);
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	465 if (ret == 0) {
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	466 /* it's corrupted - recreate it */
5820 e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	467 if (index->fd != -1) {
e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	468 if (close(index->fd) < 0)
e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	469 mail_index_set_syscall_error(index, "close()");
e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	470 index->fd = -1;
e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	471 }
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	472 }
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	473 return ret;
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	474 }
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	475
17904 0a34975b2dcc lib-index: Added path parameter to mail_index_create_tmp_file() Timo Sirainen <tss@iki.fi> parents: 17900 diff changeset	476 int mail_index_create_tmp_file(struct mail_index *index,
0a34975b2dcc lib-index: Added path parameter to mail_index_create_tmp_file() Timo Sirainen <tss@iki.fi> parents: 17900 diff changeset	477 const char path_prefix, const char *path_r)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	478 {
2879 aa93c7216722 dovecot-shared file was supposed to show permissions/gid for files created Timo Sirainen <tss@iki.fi> parents: 2877 diff changeset	479 mode_t old_mask;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	480 const char *path;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	481 int fd;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	482
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	483 i_assert(!MAIL_INDEX_IS_IN_MEMORY(index));
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	484
17904 0a34975b2dcc lib-index: Added path parameter to mail_index_create_tmp_file() Timo Sirainen <tss@iki.fi> parents: 17900 diff changeset	485 path = *path_r = t_strconcat(path_prefix, ".tmp", NULL);
2879 aa93c7216722 dovecot-shared file was supposed to show permissions/gid for files created Timo Sirainen <tss@iki.fi> parents: 2877 diff changeset	486 old_mask = umask(0);
12702 7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	487 fd = open(path, O_RDWR\|O_CREAT\|O_EXCL, index->mode);
2879 aa93c7216722 dovecot-shared file was supposed to show permissions/gid for files created Timo Sirainen <tss@iki.fi> parents: 2877 diff changeset	488 umask(old_mask);
12702 7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	489 if (fd == -1 && errno == EEXIST) {
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	490 /* stale temp file. unlink and recreate rather than overwriting,
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	491 just to make sure locking problems won't cause corruption */
19136 fefaa6d09a81 Replaced unlink() calls with i_unlink() wherever possible. Timo Sirainen <tss@iki.fi>* parents: 18849 diff changeset	492 if (i_unlink(path) < 0)
12702 7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	493 return -1;
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	494 old_mask = umask(0);
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	495 fd = open(path, O_RDWR\|O_CREAT\|O_EXCL, index->mode);
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	496 umask(old_mask);
7ecc5e10da57 lib-index: Avoid corrupting dovecot.index file when recreating it even in case of broken file locking. Timo Sirainen <tss@iki.fi> parents: 12249 diff changeset	497 }
10686 442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	498 if (fd == -1) {
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	499 mail_index_file_set_syscall_error(index, path, "creat()");
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	500 return -1;
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	501 }
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	502
9420 c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	503 mail_index_fchown(index, fd, path);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	504 return fd;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	505 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	506
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	507 static int mail_index_open_files(struct mail_index *index,
fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	508 enum mail_index_open_flags flags)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	509 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	510 int ret;
3582 bfa542369fdc When creating a new index file, mark its log offsets so that the possibly Timo Sirainen <tss@iki.fi> parents: 3556 diff changeset	511
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	512 ret = mail_transaction_log_open(index->log);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	513 if (ret == 0) {
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	514 if ((flags & MAIL_INDEX_OPEN_FLAG_CREATE) == 0)
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	515 return 0;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	516
5819 4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	517 /* if dovecot.index exists, read it first so that we can get
4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	518 the correct indexid and log sequence */
4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	519 (void)mail_index_try_open(index);
4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	520
4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	521 if (index->indexid == 0) {
6359 c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	522 /* Create a new indexid for us. If we're opening index
c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	523 into memory, index->map doesn't exist yet. */
5819 4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	524 index->indexid = ioloop_time;
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	525 index->initial_create = TRUE;
6359 c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	526 if (index->map != NULL)
c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	527 index->map->hdr.indexid = index->indexid;
5819 4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	528 }
4ea31bf18a56 Handle indexid changes better. Timo Sirainen <tss@iki.fi> parents: 5775 diff changeset	529
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	530 ret = mail_transaction_log_create(index->log, FALSE);
9950 7aea0156986a Avoid assert-crashing when two processes try to create index at the same time. Timo Sirainen <tss@iki.fi> parents: 9698 diff changeset	531 if (index->map != NULL) {
7aea0156986a Avoid assert-crashing when two processes try to create index at the same time. Timo Sirainen <tss@iki.fi> parents: 9698 diff changeset	532 /* log creation could have changed it if someone else
7aea0156986a Avoid assert-crashing when two processes try to create index at the same time. Timo Sirainen <tss@iki.fi> parents: 9698 diff changeset	533 just created it. */
7aea0156986a Avoid assert-crashing when two processes try to create index at the same time. Timo Sirainen <tss@iki.fi> parents: 9698 diff changeset	534 index->map->hdr.indexid = index->indexid;
7aea0156986a Avoid assert-crashing when two processes try to create index at the same time. Timo Sirainen <tss@iki.fi> parents: 9698 diff changeset	535 }
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	536 index->initial_create = FALSE;
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	537 }
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	538 if (ret >= 0) {
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	539 ret = index->map != NULL ? 1 : mail_index_try_open(index);
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	540 if (ret == 0) {
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	541 /* corrupted */
6322 8d9a156ec94e Handle opening corrupted index files better. Timo Sirainen <tss@iki.fi> parents: 6321 diff changeset	542 mail_transaction_log_close(index->log);
7573 de08cc81da73 Fixes to handling races in initial index creation. Timo Sirainen <tss@iki.fi> parents: 7361 diff changeset	543 ret = mail_transaction_log_create(index->log, TRUE);
6359 c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	544 if (ret == 0) {
c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	545 if (index->map != NULL)
c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	546 mail_index_unmap(&index->map);
6322 8d9a156ec94e Handle opening corrupted index files better. Timo Sirainen <tss@iki.fi> parents: 6321 diff changeset	547 index->map = mail_index_map_alloc(index);
6359 c54c07f30a97 Fixed opening indexes to memory. Timo Sirainen <tss@iki.fi> parents: 6350 diff changeset	548 }
2170 fb785d235c43 minor fix for error handling Timo Sirainen <tss@iki.fi> parents: 2122 diff changeset	549 }
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	550 }
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	551 if (ret < 0) {
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	552 /* open/create failed, fallback to in-memory indexes */
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	553 if ((flags & MAIL_INDEX_OPEN_FLAG_CREATE) == 0)
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	554 return -1;
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	555
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	556 if (mail_index_move_to_memory(index) < 0)
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	557 return -1;
2200 97bb7b127617 Beginnings of getting cache file working again. Easy to break currently, but Timo Sirainen <tss@iki.fi> parents: 2193 diff changeset	558 }
97bb7b127617 Beginnings of getting cache file working again. Easy to break currently, but Timo Sirainen <tss@iki.fi> parents: 2193 diff changeset	559
17900 6aeb2a7e5303 lib-index: Fixed cache file creation race condition. Timo Sirainen <tss@iki.fi> parents: 17579 diff changeset	560 if (index->cache == NULL)
6aeb2a7e5303 lib-index: Fixed cache file creation race condition. Timo Sirainen <tss@iki.fi> parents: 17579 diff changeset	561 index->cache = mail_cache_open_or_create(index);
13384 33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	562 return 1;
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	563 }
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	564
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	565 static int
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	566 mail_index_open_opened(struct mail_index *index,
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	567 enum mail_index_open_flags flags)
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	568 {
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	569 int ret;
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	570
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	571 i_assert(index->map != NULL);
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	572
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	573 if ((index->map->hdr.flags & MAIL_INDEX_HDR_FLAG_CORRUPTED) != 0) {
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	574 /* index was marked corrupted. we'll probably need to
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	575 recreate the files. */
17464 203b3a6f508f Removed pointless NULL checks. Timo Sirainen <tss@iki.fi> parents: 17130 diff changeset	576 mail_index_unmap(&index->map);
13384 33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	577 mail_index_close_file(index);
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	578 mail_transaction_log_close(index->log);
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	579 if ((ret = mail_index_open_files(index, flags)) <= 0)
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	580 return ret;
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	581 }
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	582
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	583 index->open_count++;
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	584 return 1;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	585 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	586
11623 c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	587 int mail_index_open(struct mail_index *index, enum mail_index_open_flags flags)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	588 {
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	589 int ret;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	590
10677 93fe3aa23bdb lib-index: Don't close index files if they're still being used by another mailbox. Timo Sirainen <tss@iki.fi> parents: 10674 diff changeset	591 if (index->open_count > 0) {
13384 33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	592 if ((ret = mail_index_open_opened(index, flags)) <= 0) {
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	593 /* doesn't exist and create flag not used */
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	594 }
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	595 return ret;
2926 c035583b4839 Make sure index is reopened if it's marked corrupted. Timo Sirainen <tss@iki.fi> parents: 2922 diff changeset	596 }
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	597
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	598 index->filepath = MAIL_INDEX_IS_IN_MEMORY(index) ?
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	599 i_strdup("(in-memory index)") :
40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	600 i_strconcat(index->dir, "/", index->prefix, NULL);
1967 b6d900688c55 memory leak fixes Timo Sirainen <tss@iki.fi> parents: 1965 diff changeset	601
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	602 index->readonly = FALSE;
fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	603 index->nodiskspace = FALSE;
fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	604 index->index_lock_timeout = FALSE;
12249 41e99ee5c1dd lib-index: Avoid unnecessary pread()s at end of transaction log file. Timo Sirainen <tss@iki.fi> parents: 12202 diff changeset	605 index->log_sync_locked = FALSE;
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	606 index->flags = flags;
6686 610f3d9813b5 Added MAIL_INDEX_OPEN_FLAG_READONLY. Timo Sirainen <tss@iki.fi> parents: 6679 diff changeset	607 index->readonly = (flags & MAIL_INDEX_OPEN_FLAG_READONLY) != 0;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	608
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	609 if ((flags & MAIL_INDEX_OPEN_FLAG_NFS_FLUSH) != 0 &&
11634 7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	610 index->fsync_mode != FSYNC_MODE_ALWAYS)
7f19062f58fd Renamed fsync_disable to mail_fsync=optimized\|always\|never. Timo Sirainen <tss@iki.fi> parents: 11623 diff changeset	611 i_fatal("nfs flush requires mail_fsync=always");
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	612 if ((flags & MAIL_INDEX_OPEN_FLAG_NFS_FLUSH) != 0 &&
126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	613 (flags & MAIL_INDEX_OPEN_FLAG_MMAP_DISABLE) == 0)
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	614 i_fatal("nfs flush requires mmap_disable=yes");
fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	615
13384 33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	616 /* NOTE: increase open_count only after mail_index_open_files().
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	617 it's used elsewhere to check if we're doing an initial opening
33dc66d4286d lib-index: Fixed reopening index file that was marked as corrupted. Timo Sirainen <tss@iki.fi> parents: 13379 diff changeset	618 of the index files */
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	619 if ((ret = mail_index_open_files(index, flags)) <= 0) {
fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	620 /* doesn't exist and create flag not used */
17579 092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	621 mail_index_close_nonopened(index);
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	622 return ret;
3824 3e756ea0474b Comment updates, small cleanups and optimizations. Timo Sirainen <tss@iki.fi> parents: 3819 diff changeset	623 }
21184 e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	624
12202 8af455e61dad lib-index: Avoid refreshing transaction log immediately after it was opened. Timo Sirainen <tss@iki.fi> parents: 12009 diff changeset	625 index->open_count++;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	626
21184 e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	627 if (index->log->head == NULL) {
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	628 mail_index_close(index);
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	629 mail_index_set_error(index, "Index is corrupted "
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	630 "(log->view->head == NULL)");
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	631 return -1;
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	632 }
e87c575aaa1c lib-index: Do not crash if log is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21176 diff changeset	633
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	634 i_assert(index->map != NULL);
10656 9d3b5cbef222 Moved mail_index lookup cache from lib-storage to lib-index. Timo Sirainen <tss@iki.fi> parents: 10655 diff changeset	635 mail_index_alloc_cache_index_opened(index);
6375 fc2f5607108d fsck the index while it's being mapped. Timo Sirainen <tss@iki.fi> parents: 6372 diff changeset	636 return 1;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	637 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	638
8953 f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	639 int mail_index_open_or_create(struct mail_index *index,
11623 c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	640 enum mail_index_open_flags flags)
8953 f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	641 {
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	642 int ret;
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	643
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	644 flags \|= MAIL_INDEX_OPEN_FLAG_CREATE;
11623 c3ee4421e86a Added mail_max_lock_timeout setting. Timo Sirainen <tss@iki.fi> parents: 10722 diff changeset	645 ret = mail_index_open(index, flags);
8953 f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	646 i_assert(ret != 0);
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	647 return ret < 0 ? -1 : 0;
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	648 }
f80f26822262 Added mail_index_open_or_create(). Timo Sirainen <tss@iki.fi> parents: 8590 diff changeset	649
6390 ff48b008150f Fix ESTALE handling when reading main index. Timo Sirainen <tss@iki.fi> parents: 6384 diff changeset	650 void mail_index_close_file(struct mail_index *index)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	651 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	652 if (index->fd != -1) {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	653 if (close(index->fd) < 0)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	654 mail_index_set_syscall_error(index, "close()");
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	655 index->fd = -1;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	656 }
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	657 }
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	658
17579 092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	659 static void mail_index_close_nonopened(struct mail_index *index)
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	660 {
12762 74300385cce0 lib-index: Give better assert-crashes if view or log view isn't closed. Timo Sirainen <tss@iki.fi> parents: 12702 diff changeset	661 i_assert(!index->syncing);
17108 3f352534af56 lib-index: Keep track of views and transactions in linked lists. Timo Sirainen <tss@iki.fi> parents: 15904 diff changeset	662 i_assert(index->views == NULL);
12762 74300385cce0 lib-index: Give better assert-crashes if view or log view isn't closed. Timo Sirainen <tss@iki.fi> parents: 12702 diff changeset	663
5827 f17d4bccd74d Memory and fd leak fix Timo Sirainen <tss@iki.fi> parents: 5820 diff changeset	664 if (index->map != NULL)
5853 d9f319a9d6e0 mail_index_map now contains pointer to index, so all functions taking map Timo Sirainen <tss@iki.fi> parents: 5827 diff changeset	665 mail_index_unmap(&index->map);
5827 f17d4bccd74d Memory and fd leak fix Timo Sirainen <tss@iki.fi> parents: 5820 diff changeset	666
f17d4bccd74d Memory and fd leak fix Timo Sirainen <tss@iki.fi> parents: 5820 diff changeset	667 mail_index_close_file(index);
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	668 mail_transaction_log_close(index->log);
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	669 if (index->cache != NULL)
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	670 mail_cache_free(&index->cache);
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	671
3879 928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to Timo Sirainen <tss@iki.fi> parents: 3863 diff changeset	672 i_free_and_null(index->filepath);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	673
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	674 index->indexid = 0;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	675 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	676
17579 092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	677 void mail_index_close(struct mail_index *index)
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	678 {
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	679 i_assert(index->open_count > 0);
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	680
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	681 mail_index_alloc_cache_index_closing(index);
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	682 if (--index->open_count == 0)
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	683 mail_index_close_nonopened(index);
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	684 }
092137bd092b lib-index: Fixed error handling in mail_index_open() Timo Sirainen <tss@iki.fi> parents: 17578 diff changeset	685
9311 0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	686 int mail_index_unlink(struct mail_index *index)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	687 {
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	688 const char *path;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	689 int last_errno = 0;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	690
10006 a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	691 if (MAIL_INDEX_IS_IN_MEMORY(index) \|\| index->readonly)
9311 0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	692 return 0;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	693
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	694 /* main index */
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	695 if (unlink(index->filepath) < 0 && errno != ENOENT)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	696 last_errno = errno;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	697
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	698 /* logs */
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	699 path = t_strconcat(index->filepath, MAIL_TRANSACTION_LOG_SUFFIX, NULL);
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	700 if (unlink(path) < 0 && errno != ENOENT)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	701 last_errno = errno;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	702
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	703 path = t_strconcat(index->filepath,
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	704 MAIL_TRANSACTION_LOG_SUFFIX".2", NULL);
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	705 if (unlink(path) < 0 && errno != ENOENT)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	706 last_errno = errno;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	707
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	708 /* cache */
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	709 path = t_strconcat(index->filepath, MAIL_CACHE_FILE_SUFFIX, NULL);
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	710 if (unlink(path) < 0 && errno != ENOENT)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	711 last_errno = errno;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	712
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	713 if (last_errno == 0)
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	714 return 0;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	715 else {
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	716 errno = last_errno;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	717 return -1;
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	718 }
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	719 }
0b3d90514e85 Added mail_index_unlink(). Timo Sirainen <tss@iki.fi> parents: 9100 diff changeset	720
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	721 int mail_index_reopen_if_changed(struct mail_index *index,
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	722 const char **reason_r)
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	723 {
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	724 struct stat st1, st2;
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	725 int ret;
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	726
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	727 if (MAIL_INDEX_IS_IN_MEMORY(index)) {
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	728 *reason_r = "in-memory index";
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	729 return 0;
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	730 }
3243 40b4ba3c55b8 In-memory indexes work again. Just pass dir as NULL to mail_index_alloc(). Timo Sirainen <tss@iki.fi> parents: 3231 diff changeset	731
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	732 if (index->fd == -1)
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	733 goto final;
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	734
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	735 if ((index->flags & MAIL_INDEX_OPEN_FLAG_NFS_FLUSH) != 0)
6857 41911abe6fa7 NFS cache flushing updates. Timo Sirainen <tss@iki.fi> parents: 6829 diff changeset	736 nfs_flush_file_handle_cache(index->filepath);
6797 a59deefc552f NFS attribute cache flushing fixes. nfs_flush_attr_cache() takes now Timo Sirainen <tss@iki.fi> parents: 6755 diff changeset	737 if (nfs_safe_stat(index->filepath, &st2) < 0) {
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	738 if (errno == ENOENT) {
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	739 *reason_r = "index not found via stat()";
6797 a59deefc552f NFS attribute cache flushing fixes. nfs_flush_attr_cache() takes now Timo Sirainen <tss@iki.fi> parents: 6755 diff changeset	740 return 0;
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	741 }
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	742 mail_index_set_syscall_error(index, "stat()");
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	743 return -1;
6797 a59deefc552f NFS attribute cache flushing fixes. nfs_flush_attr_cache() takes now Timo Sirainen <tss@iki.fi> parents: 6755 diff changeset	744 }
a59deefc552f NFS attribute cache flushing fixes. nfs_flush_attr_cache() takes now Timo Sirainen <tss@iki.fi> parents: 6755 diff changeset	745
4023 b19ccbaa3802 Try to handle ESTALE NFS errors the best way we can. Timo Sirainen <timo.sirainen@movial.fi> parents: 3912 diff changeset	746 if (fstat(index->fd, &st1) < 0) {
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	747 if (!ESTALE_FSTAT(errno)) {
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	748 mail_index_set_syscall_error(index, "fstat()");
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	749 return -1;
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	750 }
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	751 /* deleted/recreated, reopen */
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	752 *reason_r = "index is stale";
6829 dbab5e592577 NFS cache flushing APIs changed and backend implements it a bit differently. Timo Sirainen <tss@iki.fi> parents: 6819 diff changeset	753 } else if (st1.st_ino == st2.st_ino &&
dbab5e592577 NFS cache flushing APIs changed and backend implements it a bit differently. Timo Sirainen <tss@iki.fi> parents: 6819 diff changeset	754 CMP_DEV_T(st1.st_dev, st2.st_dev)) {
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	755 /* the same file */
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	756 *reason_r = "index unchanged";
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	757 return 1;
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	758 } else {
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	759 *reason_r = "index inode changed";
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	760 }
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	761
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	762 /* new file, new locks. the old fd can keep its locks, they don't
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	763 matter anymore as no-one's going to modify the file. */
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	764 mail_index_close_file(index);
c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	765
21087 3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	766 final:
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	767 if ((ret = mail_index_try_open_only(index)) == 0)
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	768 *reason_r = "index not found via open()";
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	769 else if (ret > 0)
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	770 *reason_r = "index opened";
3b909cd1264b lib-index: Improve error messages when transaction log is unexpectedly lost Timo Sirainen <timo.sirainen@dovecot.fi> parents: 20716 diff changeset	771 return ret;
2063 21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	772 }
21a3e5d08290 mail_index_refresh() - allows forcing a refresh check Timo Sirainen <tss@iki.fi> parents: 2051 diff changeset	773
2936 57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	774 int mail_index_refresh(struct mail_index *index)
57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	775 {
57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	776 int ret;
57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	777
5854 eca7e29dce0d Rewrite index lock handling. Timo Sirainen <tss@iki.fi> parents: 5853 diff changeset	778 ret = mail_index_map(index, MAIL_INDEX_SYNC_HANDLER_HEAD);
2936 57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	779 return ret <= 0 ? -1 : 0;
57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	780 }
57f975a9b63b Renamed mail_index_refresh() to mail_index_reopen_if_needed(). Added public Timo Sirainen <tss@iki.fi> parents: 2935 diff changeset	781
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	782 struct mail_cache mail_index_get_cache(struct mail_index index)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	783 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	784 return index->cache;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	785 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	786
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	787 void mail_index_set_error(struct mail_index index, const char fmt, ...)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	788 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	789 va_list va;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	790
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	791 i_free(index->error);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	792
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	793 if (fmt == NULL)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	794 index->error = NULL;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	795 else {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	796 va_start(va, fmt);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	797 index->error = i_strdup_vprintf(fmt, va);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	798 va_end(va);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	799
21176 eb5e99ab03d9 lib-index: Revert log flooding prevention / 3c014db6f Timo Sirainen <timo.sirainen@dovecot.fi> parents: 21087 diff changeset	800 i_error("%s", index->error);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	801 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	802 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	803
9099 67369ba9f447 maildir: When saving messages, in some race conditions we might have written a duplicate UID. Timo Sirainen <tss@iki.fi> parents: 8973 diff changeset	804 bool mail_index_is_in_memory(struct mail_index *index)
67369ba9f447 maildir: When saving messages, in some race conditions we might have written a duplicate UID. Timo Sirainen <tss@iki.fi> parents: 8973 diff changeset	805 {
67369ba9f447 maildir: When saving messages, in some race conditions we might have written a duplicate UID. Timo Sirainen <tss@iki.fi> parents: 8973 diff changeset	806 return MAIL_INDEX_IS_IN_MEMORY(index);
67369ba9f447 maildir: When saving messages, in some race conditions we might have written a duplicate UID. Timo Sirainen <tss@iki.fi> parents: 8973 diff changeset	807 }
67369ba9f447 maildir: When saving messages, in some race conditions we might have written a duplicate UID. Timo Sirainen <tss@iki.fi> parents: 8973 diff changeset	808
3833 318c70c1d4ec Beginnings of fallbacking to in-memory indexes when write fails with "out of Timo Sirainen <tss@iki.fi> parents: 3832 diff changeset	809 int mail_index_move_to_memory(struct mail_index *index)
318c70c1d4ec Beginnings of fallbacking to in-memory indexes when write fails with "out of Timo Sirainen <tss@iki.fi> parents: 3832 diff changeset	810 {
3911 3a2fe49912f3 Fixed mail_index_move_to_memory() to work better and made it public Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	811 struct mail_index_map *map;
3a2fe49912f3 Fixed mail_index_move_to_memory() to work better and made it public Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	812
4394 0462137fa4ce mail_index_move_to_memory(): Don't break if the index is already in memory. Timo Sirainen <tss@iki.fi> parents: 4312 diff changeset	813 if (MAIL_INDEX_IS_IN_MEMORY(index))
5430 c8bb83980ff7 If mail_index_move_to_memory() fails, make sure the future calls to it also Timo Sirainen <tss@iki.fi> parents: 5323 diff changeset	814 return index->map == NULL ? -1 : 0;
4394 0462137fa4ce mail_index_move_to_memory(): Don't break if the index is already in memory. Timo Sirainen <tss@iki.fi> parents: 4312 diff changeset	815
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	816 if ((index->flags & MAIL_INDEX_OPEN_FLAG_NEVER_IN_MEMORY) != 0)
8973 f295678cef1e dbox: Make sure indexes are never moved to memory (in case of out-of-disk space). Timo Sirainen <tss@iki.fi> parents: 8960 diff changeset	817 return -1;
f295678cef1e dbox: Make sure indexes are never moved to memory (in case of out-of-disk space). Timo Sirainen <tss@iki.fi> parents: 8960 diff changeset	818
4919 d54145736f99 If mail_index_move_to_memory() was called before index file was opened, we Timo Sirainen <tss@iki.fi> parents: 4918 diff changeset	819 /* set the index as being into memory */
d54145736f99 If mail_index_move_to_memory() was called before index file was opened, we Timo Sirainen <tss@iki.fi> parents: 4918 diff changeset	820 i_free_and_null(index->dir);
d54145736f99 If mail_index_move_to_memory() was called before index file was opened, we Timo Sirainen <tss@iki.fi> parents: 4918 diff changeset	821
5274 7641e625fbd5 When moving index file to memory, change the filename to be "(in-memory Timo Sirainen <tss@iki.fi> parents: 5273 diff changeset	822 i_free(index->filepath);
7641e625fbd5 When moving index file to memory, change the filename to be "(in-memory Timo Sirainen <tss@iki.fi> parents: 5273 diff changeset	823 index->filepath = i_strdup("(in-memory index)");
7641e625fbd5 When moving index file to memory, change the filename to be "(in-memory Timo Sirainen <tss@iki.fi> parents: 5273 diff changeset	824
3912 fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	825 if (index->map == NULL) {
5224 c3ab7e621fa6 comment cleanup Timo Sirainen <tss@iki.fi> parents: 5212 diff changeset	826 /* index was never even opened. just mark it as being in
3912 fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	827 memory and let the caller re-open the index. */
fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	828 i_assert(index->fd == -1);
fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	829 return -1;
fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	830 }
fc0b638330a4 Added mbox_min_index_size setting. Timo Sirainen <tss@iki.fi> parents: 3911 diff changeset	831
3911 3a2fe49912f3 Fixed mail_index_move_to_memory() to work better and made it public Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	832 /* move index map to memory */
5278 d569caa88a66 If we run out of disk space, move to in-memory indexes. Timo Sirainen <tss@iki.fi> parents: 5274 diff changeset	833 if (!MAIL_INDEX_MAP_IS_IN_MEMORY(index->map)) {
5689 c2362f144f15 Initial commit for major index file code cleanup. Timo Sirainen <tss@iki.fi> parents: 5686 diff changeset	834 map = mail_index_map_clone(index->map);
5853 d9f319a9d6e0 mail_index_map now contains pointer to index, so all functions taking map Timo Sirainen <tss@iki.fi> parents: 5827 diff changeset	835 mail_index_unmap(&index->map);
5278 d569caa88a66 If we run out of disk space, move to in-memory indexes. Timo Sirainen <tss@iki.fi> parents: 5274 diff changeset	836 index->map = map;
d569caa88a66 If we run out of disk space, move to in-memory indexes. Timo Sirainen <tss@iki.fi> parents: 5274 diff changeset	837 }
3911 3a2fe49912f3 Fixed mail_index_move_to_memory() to work better and made it public Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	838
4053 94b3363165c8 If transaction log opening fails, don't crash when trying to move it to Timo Sirainen <tss@iki.fi> parents: 4026 diff changeset	839 if (index->log != NULL) {
94b3363165c8 If transaction log opening fails, don't crash when trying to move it to Timo Sirainen <tss@iki.fi> parents: 4026 diff changeset	840 /* move transaction log to memory */
5714 dcf5dbb6a76a mail_transaction_log_move_to_memory() can't fail anymore. Moved the code to Timo Sirainen <tss@iki.fi> parents: 5689 diff changeset	841 mail_transaction_log_move_to_memory(index->log);
4053 94b3363165c8 If transaction log opening fails, don't crash when trying to move it to Timo Sirainen <tss@iki.fi> parents: 4026 diff changeset	842 }
3911 3a2fe49912f3 Fixed mail_index_move_to_memory() to work better and made it public Timo Sirainen <tss@iki.fi> parents: 3879 diff changeset	843
6022 8da2881df0dc mail_index_move_to_memory(): Don't log an error if index fd is already closed Timo Sirainen <tss@iki.fi> parents: 5976 diff changeset	844 if (index->fd != -1) {
8da2881df0dc mail_index_move_to_memory(): Don't log an error if index fd is already closed Timo Sirainen <tss@iki.fi> parents: 5976 diff changeset	845 if (close(index->fd) < 0)
8da2881df0dc mail_index_move_to_memory(): Don't log an error if index fd is already closed Timo Sirainen <tss@iki.fi> parents: 5976 diff changeset	846 mail_index_set_syscall_error(index, "close()");
8da2881df0dc mail_index_move_to_memory(): Don't log an error if index fd is already closed Timo Sirainen <tss@iki.fi> parents: 5976 diff changeset	847 index->fd = -1;
8da2881df0dc mail_index_move_to_memory(): Don't log an error if index fd is already closed Timo Sirainen <tss@iki.fi> parents: 5976 diff changeset	848 }
5714 dcf5dbb6a76a mail_transaction_log_move_to_memory() can't fail anymore. Moved the code to Timo Sirainen <tss@iki.fi> parents: 5689 diff changeset	849 return 0;
3833 318c70c1d4ec Beginnings of fallbacking to in-memory indexes when write fails with "out of Timo Sirainen <tss@iki.fi> parents: 3832 diff changeset	850 }
318c70c1d4ec Beginnings of fallbacking to in-memory indexes when write fails with "out of Timo Sirainen <tss@iki.fi> parents: 3832 diff changeset	851
1945 268a4096e8be cleanup Timo Sirainen <tss@iki.fi> parents: 1943 diff changeset	852 void mail_index_mark_corrupted(struct mail_index *index)
1943 e473d2df21be missing index file fixes Timo Sirainen <tss@iki.fi> parents: 1942 diff changeset	853 {
5820 e59892590a02 Handle losing index file better. Handle fsck better. Timo Sirainen <tss@iki.fi> parents: 5819 diff changeset	854 index->indexid = 0;
1945 268a4096e8be cleanup Timo Sirainen <tss@iki.fi> parents: 1943 diff changeset	855
5772 002aa9bbfcb2 Removed index->hdr. Use index->map->hdr instead. Timo Sirainen <tss@iki.fi> parents: 5714 diff changeset	856 index->map->hdr.flags \|= MAIL_INDEX_HDR_FLAG_CORRUPTED;
10006 a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	857 if (!index->readonly) {
a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	858 if (unlink(index->filepath) < 0 &&
a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	859 errno != ENOENT && errno != ESTALE)
a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	860 mail_index_set_syscall_error(index, "unlink()");
13379 7b240ecc37b5 lib-index: When marking index corrupted, delete also transaction log file. Timo Sirainen <tss@iki.fi> parents: 13327 diff changeset	861 (void)mail_transaction_log_unlink(index->log);
10006 a425ba46a64c lib-index: Take MAIL_INDEX_OPEN_FLAG_READONLY more seriously. Timo Sirainen <tss@iki.fi> parents: 9950 diff changeset	862 }
1943 e473d2df21be missing index file fixes Timo Sirainen <tss@iki.fi> parents: 1942 diff changeset	863 }
e473d2df21be missing index file fixes Timo Sirainen <tss@iki.fi> parents: 1942 diff changeset	864
10655 fc0ac73f0b36 Added support for marking mailbox index deleted. Don't allow any changes after that. Timo Sirainen <tss@iki.fi> parents: 10582 diff changeset	865 bool mail_index_is_deleted(struct mail_index *index)
fc0ac73f0b36 Added support for marking mailbox index deleted. Don't allow any changes after that. Timo Sirainen <tss@iki.fi> parents: 10582 diff changeset	866 {
10674 22354f505277 lib-index: Added support for undeleting a deleted index. Timo Sirainen <tss@iki.fi> parents: 10656 diff changeset	867 return index->index_delete_requested \|\| index->index_deleted;
10655 fc0ac73f0b36 Added support for marking mailbox index deleted. Don't allow any changes after that. Timo Sirainen <tss@iki.fi> parents: 10582 diff changeset	868 }
fc0ac73f0b36 Added support for marking mailbox index deleted. Don't allow any changes after that. Timo Sirainen <tss@iki.fi> parents: 10582 diff changeset	869
10722 31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	870 int mail_index_get_modification_time(struct mail_index index, time_t mtime_r)
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	871 {
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	872 struct stat st;
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	873
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	874 if (mail_transaction_log_get_mtime(index->log, mtime_r) < 0)
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	875 return -1;
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	876
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	877 if (*mtime_r == 0) {
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	878 if (stat(index->filepath, &st) < 0) {
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	879 mail_index_set_syscall_error(index, "stat()");
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	880 return -1;
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	881 }
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	882 *mtime_r = st.st_mtime;
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	883 }
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	884 return 0;
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	885 }
31aca1df525b lib-index: Added mail_index_get_modification_time(). Timo Sirainen <tss@iki.fi> parents: 10686 diff changeset	886
9420 c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	887 void mail_index_fchown(struct mail_index index, int fd, const char path)
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	888 {
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	889 mode_t mode;
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	890
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	891 if (index->gid == (gid_t)-1) {
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	892 /* no gid changing */
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	893 return;
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	894 } else if (fchown(fd, (uid_t)-1, index->gid) == 0) {
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	895 /* success */
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	896 return;
9421 14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	897 } if ((index->mode & 0060) >> 3 == (index->mode & 0006)) {
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	898 /* group and world permissions are the same, so group doesn't
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	899 really matter. ignore silently. */
9420 c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	900 return;
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	901 }
9698 37bdd5ce828f mail_index_chown(): Check for fchown() failing with EPERM, not EACCES. Timo Sirainen <tss@iki.fi> parents: 9696 diff changeset	902 if (errno != EPERM)
9537 b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	903 mail_index_file_set_syscall_error(index, path, "fchown()");
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	904 else {
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	905 mail_index_set_error(index, "%s",
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	906 eperm_error_get_chgrp("fchown", path, index->gid,
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	907 index->gid_origin));
b373de4973cd Whenever file's group changing fails, show the group origin in the error message. Timo Sirainen <tss@iki.fi> parents: 9535 diff changeset	908 }
9421 14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	909
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	910 /* continue, but change permissions so that only the common
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	911 subset of group and world is used. this makes sure no one
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	912 gets any extra permissions. */
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	913 mode = ((index->mode & 0060) >> 3) & (index->mode & 0006);
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	914 mode \|= (mode << 3) \| (index->mode & 0600);
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	915 if (fchmod(fd, mode) < 0)
14ed9ca980d1 More group permission handling fixes. Timo Sirainen <tss@iki.fi> parents: 9420 diff changeset	916 mail_index_file_set_syscall_error(index, path, "fchmod()");
9420 c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	917 }
c496f137c843 index: Handle better errors where a new file's group can't be changed. Timo Sirainen <tss@iki.fi> parents: 9394 diff changeset	918
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	919 void mail_index_set_syscall_error(struct mail_index *index,
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	920 const char *function)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	921 {
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	922 mail_index_file_set_syscall_error(index, index->filepath, function);
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	923 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	924
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	925 void mail_index_file_set_syscall_error(struct mail_index *index,
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	926 const char *filepath,
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	927 const char *function)
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	928 {
10686 442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	929 const char *errstr;
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	930
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	931 i_assert(filepath != NULL);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	932 i_assert(function != NULL);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	933
22361 73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	934 if (errno == ENOENT) {
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	935 struct stat st;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	936 int old_errno = errno;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	937 i_assert(index->log->filepath != NULL);
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	938 if (nfs_safe_stat(index->log->filepath, &st) < 0 &&
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	939 errno == ENOENT) {
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	940 /* the index log has gone away */
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	941 index->index_deleted = TRUE;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	942 errno = old_errno;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	943 return;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	944 }
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	945 errno = old_errno;
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	946 }
73479718bc0b lib-index: Mark index deleted if index log file is missing Aki Tuomi <aki.tuomi@dovecot.fi> parents: 21390 diff changeset	947
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	948 if (ENOSPACE(errno)) {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	949 index->nodiskspace = TRUE;
9696 126132cb1c19 index: Removed duplication of mail_index_open_flags from struct mail_index. Timo Sirainen <tss@iki.fi> parents: 9614 diff changeset	950 if ((index->flags & MAIL_INDEX_OPEN_FLAG_NEVER_IN_MEMORY) == 0)
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	951 return;
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	952 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	953
9535 bbe4c24faf21 Use eacces_get_error() in more places for handling EACCES errors. Timo Sirainen <tss@iki.fi> parents: 9421 diff changeset	954 if (errno == EACCES) {
10686 442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	955 function = t_strcut(function, '(');
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	956 if (strcmp(function, "creat") == 0 \|\|
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	957 strncmp(function, "file_dotlock_", 13) == 0)
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	958 errstr = eacces_error_get_creating(function, filepath);
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	959 else
442c980d6221 lib-index: Fixed logging file creation EACCES failures. Timo Sirainen <tss@iki.fi> parents: 10677 diff changeset	960 errstr = eacces_error_get(function, filepath);
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	961 mail_index_set_error(index, "%s", errstr);
9535 bbe4c24faf21 Use eacces_get_error() in more places for handling EACCES errors. Timo Sirainen <tss@iki.fi> parents: 9421 diff changeset	962 } else {
12009 706f30fa4028 lib-index: Give a clearer error message for syscall failing with EFBIG. Timo Sirainen <tss@iki.fi> parents: 12008 diff changeset	963 const char *suffix = errno != EFBIG ? "" :
706f30fa4028 lib-index: Give a clearer error message for syscall failing with EFBIG. Timo Sirainen <tss@iki.fi> parents: 12008 diff changeset	964 " (process was started with ulimit -f limit)";
14682 d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	965 mail_index_set_error(index, "%s failed with file %s: "
d0d7b810646b Make sure we check all the functions' return values. Minor API changes to simplify this. Timo Sirainen <tss@iki.fi> parents: 14629 diff changeset	966 "%m%s", function, filepath, suffix);
9535 bbe4c24faf21 Use eacces_get_error() in more places for handling EACCES errors. Timo Sirainen <tss@iki.fi> parents: 9421 diff changeset	967 }
1915 79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	968 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	969
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	970 const char mail_index_get_error_message(struct mail_index index)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	971 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	972 return index->error;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	973 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	974
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	975 void mail_index_reset_error(struct mail_index *index)
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	976 {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	977 if (index->error != NULL) {
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	978 i_free(index->error);
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	979 index->error = NULL;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	980 }
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	981
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	982 index->nodiskspace = FALSE;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	983 index->index_lock_timeout = FALSE;
79790750c349 importing new index code. mbox still broken. Timo Sirainen <tss@iki.fi> parents: diff changeset	984 }

Mercurial > dovecot > core-2.2

annotate src/lib-index/mail-index.c @ 22361:73479718bc0b