lib-sieve: sieve-result - Indicate in sieve_result_transaction_finalize() debug message whether actions were committed.

lib-sieve: sieve-result - Indicate in sieve_result_transaction_execute() debug message whether actions were executed.

lib-sieve: sieve-result - Assert that implicit keep is executed in sieve_result_implicit_keep_finalize().

lib-sieve: sieve-result - Fix resource leak occurring when implicit keep is executed before temporary failure at commit.

In the commit phase the implicit keep was never finalized, meaning that it was
not rolled back and thus not cleaned up properly. This leads to a memory leak
and a mailbox reference leak. This in turn causes an assert crash at the end
of delivery when the mail user is destroyed.

It will be executed in the commit phase if necessary; don't do it early; it will
only be rolled back.

lib-sieve: sieve-result - Move temp failure status checks into sieve_result_implicit_keep_finalize().

lib-sieve: sieve-result - Move temp failure status checks into sieve_result_implicit_keep_execute().

lib-sieve: sieve-result - Fix omission of action finalization happening upon result execution failure.

This occurred because the code that checks for a deferred keep action in
sieve_result_implicit_keep_execute() could end up "finding" an action that
wasn't actually a deferred keep. That action was then erroneously promoted to
the FINALIZED state, which meant that commit/rollback was never executed.

It is used like that everywhere and ubsan complains about passing char to
unsigned char parameter.

This had no real purpose and could cause NULL pointer dereference.

Found by Clang scan-build.

lib-sieve: sieve-binary-file - Fix potential NULL pointer dereference in sieve_binary_file_update_resource_usage().

Found by Clang scan-build.

This prevents wrapping the integer value and fixes an ubsan complaint.