Bug whould occur when modifier was used and the value became "" at some point. In that case not all operands (modifiers) were read, causing the subsequent operation to start at an erroneous position.

Messed up signed/unsigned characters.

Patch by Timo Sirainen (modified).

Emerged when Sieve and ManageSieve were merged into Pigeonhole.

Added --disable-shared to DISTCHECK_CONFIGURE_FLAGS (in dovecot.m4) to prevent libtool from relinking the libraries.

Forgot to rewind mail stream before header parsing.
Wrapped mail stream is obtained long before parsing, so if it is used in the mean time, it must be rewound to the beginning.

Initializing multiple Sieve instances would hang.

Used errno instead of error variable.

Made normal compile error an info log item in the administrator log.

Forgot to increment count for the clone.

Setting skip_newline is useless when the connection is transfered to another process after successful login.

If a duplicate is detected the implicit keep is canceled, as though the redirect was successful. This prevents getting local deliveries.
The original SMTP recipient is used when it is available to augment the entry in the LDA duplicate database. This way, duplicates are only detected when (initially) addressed to the same recipient.
The main goal of the duplicate detection is mail loop prevention, but this also has the effect that mere duplicate deliveries are handled specially whithout a good reason. We should fix this in a future version.

Added LIBDOVECOT_STORAGE to library dependencies in src/lib-sieve/Makefile.am.

lib-sieve: vacation extension: Fixed determination of From: address for when sieve_vacation_dont_check_recipient is active.

lib-sieve: vnd.dovecot.duplicate extension: Only track duplicate when Sieve script executes successfully.

Forgot to initialize variable, causing a segfault at runtime.