diff --git a/src/lib-sieve-tool/mail-raw.c b/src/lib-sieve-tool/mail-raw.c
index 8b6c395c5fe6e662cd7325d4f6da778f817922c6..4a9e7063c5cc4541d83b07002c3e8a999a2c7241 100644
--- a/src/lib-sieve-tool/mail-raw.c
+++ b/src/lib-sieve-tool/mail-raw.c
@@ -47,7 +47,7 @@ static const char *wanted_headers[] = {
  */
 
 struct mail_raw_user {
-	struct mail_namespace *ns;;
+	struct mail_namespace *ns;
 	struct mail_user *mail_user;
 };
 
diff --git a/src/lib-sieve/plugins/mailbox/tag-mailbox-create.c b/src/lib-sieve/plugins/mailbox/tag-mailbox-create.c
index 69ac00bbf836c613f1fec439cec82323bdefac22..923697f59cd9aeec7d020e6e7d61da5379aa1219 100644
--- a/src/lib-sieve/plugins/mailbox/tag-mailbox-create.c
+++ b/src/lib-sieve/plugins/mailbox/tag-mailbox-create.c
@@ -44,7 +44,7 @@ static void seff_mailbox_create_print
 static bool seff_mailbox_create_pre_execute
 	(const struct sieve_side_effect *seffect, const struct sieve_action *action, 
 		const struct sieve_action_exec_env *aenv, void **se_context, 
-		void *tr_context);;
+		void *tr_context);
 
 const struct sieve_side_effect_def mailbox_create_side_effect = {
 	SIEVE_OBJECT("create", &mailbox_create_operand, 0),
diff --git a/src/lib-sieve/sieve-result.c b/src/lib-sieve/sieve-result.c
index 41a15a8dfd0548dac9cccad120f830a0a6c3e2b0..529e163d65c83b9ea6c6f1f73bf9c095989fb154 100644
--- a/src/lib-sieve/sieve-result.c
+++ b/src/lib-sieve/sieve-result.c
@@ -1295,7 +1295,7 @@ struct sieve_side_effects_list *sieve_side_effects_list_create
 	list->last_effect = NULL;
 	
 	return list;
-};
+}
 
 void sieve_side_effects_list_add
 (struct sieve_side_effects_list *list, const struct sieve_side_effect *seffect)		
diff --git a/src/testsuite/cmd-test-message.c b/src/testsuite/cmd-test-message.c
index 9ec92c27a336ecb73fbce293d90a5e1df1a7b400..95d6fa1059d3246b8241d878c72125170dac74ad 100644
--- a/src/testsuite/cmd-test-message.c
+++ b/src/testsuite/cmd-test-message.c
@@ -299,7 +299,7 @@ static int cmd_test_message_smtp_operation_execute
 (const struct sieve_runtime_env *renv, sieve_size_t *address)
 {
 	sieve_number_t msg_index;
-	unsigned int is_test = -1;
+	unsigned int is_test = 0;
 	bool result;
 	int ret;
 
@@ -359,7 +359,7 @@ static int cmd_test_message_mailbox_operation_execute
 {
 	string_t *folder;
 	sieve_number_t msg_index;
-	unsigned int is_test = -1;
+	unsigned int is_test = 0;
 	bool result;
 	int ret;