lib-index: Update last_used field immediately when adding a new cache field

Otherwise the newly added field could be dropped immediately on the next
purge.

lib-index: Make sure purging clears saved bitmask offsets state between mails

This is just to make sure it doesn't try to use previous mail's offset when
merging bitmasks. This shouldn't have been possible anyway, because
field_seen array guaranteed that the bitmask_pos was always set before
reading it. This just makes extra sure of it.

lib-index: Fix mail_cache_get_first_new_seq() when all mails are old

Instead of returning "all messages are old", it returned "all messages are
new".

lib-index: mail_cache_get_missing_reason() - Add "Mail is already expunged" as reason

lib-index: Remove mail_cache_try_lock()

lib-index: Wait for lock in cache compression, not just try once

Sometimes there's an important reason for the cache to be compressed, and it
shouldn't give up just because there's a race condition with another process
that just happens to be writing to it.

lib-index: After cache is marked corrupted, don't log error if cache file isn't found

It's just an unnecessary extra error.

lib-index: Don't compress cache file if there are cache transactions with changes

The cache compression could have lost changes in the uncommitted
transactions if they had already written some changes to the file.
This seems to have happened sometimes at least with dsync.

lib-index: Cache compression could have lost data for recently added mails

mail_index_sync_ctx.view may not have seen all the newly added mails,
which caused compression to skip those mails. Use a whole new view
instead, which guarantees that all mails are seen.

lib-index: Fix cache lookup for a newly added field in non-committed transaction

The cache lookup didn't return anything unless the field already existed in
the current cache file. It could still have already been added to the
transaction and found from there.

lib-index: Don't even try to add fields to cache that exceed record_max_size

Previously these were temporarily added to memory and later on the whole
cache record was just dropped. But usually not all the cache fields are
huge, so it's better to add some fields to cache than none.

lib-index: Fix assert-crash if cache record has no content after all

This couldn't have happened yet, but will happen after the next commit.

lib-index: Use cache record's exact size when checking if it should be flushed

The field_idx wasn't included in the calculation.

lib-index: Replace mail_cache_flush_and_unlock() with mail_cache_unlock() where it's better

In various error handling code paths it's not useful to try to flush any
headers. It might just make things worse.

lib-index: Remove backward compatibility code for Dovecot v2.1 and older

We can still read old cache files, but Dovecot v2.1 can't anymore read these
newer cache files.

lib-index: Cache transaction rollback used too small value to increase deleted_record_count

Rollback used records_written to update it, but that was actually counting
the number of transaction flushes, not the number of individual mail cache
records that were written.

lib-index: mail_cache_transaction_rollback() - fix updating deleted_record_count

The deleted_record_count in cache header wasn't actually being updated
because hdr_modified wasn't TRUE.

lib-index: Add mail_cache_unlock() that doesn't flush any headers

lib-index: Rename mail_cache_unlock() to mail_cache_flush_and_unlock()

This describes better what it does, because it's not just unlocking.

lib-index: Make mail_cache_need_reopen() static

Nothing else besides mail_cache_*lock() needs to be calling it anymore.

lib-index: mail_cache_lookup_offset() - Use mail_cache_sync_reset_id()

This avoids race conditions where cache fields aren't found during cache
compression.

lib-index: Make mail_cache_sync_reset_id() public

Also it no longer requires that cache is already locked when being called.

lib-index: Mark cache corrupted after finding that reset_id permanently wrong

lib-index: mail_cache_*lock() - On reset_id mismatch wait for cache compression to finish

It was possible that the reset_id mismatch happened because cache
compression had already recreated the .cache file, but hadn't yet written
the new reset_id to the .log file. By waiting for the .log lock, we can be
sure that this won't happen.

lib-index: mail_cache_sync_reset_id() - Rewrite to simplify

This doesn't change any functionality.

lib-index: mail_cache_*lock() - Handle syscall errors correctly

If reading a cache file failed because of some temporary syscall error, it
was treated the same as if the cache was corrupted. This could have
caused compression to lose cached data.

lib-index: mail_cache_reopen_now() - Fix corruption handling remove checking if reset_id matches

reset_id checking is already done by all of the callers. Also simplify the
error vs corruption handling by using mail_cache_open_and_verify()

lib-index: Make mail_cache_need_reopen() public and require using it explicitly

mail_cache_reopen() no longer implicitly calls it.

lib-index: mail_cache_open_and_verify() - Separate error and corruption handling

lib-index: mail_cache_need_reopen() - Don't skip reopen just because compression is wanted

The new cache file may still be usable until the compression is done.

lib-index: mail_cache_map_all() - Separate error and corruption handling

lib-index: Remove cache compression dotlock

Now that .log file is guaranteed to be locked during cache compression,
there's no need for another exclusive lock.

lib-index: Fix checking if cache should be compressed because it's getting too large

Most importantly fixed updating last_stat_size whenever cache is locked.
This now guarantees that it's always fully up-to-date when it's being
checked. Most of the other places updating last_stat_size are now
unnecessary, but I left them in case they'll be useful in the future.

Fixed also off-by-one where purging was done also when cache was exactly at
its maximum size.

lib-index: Check if maximum cache file size has reached only after locking

This avoids race conditions.

lib-index: mail_cache_compress_with_trans() - Don't return lock anymore

Now that transaction log is always locked during cache compression, it
doesn't matter if the cache is unlocked before or after the transaction log
commit.

lib-index: mail_cache_compress_with_trans() - Require that transaction log is locked

This should be true now for all of its callers.

lib-index: mail_cache_compress() - Make sure compression sees the latest changes in index

lib-index: mail_cache_compress() - Keep the .log file locked during compression

lib-index: Remove mail_cache_compress_forced()

doveadm: Rewrite "doveadm mailbox cache purge" to use mail_cache_compress()

Also removed checks to verify whether the cache file existed or was usable.
Compression will create/fix the file, which is more likely to be the wanted
behavior than failing.

lib-index: mail_cache_compress*() - Add compress_file_seq parameter

This can be used to specify when the cache file should be compressed.
This was previously hidden in the cache->compress_file_seq field, but
now it's more explicit.

This commit also makes sure that cache is compressed when index rebuilding
is done.

lib-index: Use mail_cache_compress() where possible

lib-index: Add mail_cache_compress()

lib-index: Rename mail_cache_compress() to mail_cache_compress_with_trans()

lib-index: mail_cache_decisions_copy() - Create transaction internally

lib-index: Simplify opening cache when locking transaction

All this work is already done by mail_cache_lock() or
mail_cache_transaction_lock() after locking has failed.

lib-index: If cache transaction lock fails because reset_id mismatch, compress cache

This is also done in mail_cache_transaction_open_if_needed(), so it usually
worked anyway. But that function is otherwise unnecessary, so it's going
away in the next commit.

lib-index: Refresh caching decisions from cache file once per transaction

If another process had updated the decisions, mail_cache_field_want_add()
and mail_cache_field_can_add() may have returned obsolete values.

lib-index: If cache transaction flush fails, don't discard the last mail's cached data

The last mail's cached data could still be accessed within memory.

This can now be done because the field indexes are internal indexes instead
of file-specific indexes.

lib-index: If cache file is compressed during transaction, discard only flushed records

Cache records that are still in memory can be written to the new file.

lib-index: Simplify code to add missing header fields to cache

lib-index: Add only used fields to cache header, not everything

lib-index: Delay using file-specific field numbers in uncommited cache records

While adding data to cache, it's first kept in memory. Previously already
at this time the memory contained cache field numbers specified to the
current cache file. If the cache file was compressed, all the cached data
had to be discarded because the field numbers changed.

This commit changes the file-specific field indexes to internal index
numbers and converts them to file-specific numbers only just before writing
them to the cache file. This sigfinicantly simplifies the code. It also
allows following commits to avoid having to discard the data when cache is
compressed.

This change also required changing the cache lookup code to realize that
the uncommitted data contains internal field indexes.

lib-index: Keep a linked list of mail_cache_views

Assert-crash on cache deinit if any views are still open.

lib-ntlm: Replace strncpy() with memset()+memcpy()

There was nothing wrong with it, except gcc 9.2 logs a warning about it.

lib-imap: test-imap-url - Fix error handling

lib-http: Fix generating label for incomplete HTTP server request

lib-master: Fix error logging if reading "instances" file fails

The path was written as "(null)" in the error message (or it crashed in some
OSes).

lib-program-client: Do not derefence set when it is NULL.

Found by coverity. Broken in 23cfbf2451f7191dee1d9995011c898a7bf32bd4

log: LOG_OPTION is not supported here

Found by coverity, introduced in d176f84ce5ca2073f4dfbafb457b9c74f6bf0d76

lib-mail: message_header_decode_encoded - ensure num is QCOUNT

Satisifies static analyzers

auth: test-lua - Do not initialize NULL script

Satisfies code analyzers

lib-fts: Assert that data is non-NULL when size > 0

Satisfies code analyzers

lib-index: mail_cache_lookup_headers() - Use const for field_idxs[]

lib-index: Move MAIL_CACHE_MAX_WRITE_BUFFER to mail-cache-private.h

This will help unit tests.

lib-index: unit tests: Split off test_mail_index_close/delete()

lib-index: Remove mail_cache_field_private.adding field

This was used to make sure that cache compression didn't drop a field that
was just being added. After previous commit the cache is locked before this
code is even reached, so it's no longer possible.

lib-index: Use mail_cache_map_all() wherever possible

lib-index: Add mail_cache_map_all()

lib-index: mail_cache_transaction_update_index() - Return void since it can't fail

lib-index: mail_cache_sync_reset_id() - Move cache unlocking to caller

lib-index: Split off mail_cache_sync_reset_id()

lib-index: Split off mail_cache_lookup_rec_get_field()

lib-index: Split off mail_cache_transaction_forget_flushed()

lib-index: mail_cache_header_add_field_locked() - Move unlocking to caller

lib-index: Split off mail_cache_header_add_field_locked()

lib-index: mail_cache_header_fields_write() - Take cache parameter instead

lib-index: Split off mail_cache_trans_get_file_field()

lib-index: mail_cache_lock() - Clarify why index->mapping might be TRUE here

lib-index: mail_transaction_log_sync_lock() - Fix function's comment

lib-auth: Rename auth_client_request_continue event to ...continued

auth: auth_request_finished event - Rename translated_username to translated_user

All the other fields are also "user", not "username".

auth: auth_request_finished event - Rename original_username to orig_user

This way it matches %{orig_user} variable.

auth: Remove unnecessary NULL checks for adding event fields

The NULL values are ignored internally. No need to complicate the code.

auth: Remove duplicate "mechanism" field in auth request events

The "mech" field already existed, and it matches %{mech} variable.

lib-master: Replace auth-master-client-login category with auth-client

This is just a reimplementation of lib-auth/auth-master code. This commit
also changes lib-master to require lib-auth to get to the category. This
dependency shouldn't really be a problem, since eventually this
master-login-auth code should be replaced by lib-auth anyway.

lib-oauth2: Unit tests don't need to link libmaster

lib-auth: Replace auth-master-client category with auth-client

The auth-client and auth-master connections are using mostly the same auth
API. Also the auth-master event names were already auth_client_*. So
there's not much point in having separate categories for them - it's just
confusing.

push-notification: Rename event category from push_notification to push-notification

All other categories are using "-" instead of "_".

lib-dcrypt: Avoid using BN_bn2binpad

It's not available in older versions.

lib-dcrypt: Add EC_GROUP_order_bits if missing

Not there in older OpenSSL

lib: ostream.h - Fix indent in enum ostream_create_file_flags

lib-dcrypt: Require ECDSA signature size is even

Valid ECDSA signature has two parts of equal sizes

lib-dcrypt: Pad signature parts to match ECDSA key size

This is required for ECDSA signatures.

lib-compression: Add unit tests for gz header handling bugs

lib-compression: Fix length checks for zlib header

lib-mail: test-message-address - Add unit test for the one byte overread

Note that this is visible only with valgrind.

lib-mail: message_address_parse_path() - Fix one byte overread

lib-master: If stats_writer_socket_path is empty, don't try to send stats

lib-test: test_exit() - Use regular exit() instead of _exit()

Otherwise code coverage output isn't written for the forked process.