plugins: doveadm-sieve: doveadm-sieve-cmd-put - Fix script memory leak occurring when saved script is also activated.

lib-sieve: sieve-storage - Fix script memory leak occurring when saved script is implicitly activated.

Add NEWS from releases to master branch too

@@
expression E;
@@

- if (hash_table_is_created(E)) {
-       hash_table_destroy(&E);
- }
+ hash_table_destroy(&E);

lib-sieve: redirect action: Assert that dupeid is not NULL when act_redirect_get_duplicate_id() is successful.

Addresses scan-build report.

lib-sieve: redirect action: Fix lack of NULL checking in new X-Sieve-Redirected-From header comparisons.

Problem found by scan-build.

In this context, it can never be NULL and the check confuses Coverity.

In this context, it can never be NULL and the check confuses Coverity.

Also check the X-Sieve-Redirected-From header for our own e-mail addresses. This
header is added by the redirect action itself and in a mail loop it would see
that same header with that same content. This is less reliable than the other
mail loop detection (sender may set such a header), so, unlike the existing loop
detection based on the duplicate db, the implicit keep is not canceled when the
new loop detection is triggered.

Serves as an abbreviation.

It was errorneously using the (potentially) modified mail struct for error
reporting.

This prevents users from messing with redirect loop detection.

plugins: sieve-extprograms: Use sieve_ prefix consistently for sieve elements visible as exported symbols.

This fixes a symbol clash with the imap-filter-sieve plugin. Both modules had
the symbol "cmd_filter".

plugins: imap-filter-sieve: Properly discard the originally stored message when a modified version is stored by Sieve.

The Sieve interpreter can return a flag that indicates whether the original
message should be kept, but that was not actually being used.

plugins: imapsieve: Properly discard the originally stored message when a modified version is stored by Sieve.

The Sieve interpreter can return a flag that indicates whether the original
message should be kept, but that was not actually being used.

Instead use the detected ones.

Otherwise stats process startup can get into a loop.

Coverity complains about last_script possibly being empty in
imap_sieve_filter_run_scripts(), which is actually not possible, since the
function would not be called if there is no script to execute. Added assertion
to make that clear.

The debug message pertains to skipping secondary scripts that failed to
compile. This debug message will not actually be triggered in the current
implementation, but this will become problematic in the future.

Problem found by Coverity.