lib-smtp: smtp-server-recipient - Remove smtp_server_recipient_initialize().

It is no longer needed.

lib-smtp: smtp-server-recipient - Create transaction event early if there is no transaction yet.

This allows having a parent for the recipient event once the recipient is
created, which can precede the creation of the transaction.

lib: Free unused data stack memory once per second while running ioloop

This way if the data stack has grown excessively large temporarily, it
won't permanently waste memory. And if the data stack grows back to the
same large size, re-allocating it once per second doesn't cause performance
problems.

lib: data-stack - Keep the largest unused block in memory after all

This reverts 2da21080dbbe50b4924ac0135c84babfb404dcce, but that alone
doesn't produce working code so there are also some further fixups.

lib-smtp: test-smtp-payload - Double the client progress timeout.

Otherwise, SSL tests can fail on systems with low entropy.

lib-smtp: smtp-client-transaction - Allow destroying connection while transaction still holds reference.

The lifetime of the transaction is controlled by the application and not so much
by the connection. The transaction holds a reference to the connection, so if
the connection is forcibly destroyed it needs, to notify the transaction of this
event.

This fixes:

Panic: file connection.c: line 938 (connection_list_deinit): assertion failed: (conn != list->connections)

lib-smtp: smtp-client-transaction - Properly handle pending failure upon smtp_client_transaction_send_reset().

lib-smtp: smtp-client-connection - Add smtp_client_connection_transactions_fail() and use it.

Use it to always fail the whole transaction first before the individual commands
if possible.

lib-smtp: smtp-client-connection - Split off smtp_client_connection_transactions_fail_reply().

lib-smtp: smtp-client-connection - Always abort all commands upon smtp_client_connection_close().

Before, these would get a disconnect failure, which makes no sense when the
client is causing the disconnect.

lib-smtp: smtp-client-connection - Abort all transactions upon smtp_client_connection_close().

Before, these would get a disconnect failure, which makes no sense when the
client is causing the disconnect.

lib-smtp: test-smtp-payload - Finish client transaction only when it finishes completely.

lib-smtp: Reformat smtp-client-transaction.c.

lib-smtp: Reformat smtp-client-connection.c.

auth: oauth2 - Support passing openid-configuration URL to client on failure

auth: mech-oauth2 - Pass error fields as array

Makes it easier to add fields to it later on.

imap: copy/move: Fix memory leak when no messages were found

doveadm: Delay reading settings until doveadm plugins are loaded

doveadm: Add doveadm_setting_roots_*() API for plugins to register settings

doveadm: Move settings reading code into doveadm-settings.c

This code is now shared between doveadm and doveadm-server.

lib-master: Add master_service_settings_input.extra_modules

This allows specifying more than one module.

lib-master: master_service_exec_config() - Build argv using dynamic array

auth: auth-worker - Respect service_count

If service_count is set, then auth-worker will stop after
given number of requests.

imap: Improve sending internal error to client if initialization fails

The "Internal server error" should be sent using the same ostream that
was created. Also if service_user initialization failed there was no
OK reply to sent to login.

imap: Send initial tagged reply directly using ostream

This simplifies the following commits.

imap: Send initial tagged reply and capability after rawlogs are initialized

This way the initial tagged reply and capability is written to the rawlog.
Broken by 16328738d35370685387ec51d20657265855e770

imap: Split client_add_input_capability() into two functions

imap: Rename struct client_input to imap_login_request

This better indicates what it is about, and causes less confusion with
all the other client_input variables.

pop3: Handle pipelined input via istream-concat

This fixes rawlog_dir to work properly. It didn't log the pipelined input
to rawlog and it may have also caused the connection to hang.

pop3: Move finishing istream to client_create_finish()

imap: Handle pipelined input from imap-hibernate via istream-concat

Similarly to the previous commit, this fixes issues with rawlog.

imap: Handle pipelined input via istream-concat

This fixes rawlog_dir to work properly. It didn't log the pipelined input
to rawlog and it may have also caused the connection to hang.

imap: Move iostream-rawlog creation to client_create_finish()

This allows the caller to modify the istream before rawlog.

lib: Add i_stream_copy_fd()

acl: Fix broken LIST for shared namespaces

Due to the recent changes in the usage of the acl_ignore_namespace
setting shared namespaces where trying to use fast listing too. This
resulted in wrong LIST IMAP command outputs when using acl plugin.

Broken by dc8ecd38a7e54b8bb80ae97712a0d8ad4edcbed3

lib-lua: dlua-compat - Undefine lua_isinteger

It's being defined in compat to return boolean, so undefine it
here to make sure it compiles. Broken in b59e89ebd97fa52de25b8e976a7dfe61f0d8a639

acl: Prevent crashes with acl_ignore_namespace

In case a namespace was ignored for ACLs an crash could occur because of
"Module context acl_storage_module missing". This panic is prevented by this
change.

acl: Move static acl_namespace_is_ignored

lib-program-client: Change program_client_run*() to return enum program_client_exit_status

Their behavior is clearer when result is expressed as enum than int.

lib-program-client: Rename "exit_code" to "exit_status"

exit_code typically refers to the numeric 0..255 value that processes exit
with.

lib-program-client: Move enum program_client_exit_code to public .h file

lib-lua: dlua-dovecot - Ref event on push

Matches with unref on __gc.

lib-lua: test-lua - Use bool for expected_isnum in check_tointegerx_compat

lib-lua: Define lua_isinteger as boolean function

lib-lua: Fix error reporting for script loading

lib-lua: Use dluaL_error for luaL_error

lib-lua: Fix luaL_error usage

lib-lua: Add dluaL_error

lib-lua: dlua-script - Add dlua_pushfstring

Uses full sprintf

lib-lua: dlua-wrapper - Constify extra functions

lib-index, lib-compression: Fix explicit integer wrapping

Using modulo instead of AND wrapped it one number too early.
Luckily these didn't break anything.

lib: abort() on unexpected exit()

This can especially be useful in catching bugs in Lua script that could
unexpectedly exit.

global: Replace exit() with lib_exit()

lib-http: http-client-peer - Fix segfault occurring upon backoff connect.

If peers decide to close and destroy immediately, the iteration in the shared
peer yields an invalid pointer for the next peer to connect. Fixed by preserving
pointer to next peer inside the loop.

lib: event filter - Avoid using event field wildcard matching when not needed

If the right hand side of an event field comparison is not using any
wildcards, we can compare using strcasecmp() instead of the more expensive
wildcard_match_icase().

This wildcard matching avoidance change speeds up matching quite a bit.
When the desired comparison is *not* a wildcard match (i.e., the filter is
'fieldname=abc'), microbenchmarks show at least a 11% speedup in filter
matching speed.  When the comparison includes a wildcard (i.e., the filter
is 'fieldname=abc*'), microbenchmarks show approximately 0.9% filer matching
slowdown.

Since there are so many non-wildcard matches in a typical filter, this is a
very good trade-off.

lib: event filter - Rename EVENT_FILTER_NODE_TYPE_EVENT_FIELD to denote wildcard matching

lib: event filter - Avoid using event name wildcard matching when not needed

If the right hand side of an event name comparison is not using any
wildcards, we can compare using strcmp() instead of the more expensive
wildcard_match().

This wildcard matching avoidance change speeds up matching quite a bit.
When the desired comparison is *not* a wildcard match (i.e., the filter is
'event=abc'), microbenchmarks show at least a 30% speedup in filter matching
speed.  When the comparison includes a wildcard (i.e., the filter is
'event=abc*'), microbenchmarks show approximately 0.4% filer matching
slowdown.

Since there are so many non-wildcard matches in a typical filter, this is a
very good trade-off.

lib: event filter - Rename EVENT_FILTER_NODE_TYPE_EVENT_NAME to denote wildcard matching

lib: Add wildcard_is_literal to check if a mask has * or ?

lib: test-env-util - Make static analyzer happy

lib-dict: dict-lua - Don't yield again until previous one has completed

lib-dict: dict-lua - Use absolute stack positions for arguments

acl: Reinitialize local acl object after creating mailbox

For INDEX layout this is needed to actually get the local mailbox path.
This fixes inheriting ACLs from parent folder for INDEX layouts.

lib: Add unit tests to env-util

lib-index: Use env_put() instead of putenv() directly

configure, lib: Remove workarounds for unsetenv()

It's in POSIX.1-2001 and available on all modern systems.

lib: env_put() - Use setenv() instead of putenv()

It's already in POSIX.1-2001 so it should be available everywhere.

lib, global: Change env_put() API to take name,value pair

lib-program-client, script: Use env_put_array()

lib: Add env_put_array()

util/dovecot-sysreport: Finish report even if doveadm calls fail

It can be useful to generate the report even when dovecot isn't running.

util/dovecot-sysreport --core: Support multiple core files

util/dovecot-sysreport --core: Check gdb existence earlier

Simplifies the next commit.

util/dovecot-sysreport --core: Change binary path to be optional

If not specified, detect it from the core file itself

global: Use array_idx_elem() where possible

lib: Add array_idx_elem()

This returns array_idx()'s value dereferenced, so it only works for
arrays of pointers.

Note that using *array_idx() does the same, but it won't compile if the
compiler doesn't have typeof() support.

lib: Remove array_swap()

It's not used anywhere, and it's a bit too weird.

global: Use array_foreach_elem() where possible

lib-fts: Use array_foreach_elem() where possible

submission: Use array_foreach_elem() where possible

stats: Use array_foreach_elem() where possible

lib-smtp: Use array_foreach_elem() where possible

lib-old-stats: Use array_foreach_elem() where possible

config: Use array_foreach_elem() where possible

plugins: Use array_foreach_elem() where possible

master: Use array_foreach_elem() where possible

lib-index: Use array_foreach_elem() where possible

lib-imap-client: Use array_foreach_elem() where possible

lib-http: Use array_foreach_elem() where possible

lib-dict: Use array_foreach_elem() where possible

imap: Use array_foreach_elem() where possible

doveadm: Use array_foreach_elem() where possible

director: Use array_foreach_elem() where possible

auth: Use array_foreach_elem() where possible

lib-storage: Use array_foreach_elem() where possible

lib: Use array_foreach_elem() where possible

lib: array.h - Update array_foreach*() comments

lib: array.h - Require C99

We're already requiring C99 elsewhere.

lib: array_foreach_elem() - Don't allow using it for sizes larger than a pointer

Dovecot doesn't use any types larger than the pointer size (e.g. long
double). Restricting it to max pointer size might prevent inefficient usage.

lib: array_foreach_elem() - Small optimization

Avoid having to recalculate the end of the array after each loop.
This makes the resulting assembler output similar to array_foreach().