auth: Fix compiling crypt-blowfish with i386

We're not going to support any assembly code, so remove BF_ASM entirely.

doveadm-server: http: Fixed lingering connections after the request is sent.

Turns out conn->http_client is already NULL while doveadm_http_server_connection_destroy() is called.
This is because http_server_connection_unref() sets it to NULL;
Fixed by removing useless HTTP connection reference.

cassandra: Support "timestamp" type fields properly

Fixes setting them with prepared statements. Reading them never worked
earlier.

cassandra: Include "prepared" when logging about prepared statement queries

Mainly useful for debugging/testing.

cassandra: Fix using bigint types with unprepared statements

This reverts the code to not using the statements with binding at all.
Alternative fix would be to start using explicit int32 or int64 parameter
types, but that breaks backwards compatibility a bit.

libpassword: Deinitialize hash iterator in password_scheme_detect

Otherwise it will leak memory

ipc: Improve error logging when IPC server is disconnected

lib-dict-backend: Fix linking test-dict-sql

dict_drivers_register_builtin() wasn't found.

libpassword: Add BLF-CRYPT support independent of crypt

auth: Fix typo in blowfish name

lib-fs: Fix fs_wrapper_write_stream_finish() to work with async parent fs

This only became a problem with the previous fs-compress change.

login-common: Use HAproxy provided proxy.ssl information

If the connection is proxied via system that can terminate
ssl for us, such as HAproxy, use that information only.

lib-master: Support HAproxy TLVs

This allows passing in information about SSL and client
certificate(s) and other information from haproxy.

lib-master: Use pooled memory for haproxy

lib-master: Fix haproxy reading code to read data incrementally

libpassword: Add support for ARGON2 scheme

configure: Detect libsodium presence

acl plugin: avoid "'struct stat' declared inside parameter list" warning

fs-compress: Allow compress level 0 to skip compression

This can be useful when combined with the "maybe-" prefix, so Dovecot will
support reading compressed files without creating new ones.

Added "ULL" to the hex literals that needed it.

On 32-bit platforms with older compilers (e.g. gcc 4.2 on MacOS 10.6
running on a 1st-gen Core Duo) a 'long' is 4 bytes and the compiler does
not automatically use a 'long long' when needed, but instead generates
an error. e.g.:

libtool: compile:  /usr/bin/g++-4.2 -DHAVE_CONFIG_H -I. -I../../.. -I../../../src/lib -I../../../src/lib-mail -I../../../src/lib-index -I../../../src/lib-storage -I../../../src/plugins/fts -I../../../src/doveadm -I/opt/local/include/openssl -I/opt/local/include -I/opt/local/include/CLucene/ext -pipe -Os -arch i386 -D__STDC_LIMIT_MACROS -MT lucene-wrapper.lo -MD -MP -MF .deps/lucene-wrapper.Tpo -c lucene-wrapper.cc  -fno-common -DPIC -o .libs/lucene-wrapper.o
In file included from ../../../src/lib/lib.h:33,
                 from lucene-wrapper.cc:4:
../../../src/lib/byteorder.h:94: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:95: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:96: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:97: error: integer constant is too large for ‘long’ type
make[4]: *** [lucene-wrapper.lo] Error 1

Adding the 'ULL' to the end of the 16-digit hex literals that are used
to test the structure of 64-bit integers fixes this and avoids any
problem which could arise from the compiler using a 32-bit type for
those literals that could fit in 32 bites.

doveadm proxy: Don't crash if remote doesn't support log proxying

lib-storage: Add settings to configure lib-index optimization parameters

The defaults are expected to be pretty good, but these settings make it
easier to test whether other values might be more optimal.

lib-index: Add mail_index_cache_optimization_settings

lib-index: Add mail_index_base_optimization_settings

lib-index: Replace mail_index_set_log_rotation() with mail_index_set_optimization_settings()

This allows more easily adding optimization-related settings.

lib-dict: Remove test_dict_sql

It got accidentically added in 149299c7

lib-master: Allow a second log initialization after settings have been read.

This fixes logging with services that bypass the log service (lda or -L
parameter).

fs-compress: Support reading uncompressed input by prefixing compression format with "maybe-"

lib: Add istream-try

This can be used to automatically detect the underlying istream format from
a given list of choices.

lib: io_stream_set_error() - Allow one of the parameters to be the old error

global: Use check-local in Makefile.am instead of overriding check directly

This helps with dependency problems, like running "make check" in
lib-storage without "make" first would try to compile the test programs
too early and fail.

lib-sql, lib-ldap: Fix link dependencies

Fixes using the gold linker.

lib: ostream-multiplex - ignore flush return value

There's nothing that can be done about it at close() time. Silences static
analyzer warnings.

lib: ostream-multiplex - remove unnecessary flushes

lib: istream-multiplex - Minor optimization

There's no need to find channel when we already know it. This also helps
static analyzers to understand that req_channel can't be NULL.

ostream-multiplex: Check flush return value

global: Use i_close_fd instead of close

Some were missed, found by coverity

test-ostream-escaped: Check flush return value

Found by coverity

doveadm-server: Remove flush before multiplex

It's not really necessary, found by coverity

iostream-multiplex: Check return values in tests

doveadm: Return after destroying connection

Prevents NULL deferences, found by coverity.

ostream-multiplex: Unreference stream parent

Otherwise it won't get free'd.

doveadm: Add remote(host) prefix to all proxied logs

doveadm log test: Fix it to work again

Broken by 719abeb2088987f213a33a7dd1fe78958beaef03

lib: ostream-multiplex - set ostream_private.parent

Unlike with istream-multiplex, there are no issues with I/Os. Only the
parent stream will have the I/O. Using the default parent adds the
missing methods that otherwise would have needed to be implemented:
 - cork
 - flush_pending
 - switch_ioloop

lib: istream-multiplex - Forward i_stream_switch_ioloop() to parent

Most istreams do this because istream_private.parent is set to the parent
stream, but this can't be done with istream-multiplex. The main problem
with attempting to do the same with istream-multiplex is that the different
channels don't share the same I/O. Just because one channel received data
doesn't mean that other channels received any data. (It would be possible
to solve this by implementing a new method that allows overriding
i_stream_set_io(), but I'm not sure if that's a good idea either.)

acl: Fix compiler warning with -Wstrict-bool

doveadm-mail-crypt: Print existing folder key hash when aborting generate

acl: Fix checking create (k) permission in global acl-file

Just because the global ACL file hasn't changed since it was last refreshed
for another ACL object, it doesn't mean that those ACLs don't need to be
applied to this ACL object.

This didn't usually cause problems, because the initial ACL object refresh
was always done due to local-path refresh returning "needs a refresh".
The only exception was when acl_object_init_from_parent() was called,
because it added an empty non-NULL validity for the local-path, so the
"needs a refresh" wasn't returned. This happened only when trying to
CREATE or RENAME mailbox under a parent where user didn't have create
permissions.

This affected only when using a single global acl-file, not when using
global acl directory containing per-mailbox files.

acl: Cleanup - move code to a new acl_vfile_validity_has_changed()

acl: Fix checking whether global acl-file has changed

We always assumed that it was changed and re-read it.

dict-sql: dict doesn't support NULL values, so convert SQL NULLs to ""

lib-sql: Add comments about NULL values.

global: Use PRIdTIME_T and PRIxTIME_T format specifiers

master,stats: Use time_t instead of unsigned long to count seconds

m4: Add PRIdTIME_T and PRIxTIME_T format specifiers

global: Use PRI* macros and %zu instead of casting

pop3: Expand settings to fix rawlog_dir

Even if %variables weren't used in rawlog_dir, the path was always prefixed
with "0".

lib-settings: Add assert to help static analyzer

mail-crypt: test-mail-key - Fix potential crash on a failed test

Avoids "Uninitialized argument value" warning from static analyzer.

lib: test-mempool-alloconly - help static analyzer understand that pool is non-NULL

lib: istream-multiplex - remove dead assignment

doveadm-mail-crypt: Print existing userkey hash when aborting generate

lib: don't use foo_real()-style symbols in a header file

While there is precedent for symbols getting _real suffix to hide
implementation details, all the existing symbols were static until
the recent i_fd_close*() changes.  This commit makes this true again.

lib: move i_fd_close*() to fd-util.[ch]

lib: move fd_close_maybe_stdio() to fd-util.[ch]

lib: move fd_set_nonblock() to fd-util.[ch]

lib: move fd-close-on-exec.[ch] code into fd-util.[ch]

global: stop including fd-set-nonblock.h & fd-close-on-exec.h directly

lib: always include fd-close-on-exec.h & fd-set-nonblock.h via fd-util.h

lib: iostream-multiplex tests - Fix hangs by setting the pipe fds non-blocking

lib: istream-multiplex - Minor code cleanup

Avoid propagating the error twice, and avoid any confusion about what "got"
actually contains.

dict-ldap: Link directly to dict binary, unless --with-ldap=plugin was used

This is similar to how authdb_ldap plugin is built.

dict-ldap: Move from plugins/dict-ldap to lib-dict-backend

lib-dict-backend: Build test-dict-sql even without "make check"

There's no longer a dependency problem.

lib-dict: Move libdict_backend to lib-dict-backend

mail-crypt: Improve doveadm output

mail-crypt: Fix key generation handling

Userkey generation would not set all required fields.

director: When ring is synced, purge any pending "removed" directors

This allows adding a director back to the ring without having to wait for 30
seconds.

director: Rename director_delayed_dir_remove_timeout() to director_hosts_purge_removed()

This allows using it for other purposes without having a confusing name.

director: Fix potential panic when director is alone

If director is alone and it can't connect to other directors, it might crash
with:

Panic: file director.c: line 318 (director_set_ring_synced): assertion failed: (!dir->ring_synced)

global: Remove dead code

Found with clang static analyzer.

dict-memcached: Response status may not be included in enum memcached_response

This helps at least static analyzers figure out that other statuses are
possible, although unexpected.

lib-test: Use i_unreached() for marking unreachable code in test fatal handler

This avoids a warning with static analyzer.

auth: LDAP request queue has no size limit anymore - remove dead code

mbox: Remove unnecessary check from assert - move_diff is always negative here

zlib: zlib_mailbox_open_input() can't fail - remove dead code

lib-storage: imapc_mailbox_get_selected_status() can't fail - remove dead code

doveadm-server: Use i_close_fd

doveadm: Deliver remote logs over doveadm socket

doveadm: Refactor server/client code to support versioning properly

This way we can distinguish between old and new server side

doveadm-server: Refactor connection handshake and authentication

Simplifies next change

lib: Add multiplex stream support

This allows having multiple channels of data in single stream.

director: Allow doveadm director ring remove for the same director

Fixes:
Panic: file doveadm-connection.c: line 859 (doveadm_connection_cmd_run): assertion failed: (conn->dir->right == NULL && conn->dir->left == NULL)

director: Don't crash on doveadm director ring remove for unknown director

Ring syncing isn't started, so it shouldn't try to wait for ring sync.

Fixes:
Panic: file doveadm-connection.c: line 859 (doveadm_connection_cmd_run): assertion failed: (conn->dir->right == NULL && conn->dir->left == NULL)

director: Don't crash if DIRECTOR-REMOVE is received for itself

This triggers the director removal from the ring, which causes the
connection to be destroyed. But since we're still in the middle of handling
the connection it needs refcounting.

lib: istream-file - set blocking=TRUE automatically for blocking sockets

This fixes it to be correct with various doveadm commands.

lib: istream-file/unix - don't ignore EINTR for blocking istream reads

Just fail the istream read entirely. Although there's a small possibility
that this interrupt was unwanted and should be retried, it's more likely
that a blocking istream is hanging and admin wants to stop the process.
If the EINTR is ignored all the time, it's not possible to abort a
blocking read with ^C or anything else than SIGKILL.

lib: test-istream-unix - Use i_stream_set_blocking()

lib: Add i_stream_set_blocking()