.clang-tidy: Silence some lints

lib: connection - Make connection_list_deinit(NULL) no-op

configure: Default to main if asset version is empty

lib-auth-client: auth-master - Fix crash if failing to connect to socket

auth_master_unset_io() would be called twice: once by
auth_master_run_cmd_pre() and again by auth_master_run_cmd_post(), which
attempts to add the idle timeout again.

Fixes:
Panic: file auth-master.c: line 440 (auth_master_unset_io): assertion failed: (conn->to == NULL)

lib-mail: message_skip_virtual() - Optimize with memchr()

lib-mail: message_skip_virtual() - Fix handling plain CRs in message body

The plain CRs should be handled like any other character, not returning
last_cr_r=TRUE.

This fixes partial IMAP FETCHes where body contains e.g. "\rfoo" and
doing a partial FETCH BODY[TEXT]<1> to skip over the CR.

Fixes errors like:
Deleting corrupted cache record: Broken virtual size in mailbox .. FETCH BODY[]<1> got too little data: 2 vs 3

dovecot-config: Propagate mail utf-8 presence properly

lib: test-data-stack - Explicitly ignore result from t_malloc_no0() when neeeded

Satisfies static analysers

lib: test-data-stack - Drop bogus assertion

This assertion goes back to 992a1726a41b42fa47204565ff17f7c635fcb421 when
test-data-stack.c was added.

It starts to fail with (upcoming) GCC 15 which has improvements for
optimising out redundant pointer-vs-pointer comparisons, specifically
r15-580-gf3e5f4c58591f5 for gcc bug PR13962.

Anyway, this is a problem for this assertion because t_malloc_no0
is marked with `__attribute__((malloc))` which guarantees that the
returned pointer doesn't alias, hence a == b must be false.

Bug: https://bugs.gentoo.org/939857

lib-imap: Fail parsing on impossible imap-dates/times

For example 32-Jan-2023 was converted into a max-timestamp before instead of
being rejected.

lib-index: Fix handling indexid change in dovecot.index file

Depending on the situation the dovecot.index should be either deleted or
the mailbox reopened.

lib-index: Split off mail_transaction_log_has_changed()

lib-index: Don't delete dovecot.index.log when finding that indexid changes

The indexes were recreated, so the new indexes should be used instead of
deleting them.

lib-storage: Fix duplicate mailbox name detection in list index rebuild

lib-storage: Split off mailbox_name_add_random_suffix()

lib-index: mail_transaction_log_view_mark() - Fix crash when log view is empty

This happened when IMAP NOTIFY was used and syncing was delayed enough for
dovecot.list.index to have become rotated twice and gotten missing
transactions.

imap-hibernate: Add local and remote port as variables to mail_log_prefix

lib-storage: Add local and remote port as variables to mail_log_prefix

lib-ldap: Rename libldap.la to libdldap.la

Using libldap.la caused some strange compiling issues at times.

lib: Move i_faccessat2() to lib.c

imap: Fix ifdef location

lib,doveadm: Add missing includes

lib: Add NOLINT clang-tidy directives for some helpers

lib-smtp: Remove duplicate function declaration

doveadm: Remove duplicate extern

lib-fs,lib-storage: Fix posix_fadvise usage

Return value is non-zero on error, not negative.

auth: Remove duplicate extern

auth: auth-cache - Remove duplicate include

lib: time-util - Fix implicit casts

lib-index: Remove duplicate function declaration

global: Add NOLINT for noisy macros

Remove warnings with clang-tidy

global: Fix whitespace -> tabs

global: Add clangd config file

global: Add clang-tidy config file

global: Add clang-format config file

gitignore: Ignore clangd and emacs directories

global: Add dovecot-style editor config

configure: Replace "if" with AS_IF

configure: Fix config version checks for x.y versioning (without .z)

lib-smtp: smtp-command-parser - Explicitly disallow 8-bit characters for AUTH response

Later Base64 parsing would catch that, but command parser errors become clearer
when this is caught early. Also, this fixes a unit test failure when
experimental SMTPUTF8 support is enabled.

When SMTPUTF8 support is disabled, smtp_char_is_textstr() disallows 8-bit
implicitly, but when SMTPUTF8 is enabled 8-bit octets will be let through.
This caused the invalidity unit test to fail.

lib-smtp: test-smtp-command-parser - Fail invalidity tests properly in case of valid result

lib-smtp: test-smtp-command-parser - Properly test bad parameter characters vs. missing EOL

lib-smtp: test-smtp-address - Vary tests based on SMTPUTF8 support

lib-sql: cassandra - Add data stack frame

lib-fs: Add fs_get_root_fs()

doveadm auth: Fix potential crash when auth is aborted with a signal

dict-ldap: Avoid linking to a dovecot-ldap shared library

The dict-ldap itself is the plugin, which is dynamically loaded.
There's no need for dovecot-ldap shared library to be used. This
changes the lib-ldap linking behavior to same as lib-sql.

lib-dns: dnc_client_lookup*() - Add data stack frames

doveadm: Use a separate memory pool for parsing doveadm command line parameters

This avoids spending a lot of the data stack for the actual command
processing, causing them to grow the data stack.

doc: Replace html documentation with redirect to documentation

NEWS: Add news for 2.3.21.1

lib-http: Add http_client_request_retrieve_headers()

lib-sql: Link test binaries to libdovecot.la

Otherwise it tries to use installed one, if available.

configure: Use $SED and $GREP to make sure we get correct tools

imap: Don't try to hibernate if it's not enabled

imap-hibernate: If not enabled, do not include in settings either

net: net_connect_ip*() - Handle OpenBSD errors similarly to FreeBSD

auth: Replace random() with proper i_rand_minmax()

dsync: Use correct format for time_t in dsync_brain_recv_mailbox_tree_deletes()

configure: Split grep into multiple -e expressions

This is more portable.

lib-ssl-iostream,lib-dcrypt: Raise minimum OpenSSL compatibility level to 1.1.1

But retain compatibility with LibreSSL.

lib: mkdir_parents() - Avoid unnecessary syscalls when uid/gid doesn't change

There's no need to open() + fchown() the directory after mkdir(), if neither
uid nor gid changes.

lib-settings: Fix variable type mismatches

Just use size_t everywhere since it points to mmap()ed memory.

auth: Fix userdb auth cache with username changes

The problem was for example when userdb lookup uses only the username part
of the username@domain lookup. Then:

 * "username" lookup caches the results for "username". Since the username
   didn't change, it doesn't store in the cache the "user" field.
 * "username@domain" lookup looks up "username" from cache. Since there is
   no "user" field, the code didn't think the username had changed.

Fix this by saving the "user" field to auth cache, regardless of whether
it's the same as the current username.

mdbox: Fix crash if resync fails because of an early failure

This mainly happened if dovecot.map.index.log was locked and resync timed
out waiting for it.

lib-lua: test-lua-http-client - Prevent HTTP client context from traversing fork

The children will start fighting over the connection FDs and the test will
panic out:

DNS: Panic: epoll_ctl(mod, 8) failed: No such file or directory
CLIENT: Panic: epoll_ctl(mod, 8) failed: No such file or directory

This happens very infrequently.

configure.ac: Fix generating DOVECOT_CONFIG_VERSION

configure.ac, config: Change supported config versions to work based on PACKAGE_VERSION

For git this means 0.0.0, but the supported versions still include hardcoded
2.4.0 / 3.0.0, which can be used.

lib-lua: Fix lua library dependencies when building --without-shared-libs

Linking against libdlua didn't automatically link with liblua, causing
lua plugin loading to fail.

lib: istream-concat - Fix potential hang when reaching max buffer size

imap: Fix potential hang when client sends too long line

imap: Fix potential hang when client sends too long APPEND line

imap: Fix potential hang/assert-crash when client sends too long line

If client sent some commands pipelined, followed by a command with too long
line, the previous commands' tagged replies weren't sent. This caused it to
hang until more input was received. After that it crashed.

Fixes:
Panic: file imap-client.c: line 1098 (client_check_command_hangs): assertion failed: (!have_wait_unfinished || unfinished_count > 0)

lib-smtp: Fix UTF-8 local-parts

configure: Explicitly check for icu-uc

Ensure we link against libicu-uc when it's split in v76.

lib-index: Fix potential crash after resetting index

The log's sync_offset wasn't updated after the reset if there were no other
changes. This ended up in a broken state in
mail_transaction_log_append_locked() where it thought there was garbage at
the end of the transaction log, and crashed trying to truncate it.

imapc: If a shared namespace INBOX is listed as \Noselect enforce it

In specific configurations a shared namespace INBOX is listed as
\Noselect
* LIST (\HasChildren) "/" shared/user
* LIST (\HasChildren \Noselect) "/" shared/user/INBOX

This is fixed by adding this extra check.

dsync: Fix potential crash when handling mailbox renames

Fixes:
Panic: file dsync-mailbox-tree-sync.c: line 575 (node_mailbox_trees_cmp): assertion failed: (ret != 0)

lib-storage: Mailbox list rebuild - avoid opening mailbox on recovery name collision

If recovered-lost-folder-$guid already unexpectedly exists, there's no need
to open it (at this point) to try to recover its name. This didn't really harm
anything, so it mainly fixes the code logic to make sense.

imap: Fix potential hang/crash when unhibernating

This fixes two potential bugs:

a) Unhibernation is triggered by new mailbox changes. While sending these
to IMAP client, Dovecot notices that the client has disconnected. The
process will stay IDLEing for 30 minutes until it gets disconnected with
"Disconnected: Inactivity - no input for 1800 secs"

b) Unhibernation is triggered by DONE command with some further pipelined
commands. During unhibernation new mailbox changes are noticed, and
again IMAP client is found to be disconnected while sending the changes.
This causes a segfault.

doveadm: doveadm-print-json - Prevent crash on subsequent user prints

Move closing the final json array delimiter to the deinitialization
routine instead of on flush.

lib-dcrypt: Ensure bd is not used uninitialized in store_jwk_key()

Makefile: Fix LIBDOVECOT_OPENSSL for installed dovecot-config file

lib-http, lib-smtp: unit tests - Make sure expected DNS lookup failures will fail

Use ".invalid" suffix just to be safe. More importantly, use a trailing
dot to disable search domains.

doc: Remove strict library version requirement from solr-config-9.xml

doc: Use solr.install.dir in solr-config-9.xml

Removes hardcoded path

lib: test_hash - Reduce loop counts with valgrind to save time

lib: test_hash - Add missing test_begin()/test_end()

lib: istream-multiplex tests - Reduce loop counts with valgrind to save time

lib: base64 tests - Reduce loop counts with valgrind to save time

lib: test-hash-method - Skip "large input" test with valgrind to save time

run-test-valgrind.supp: Add leak suppression for fatal_ostream_file test

The test is supposed to panic while destroying ostream, so there's no way to
prevent the leak.

auth: Fix typo in ssl_ja3_hash field name

Fixes passing ssl_ja3_hash to a blocking passdb

Broken in 70baf893e85222d071d21cdd5ead546a905d477a

lib: test-ostream-file - Make sure lack of error handling panics

lib: o_stream_close() - Don't reset error checking on automatic flush

Otherwise it will be difficult to find missing o_stream_flush() calls.
The missing error handling would be found only when the flushing fails,
which would result in assert-crash.

This is how the code was originally intended to behave. Broken by
5ec4fc44e8d4e2160f07b1a7f4fce1ccfec3f6c1

imap: Ignore error handling for side channel output

lib-index: Ignore ostream errors when cache purging is aborted due to file too large

lib-smtp: test-smtp-client-errors - Add missing ostream finish

mbox: Fix ostream error handling in mbox_move()

o_stream_nsend_send_istream() needs to be followed by o_stream_flush().
It also detects the istream error.

virtual: Fix copying storage error on mail_precache() failure