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.

Fixes ubsan runtime error.

This is currently not used, but prevents mishaps once such code is written.

This is not currently used, but the absence of the call could cause memory leaks
in the future.