lib: net_*() - Make sure sockaddr_union is always cleared

Fixes scan-build warnings

lib: data-stack - Make STACK_BLOCK_DATA() behavior clearer

Otherwise scan-build thinks that it's pointing outside valid memory in the
struct and gives an error:

"Memory set function accesses out-of-bound array element"

global: Change CALLBACK_TYPECHECK() to used with subtraction

This is because clang's -Wstring-plus-int gave warnings when using
"string" + CALLBACK_TYPECHECK(), but there are no warnings when using
"string" - CALLBACK_TYPECHECK(). It would have been enough to change
only the string parameters to use "-", but for consistency this commit
changes all of them.

lmtp: local: Make the local_deliver client vfunc responsible for submitting the reply.

This gives plugins more power to override delivery behavior.

lmtp: Load all globally-defined mail_plugins at client connect.

Before, this was not performed until after a user lookup was performed for a
recipient. That is not acceptable when plugins need to modify the LMTP protocol
itself.

lmtp: Install header files.

This allows creating external lmtp plugins.

lmtp: Rename commands.h to lmtp-commands.h.

lmtp: Rename client.h to lmtp-client.h.

lmtp: Make include directory available to external plugins.

lmtp: local: Create a client vfunc for local delivery.

This way,for example, the struct mail_deliver_context passed to mail_deliver()
can be adjusted by plugins.

lib-smtp: server: Add support for adding (non-standard) extra capabilities.

lib-smtp: syntax: Add new smtp_ehlo_params_are_valid().

This new version checks whether the parameters contained in the provided string
array are all valid EHLO parameters.

lib-smtp: syntax: Remame smtp_ehlo_params_are_valid() to smtp_ehlo_params_str_is_valid().

Moves it out of the way for a function that is better suited for this name;
i.e., it operates on an array of parameters.

lib-smtp: server: Add smtp_server_reply_ehlo_add_params().

lib-smtp: server: EHLO command: Move composition of individual reply lines to separate functions.

lib-smtp: server: EHLO command: Move composition of default EHLO reply to a separate public function.

lmtp: Add support for module contexts to recipient.

lmtp: Add support for module contexts to client.

lmtp: Implement client vfuncs for transaction start and free.

lmtp: Implement client vfuncs for SMTP transaction commands.

lmtp: RCPT command: Allocate lmtp_recipient object immediately.

lmtp: DATA command: Add data size to client state.

lmtp: DATA command: Put the data input stream in the client state rather than the transaction context.

lmtp: Implement basic client vfuncs.

Currently, only client_destroy can be overriden.

lmtp: Add hook for client_create().

lmtp: Remove obsolete declaration from lmtp-common.h.

lmtp: Make lmtp-common.h the primary include.

This better matches other services.

lmtp: Rename main.h to lmtp-common.h.

This better matches other services.

lmtp: Rename lmtp-common to lmtp-recipient.

fts-solr: Remove obsolete break-imap-search setting

lib-program-client: Add more detailed debug logging.

lib-program-client: Use event logging.

This also makes the overall log messages better match similar messages produced
by other parts of Dovecot.

lib-program-client: Use i_close_fd() rather than close() when error is not supposed to happen.

lib-program-client: Make debug mode for test suite optional.

It is now disabled by default.

lib-program-client: local: Cleanup any kill timeout while program client is destroyed.

lib-program-client: local: Make sure the child is dead once the program client is destroyed.

lib-program-client: Simplify disconnect handling.

Perform all of the cleanups in a separate shared function.
Always close all fds, since there is nothing left to be read or written at
disconnect.

lib-program-client: remote: Don't try to read remaining input at disconnect.

This code is no longer relevant. At disconnect, there is nothing left to be read
by definition.

lib-program-client: Use iostream-pump for streaming data from program.

lib-program-client: Destroy timeout once client is passively disconnected.

lib-program-client: remote: Initialize program_client_istream with parent FD.

This way, io_add_istream() works as expected.

lib-program-client: local: Stop immediately when child program without I/O ends.

Particularly, don't perform this by calling the input handler.

lib-program-client: Change return type of program_client_connected() to void.

Returning an int status code is no longer useful.

lib-program-client: Use iostream-pump for streaming data towards program.

lib-program-client: Initialize dot streams right after the connection is established.

This avoids creating the dot streams in the input/output handlers, which makes
the code cleaner and easier to port to using iostream-pump.

lib-program-client: Use iostream-temp rather than istream-seekable for seekable output.

This considerably simplifies the code.

lib-program-client: test-program-client-net: Read the client input stream to EOF after the dot stream is closed.

The low-level file stream is not necessarily read to EOF when the body dot
stream sees EOF.

lib-program-client: test-program-client-unix: Properly recognize the noreply script protocol option.

Somehow the equivalent test-program-client-net code does not have this problem.
And somehow this test did not fail before.

lib-program-client: test-program-client-net: Destroy test client upon error.

This prevents infinite input event loop when something goes wrong.

lib-program-client: test-program-client-unix: Destroy test client upon error.

This prevents infinite input event loop when something goes wrong.

lib-program-client: test-program-client-net: Handle test client connections non-blocking.

lib-program-client: test-program-client-unix: Handle test client connections non-blocking.

lib-program-client: test-program-client-net: Create body streams only once in test_program_input_handle().

lib-program-client: test-program-client-unix: Create body stream only once in test_program_input_handle().

lib-program-client: test-program-client-net: Match client input handling code with test-program-client-unix.

lib-program-client: test-program-client-unix: Further fix error handling in test_program_input().

lib-program-client: remote: Always shut down the output when all is written.

Before, this step was omitted when dot streams were used. However, shutting down
the output is necessary no matter what. Otherwise, the receiving end will not
see that no more input will follow, causing deadlock.

lib-program-client: Make sure program client callback is called only once.

lib-program-client: remote: Use struct program_client_remote as parameter where possible.

Avoids the need to cast it everywhere.

lib-program-client: remote: Call program client variable "prclient" consistently.

lib-program-client: local: Call program client variable "plclient" consistently.

lib-program-client: Reformat test-program-client-unix.c.

lib-program-client: Reformat test-program-client-net.c.

lib-program-client: Reformat program-client-private.h.

lib-program-client: Reformat test-program-client-local.c.

lib-program-client: Reformat program-client-remote.c.

lib-program-client: Reformat program-client-local.c.

lib-program-client: Reformat program-client.h.

lib-program-client: Reformat program-client.c.

lib: iostream-pump: Properly implement running a pump with one of the streams blocking.

Having both streams blocking is not useful and that is now explicitly forbidden.

lib: iostream-pump: Add iostream_pump_destroy().

This functions always stops the pump and dereferences the streams before
dereferencing the pump itself.

lib: iostream-pump: Make iostream_pump_unref() implementation match other similar code.

This also means that iostream_pump_unref(NULL) is now a no-op.

lib: iostream-pump: Use refcount as name and int as type for reference count field.

This is what the rest of Dovecot uses.

lib: Reformat test-iostream-pump.c.

lib: Reformat iostream-pump.c.

lib: Reformat iostream-pump.h.

lib-index: Make sure tail_offset is updated after calling expunge handlers

Since it's a bit troublesome to track whether expunge handlers were actually
called, we can just assume that as long as syncing sees expunges, they were
called (and thet most likely were).

This fixes a bug where dovecot.index.cache header's record_count kept
shrinking and deleted_record_count kept increasing after each sync.
These values were only used to determine when to compress the cache, so
its only effect was that the cache could have been compressed earlier
than necessary.

lib-dict: dict-sql - Add username as parameter

SQL statement caching works better when the username is provided
as parameter instead of part of the query, since the caching
can now cache same statements despite username changes.

lib-dns: Remove unused Valgrind disable

lib: Add test-fd-util.c to test i_close() panic message

cassandra: Handle cass_future_set_callback() calling callback immediately

The previous code always assumed it would be called from another thread, but
due to race conditions this wasn't true always. The lookup could have
already finished by the time cass_future_set_callback() was called, in which
case it called the callback immediately.

If the callback was called from the main thread, it still wrote to the
fd_pipe. Because this was a blocking write, it was possible that the pipe
ended becoming full, which caused the whole dict process to hang.

cassandra: Move cassandra_callback_run() earlier

Simplifies the next commit.

lib: ostream-file - Use TCP_NODELAY when uncorking, not corking

lib: ostream-file - Don't try to set TCP options after stream is closed

Flushing the buffer could close the stream and the fd.

lib: ostream-file - Ignore ENOPROTOOPT error also when setting TCP_NODELAY

This happens in some OSes.

lib: ostream-file - Don't use TCP_CORK or TCP_NODELAY for UNIX sockets

Either they fail or just don't do anything.

virtual: Fix calling mail_add_temp_wanted_fields() before mail is set

Fixes:
Panic: file virtual-mail.c: line 123 (backend_mail_get): assertion failed: (bbox != NULL)

virtual: Fix mail.add_temp_wanted_fields() to handle headers correctly

The header that was passed to backend contained wrong index numbers, causing
wrong headers to be used.

lib-storage: Update mail_add_temp_wanted_fields() comment

lib-storage: Move code to a new mailbox_header_lookup_merge()

lib: ostream-file - Don't try to set TCP_NODELAY for non-socket fds

Also log if there are any unexpected errors.

lib-storage: Don't try to set attachment keywords if bodystructure wasn't parsed

This was broken in 057028e3f5567e21c964005ffccda1055fa86508 as there are
code paths leading here that omit bodystructure processing.

Fixes Panic: file index-mail.c: line 1150 (index_mail_parse_body_finish): assertion failed: (mail->data.parsed_bodystructure)

lib: Make sure uncorking ostream-file flushes network output immediately

TCP_CORK and TCP_NODELAY are separate things, but we'll treat them similarly
for our ostream corking API. It was always intended that all data written
inside corking would be delayed until uncork, and immediately sent
afterwards.

This change removes unnecessary extra 40ms delays, which may happen when
proxying.

doveadm: Add error_r to server_connection_create()

doveadm-dsync: Always cleanup after connection

Go through the same error handling path every time. Fixes
Panic: file ioloop.c: line 826 (io_loop_destroy): assertion failed: (ioloop == current_ioloop)

This happened because the ioloop wasn't destroyed
on error.

doveadm-dsync: Move actual command execution to dsync_server_run_command

Simplifies next commit

lib-storage: Attachment flag detection shouldn't cause mail body to be opened

Fetching cached mail headers with
mail_attachment_detection_options=add-flags-on-save caued IMAP BODYSTRUCTURE
to be fetched as well. If it wasn't already cached, this caused opening the
mail body to generate it. Also the result wasn't added to cache, so the
inefficient behavior continued.

dict: Make sure connection IO is stopped when client is disconnected

I'm not sure if this fixes anything, but it makes the behavior closer to
what it was before 2ff2da52146609f4459bd0f7fd603e13400cb85e.

dict: Fix client connection reference counting

If the client disconnected while commands were still running, the destroy
callback could have been called multiple times.

Broken by 2ff2da52146609f4459bd0f7fd603e13400cb85e

Fixes:
Panic: file dict-connection.c: line 149 (dict_connection_unref): assertion failed: (array_count(&conn->cmds) == 0)

lib-storage: maildir: maildir-copy - Fix omission of keywords while copying message with hardlinking.