Doveadm-sieve plugin may be broken at this point; not tested so far.

Most important change is that mail service_user was not freed.

Would cause a segfault.
Reported by Coverity.

Handling of edit_mail_header_find() result was incoherent.
Problem reported by Coverity.

Reported by Coverity.

Switch statement item is never reached and return value was bool instead of int.
Problem reported by Coverity.

Forgot a few '*' there.
Reported by Coverity.

Forgot break in switch statement.
Reported by Coverity.

lib-sieve: program client: Forgot break in switch statement for handling of response from script service.
Reported by Coverity.

Reported by Coverity.

Reported by Coverity.

Parallel to identical Dovecot change.

Parallel to identical Dovecot change for imap and pop3.

Reported by Coverity.

Reported by Coverity.

Found by Coverity.

A va_copy creates a initialised va_list, as if a va_start had been done on it.
Therefore, pedantically, a va_end should also be done on it. On most platforms
this is a no-op, and for those where it isn't, the pairing is important.

This change matches parallel change in Dovecot. Also signalled by Coverity.

Flags weren't actually updated.

NOTE: Dovecot doesn't currently MIME-encode these in the headers.

Also cleaned up the surrounding code a bit.

This way, a duplicate is only marked when the script execution succeeds, as required in the specification.

Presence of :addresses argument was represented incorrectly.

SMTP API now supports adding multiple recipients to a single SMTP transaction. This makes enotify/notify more efficient, since only one message is actually sent.

lib-sieve: Fixed sieve_result_global_log_error to log only as i_info in administrator log (syslog) if executed from multiscript context.
Previously, this would work correcly only from the user's personal script and not from multiscript (sieve_before/sieve_after).

Backwards compatibility is provided for vnd.dovecot.duplicate.
Still need to fix the constraint that it must not track duplicates from failed Sieve executions.

This function now returns the binary prefix (file path until '.svbin) rather than just the path of the directory where the binary would be stored.

Some programs can be instructed to output special status information to an extra fd other than stdout or stderr.
Also added an option to drop stderr output from program to /dev/null.