From b7a7e767dbb9e831f696be5e4e659380ad5c8d24 Mon Sep 17 00:00:00 2001
From: Timo Sirainen <timo.sirainen@open-xchange.com>
Date: Mon, 29 Apr 2019 14:57:25 +0300
Subject: [PATCH] lib-sieve: Fix minor logic bug in handling duplicate keep
 actions

This didn't seem to result in any visible bugs though.
---
 src/lib-sieve/sieve-result.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/lib-sieve/sieve-result.c b/src/lib-sieve/sieve-result.c
index a6694f712..7998a11ad 100644
--- a/src/lib-sieve/sieve-result.c
+++ b/src/lib-sieve/sieve-result.c
@@ -407,9 +407,7 @@ _sieve_result_add_action(const struct sieve_runtime_env *renv,
 				return sieve_result_side_effects_merge(
 					renv, &action, raction, seffects);
 			}
-
-		}
-		if (act_def != NULL && raction->action.def == act_def) {
+		} else if ( act_def != NULL && raction->action.def == act_def ) {
 			instance_count++;
 
 			/* Possible duplicate */
-- 
GitLab