From 01f7d9f3b2d107c366e626d4bf4f38b6cd20b62d Mon Sep 17 00:00:00 2001
From: Stephan Bosch <stephan@rename-it.nl>
Date: Sun, 21 Aug 2016 11:23:03 +0200
Subject: [PATCH] lib-sieve: mailbox extension: Fixed invalid use of
 sieve_ast_stringlist_map().

Mixed up bool and int return types.
Found with clang -Wstrict-bool.
---
 src/lib-sieve/plugins/mailbox/tst-mailboxexists.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/lib-sieve/plugins/mailbox/tst-mailboxexists.c b/src/lib-sieve/plugins/mailbox/tst-mailboxexists.c
index 9aa3a9281..8d538111b 100644
--- a/src/lib-sieve/plugins/mailbox/tst-mailboxexists.c
+++ b/src/lib-sieve/plugins/mailbox/tst-mailboxexists.c
@@ -85,7 +85,7 @@ static int tst_mailboxexists_mailbox_validate
 		}
 	}
 
-	return TRUE;
+	return 1;
 }
 
 static bool tst_mailboxexists_validate
@@ -108,8 +108,8 @@ static bool tst_mailboxexists_validate
 	valctx.valdtr = valdtr;
 	valctx.tst = tst;
 
-	return sieve_ast_stringlist_map
-		(&aarg, (void*)&valctx, tst_mailboxexists_mailbox_validate);
+	return ( sieve_ast_stringlist_map(&aarg,
+		(void*)&valctx, tst_mailboxexists_mailbox_validate) >= 0 );
 }
 
 /*
-- 
GitLab