imap: client_send_mailbox_flags() - Don't send PERMANENTFLAGS if not selecting and we have allow_new_keywords

lib-lua: test_io_lua() - Test return code from lua_script_init()

Found by coverity, 40458 Unchecked return value

lib-lua: dlua_dovecot_io_register() - Fix unbalanced stack at exit

lib-lua: dlua_script_init() - Remove unused variable

lib-lua: Add missing lua scripts location

without this make check fails when the build dir is different than the source dir

lib-compression: Add o_stream_create_deflate()

lib-index: Fix storing cache fields' last_used with 32bit big endian CPUs

Debian started in 2025 to build packages on 32-bit platforms with 64-bit
time_t support by default. With that change, dovecot suddenly fails to
build on 32-bit big endian architectures (e.g. hppa, powerpc) with those
testsuite errors:

test-mail-cache-fields.c:52: Assert failed: cache_field.last_used == priv->field.last_used && cache_field.decision == priv->field.decision
test-mail-cache-fields.c:67: Assert failed: cache_field.last_used == priv->field.last_used && cache_field.decision == priv->field.decision
test-mail-cache-fields.c:96: Assert failed: cache_field.last_used == priv->field.last_used && cache_field.decision == priv->field.decision
mail cache fields read-write ......................................... : FAILED

Change the existing code for big endian architectures to actually check
the size of time_t at compile time instead of hardcoding a check for
SIZEOF_VOID_P to fix the issue for 32- and 64-bit big endian
architectures.

Signed-off-by: Helge Deller <deller@gmx.de>
Cc: Timo Sirainen <timo.sirainen@open-xchange.com>
Fixes: 1ee84ba0659f ("lib-index: Fix storing cache fields' last_used with 64bit big endian CPUs")

auth: Fix LDAP SASL support

The settings code didn't see the necessary defines.

Based on patch by Jakob Haufe

Broken by 961275fdb54878fdfa4ee1b9f1a4f00e82bf4a83

lib-master: Don't use USER environment in log prefix

USER environment is not so generally supported anymore. The tools
supporting it should change the log prefix themselves if they want it.

Also this fixes inconsistency where some tools logged "tool(USER): "
or "tool: " log prefix depending on whether the configuration was read
by executing doveconf or not (because doveconf dropped the USER
environment).

lib-master: Always process import_environment for standalone programs

It wasn't processed if config was read via config socket or from config
cache.

lib-settings: Fix settings cache validity checks

inode and size checks were swapped, so settings caching didn't actually
work.

lib-settings, config: Panic if SETTING_DEFINE_LIST_END is missing from settings lists

auth: Terminate properly auth_oauth2_post_setting_defines list

Fixes:
Error: xoauth2: oauth2 failed: Local validation failed: auth_oauth2_fields settings: Failed to parse configuration: settings struct auth_oauth2_fields #1 key mismatch

auth: Fix empty certificate fingerprint error handling

Broken in db01107763ff3ad6afbf91f965ee46d2e3412b05

lib: cpu-count - fix compilation using musl

The macros, types and symbols CPU_* and cpuset_t are not exposed in
musl's sched.h unless _GNU_SOURCE is set.

Signed-off-by: Fabian Groffen <grobian@gentoo.org>

imapc: Automatically enable IMAP4rev2 if server has the capability

imapc: Ignore or disable RECENT if IMAP4rev2 is enabled

lib-http: test-http-client-errors - Increase client progress timeout

Accounts for timeouts sometimes occurring while running in Valgrind.

config: Add asserts to make static analyzer happy

imap: Remove dead assignment

lib-dict-backend: Fix building ldap as plugin

Broken by 49f2c4d3365de502d705db28376bf41927a1a900

configure: Do not modify user-provided CFLAGS variable

As per
https://www.gnu.org/software/automake/manual/html_node/User-Variables.html
CFLAGS is reserved for the user and should not be changed.

AM_CFLAGS is the right variable to use here.

global: Fix wrong inheritance of AM_CFLAGS/AM_CPPFLAGS in target_CFLAGS

target_CFLAGS overrides AM_CFLAGS, not AM_CPPFLAGS, thus $(AM_CFLAGS),
not $(AM_CPPFLAGS), is needed in target_CFLAGS.

m4/dovecot.m4: Use AM_CFLAGS instead of EXTRA_CFLAGS with --enable-ubsan

Mainly to make it consistent with how all other build options are used.

login-common: Copy haproxy fields to client pool

They are stack allocated in lib-master now.

lib-master: Copy haproxy provided fields before calling callback

Otherwise fields get lost, broken in 894610212596c35aade07a4d0af9d5e7fd6245c7

lib: strfuncs - Make t_memdup public

lda: Default mail_home=$HOME environment if not using userdb lookup

The previous code to do this was removed by
e57d5b9002f910c095ee5b55821395fcf1da016a

lda: Fix using USER environment if -d hasn't been specified

This became broken at some point.

auth: Allow ssl cert to be validated by fingerprint

login-common: Allow invalid client cert if ssl_server_request_client_cert=any-cert

auth: Support check_client_fp, check_client_cert_fp, check_client_pubkey_fp fields

When set, these are matched against provided fingerprints, and
must be present in the request.

Field check_client_fp matches either certificate or public key fingerprint.

lib-ssl-iostream: Change ssl_server_request_client_cert to enum

If set to no, client certificates are not asked or verified.
If set to yes, client certificates are asked and verified.
If set to any-cert, client certificates are asked but verified with
fingerprinting

lib-ssl-iostream: Allow missing ca if invalid certs are allowed

login-common: Add ssl_client_cert_fp and ssl_client_cert_pubkey_fp if configured

lib-auth-client: Add ssl_client_cert_fp and ssl_client_cert_pubkey_fp fields

lib-auth-client: Compare ssl_ja3_hash to NULL and not 0

ssl_ja3_hash is a pointer, not a number, so it's more correct
to compare it as pointer.

auth: Add ssl_client_cert_fp and ssl_client_cert_pubkey_fp fields

lib-ssl-iostream: Replace ssl_iostream_has_broken_client_cert() with ssl_iostream_has_client_cert()

Broken cert does not do anything different from valid cert, but we need
to know if there was cert in the first place.

lib-ssl-iostream: Add ssl_peer_certificate_fingerprint_hash setting

lib-ssl-iostream: Add ssl_iostream_get_peer_cert_fingerprint()

Provides fingerprint of peer certificate and it's public key
using the configured hash algorithm in context.

lib-ssl-iostream: Reformat iostream-openssl.c

lib-http: test-http-client-errors - Fix memory leak at sub-process deinit

Broken by dd6f9b3a3fab7d1c4b93d4f65086e8f019338b83

configure: Fix building without LDAP

lib-dcrypt: test-crypto - Store comparison key in DOVECOT format

PEM format is not stable between openssl versions

global: Fix warnings about uninitialized variables when compiling with -flto

These were only false positives, except for the unit test ones, which don't
really matter.

lib-dcrypt: Increase salt to 16 bytes for dovecot v2 keys

This makes it FIPS compatible.

lib-crypt: test-crypto - Enable testing more keys in test_load_v2_public_key()

fts-flatcurve: fts_flatcurve_xapian_db_populate() - Remove memory leak on returning error strings

Discovered on coverity

lib-lua/dlua-iostream: Remove unnecessary stream state assertion

lib-lua: test_io_lua() - Ensure a read error is handled correctly

lib-lua/dlua-iostream: dlua_i_read_common() - Optimize error handling

This also makes the error handling more explicit.

lib-lua/dlua-iostream: dlua_read_line() - Ignore result of i_stream_read()

Issued by coverity unchecked return value.

config: settings_add_include() - Close file descriptor if fstat() failed

Issued by coverity resource leak error.

submission-login: cmd_helo_reply() - Only check against SMTP UTF8 if feature is enabled

Issued by coverity deadcode error.

imapc: Do not forward ALERT response codes from sources with insecure connections

lib: hostpid - Use getaddrinfo instead of gethostbyname

gethostbyname is obsolote, so we should stop using it.

imap: imap-select: Disable SELECT UNSEEN response if IMAP4rev2 is enabled

lib-dns-client: test-dns-lookup - Add DLLIB for dlopen()

lib-http: Use explicit numbers for HTTP_CLIENT_REQUEST_ERROR_* codes

We're beginning on rely on these numbers not to change, so make the numbers
explicit to avoid accidental changes.

lib-dns src: Change dns_client_settings to config setting.

lib-http: Pass http_client_host to http_client_host_shared_lookup.

The following commit will make dns_lookup retrieving DNS settings from
event. http_client_host is need to pass http_client event to dns_lookup, the event
contains client-specific DNS settings.

lib-dns-client: Move struct dns_lookup initialization code into separate function.

Upcoming changes will add more usages of that.

lib-http: Set http_client_context.dns_lookup_timeout_msecs to default value unconditionally.

DNS lookup timeout will be made configurable in the following commit.

lib-smtp: Remove fallback to net_gethostbyname as not supported by upcoming changes.

lib-http: Remove fallback to net_gethostbyname as not supported by upcoming changes.

lib-dns-client: Move idle_timeout_msecs and cache_ttl_secs from dns_client_settings to dns_client_parameters.

As a preparation to make dns_client_settings configurable.

submission: Originate submission_client event from event having settings_root set.

lib-http: Fix bug with removal of http_client_request from delayed_requests.

Setting release_time to 0 in the beginning of
http_client_queue_submit_now() prevents removing of the request from
delayed_requests in case of failures.

lib-dns-client, lib-http: remove ioloop from struct dns_client_settings.

src: Remove event_parent from struct dns_client_settings.

src: Rename dns_lookup_settings to dns_client_settings.

config: Improve comment about self-referencing keys expansion.

config: Cleanup config_write_keyvariable() function.

config: Fix expanding $SET:key inside filters.

expand_parent parameter was added to config_get_value() in
92a9ff5e799adf9bffb8831576d84eb84b8ef09a, so its behavior was likely
useful only in the earlier hierarchical configuration.

pop3: Free var_expand program in parse_uidl_keymask()

Forgotten in 37c9d7802a6713443bcab3cfa3d01a57b154c4c1

lib-var-expand: Unload loaded crypt module at exit

Forgotten in 73ae2a9e45144f05a179d17d9b201226d93b81a0

imap: cmd_append_handle_args() - Fix calling i_stream_create_limit() twice

f12df6e0eed634aa4fe92c225b54ef5ee965b35c accidentally added a second
call to i_stream_create_limit() was which led to leaking events and
wrong iostream->refcount in other plugins

config: Fix config filter sorting order

It was somewhat luckily working in glibc, but broke with e.g. musl libc.
This caused a crash at startup.

Broken by 5acdd2b97ed6092d1f0204bdeb3e418180519370

imap: If IMAP4rev2 is enabled by the client also enable QRESYNC

lib-lua: Add base64 encode/decode support

lib-ssl-iostream: Fix OpenSSL 3.0 provider/engine preference logic

lib, lib-language: Use perl found by configure

This allows better errors if perl wasn't found.

lib-settings: Distribute settings-history-core.h

It does not need to be regenerated every time.

lib-settings: Use python3 found by configure

This allows better errors if python3 wasn't found.

config: Change config_dump_full_context.config const

config: Change struct config_parsed parameters const for most functions

lib-storage: Add explicit "set/" userdb prefix for settings

If the setting name is unknown, it's an error. At least for now this prefix
is optional.

lib-settings: Fail settings lookups if there are unknown setting overrides

This is now safe, because userdb no longer adds unknown settings to
overrides.

lib-storage: Don't add userdb fields to settings if they're unknown

Userdb extra fields can be used for other purposes than as being settings.
Now that we know all setting names, don't add them to settings overrides
if we know they're not known settings.

lib-settings: Add settings_key_exists()

lib-settings: Remove unused settings_override.filter_event

lib-settings, config: Remove unused override event filter

lib-settings, config: Fix and simplify override order handling

The order is now based on the number of named list filter elements in the
override path vs binary config's event filter.

This also happens to fix at least some issues where @group include was
wrongly overriding userdb fields.

lib-settings: settings_mmap_apply_filter() - Refactor to use settings_mmap_event_filter parameter

lib-settings: Introduce struct settings_mmap_event_filter

lib-settings: Optimize skipping overrides that aren't in the looked up setting_parser_info

lib-settings: Remove unnecessary settings_override.path_element_count

It has not been used since 09e551901eeb21bd8fc7b07eb67d80fd43fbb71e

lib-settings: Build override settings filters using "all settings" hash table in binary config

If there is no binary config (= unit tests), require that all the settings
infos have been explicitly registered.

This simplifies the override handling code and allows further optimizations.

lib-settings: Move settings override "*" handling out of the loop

This simplifies the code a bit.