Used the stream's cur_header field, rather than the local variable.
Problem found by Coverity.

The merge_from_parent() function may not actually read anything, potentially leaving the stream buffer unassigned.
Problem found by Coverity.

This makes the implementation easier to understand and amend once Dovecot APIs changes.
This implicitly makes the implementation suitable for the new istream snapshot handling.

Added assertions on subtractions.
Restructured the code for clarity.
Added comments.

Ported Dovecot's istream-jsonstr for ManageSieve syntax.

Call master_login_init() before master_service_init_finish(), which frees
all the data stack done in initialization.

This didn't normally cause any visible problems, because data stack wasn't
currently being used in a way that the strings were invalidated. However,
it was causing failures if --enable-devel-checks was used.

Patch by Andreas Schulze.

Patch by Andreas Schulze.

Patch by Andreas Schulze.

lib-sieve: enotify plugin: mailto method: Make sure from header is set to a usable address and not (null).

It's replaced with fd-util.h which is automatically included now.

It's replaced with fd-util.h which is automatically included now

Mirrors imap fix in Dovecot: 42149f48624b82fdf9631c256497580154c2e412

It's impossible for the command's pointer to be NULL at this point.
Previously, the command_find() would have returned NULL, but this check
presumably short-circuits that search in the trivial case, so has some
real use.

Problem now found by GCC 7.

Problem found by GCC 7.

lib-sieve: sieve-result: Added /* fall through */ comment to mark intentional switch case fall through.

Addresses GCC 7 warning.

plugins: Migrated from lib-lda/smtp-client.h to using lib-smtp/smtp-submit.h for message submission.

plugins/imapsieve: Initialize struct imap_sieve with full struct client, rather than individual fields.

lib-sieve: redirect action: Always set X-Sieve-Redirected-From header to sieve_user_email if configured.

Set it to the final recipient only when there is an envelope and sieve_user_email is unconfigured.

lib-sieve: vacation extension: Allow ignoring the envelope sender while composing the "To:" header for the reply.

This adds a new setting "sieve_vacation_to_header_ignore_envelope".
With this setting enabled, the "To:" header is always composed from the first "Sender", "Resent-From" or "From" header found in the message (in that order).
Normally, the "To:" header is composed from the address found in the "Sender", "Resent-From" or "From" headers that is equal to the envelope sender.
If none is then found, the bare envelope sender is used.
The new setting allows ignoring the envelope, which is useful e.g. when SRS is used.

The sequence was broken when the LDAP script attribute does not exist; i.e., when the script is not found.
The NOT_FOUND error is returned at a later instance than normal fs-based scripts, explaining why this problem does not normally occur.

LDA Sieve plugin: Restructured execution of scripts by putting execution of individual scripts in separate function.

This is needed to register the implicit side effect for the internal flag variable.