lib-sieve: deprecated notify extension: Fixed various bool vs. int/pointer mixups.

Found with clang -Wstrict-bool.

lib-sieve: deprecated notify extension: Fixed various bool vs. int/pointer mixups.
dabacd9a · Stephan Bosch · Stephan Bosch · f95c6c0c · dabacd9a
Commit dabacd9a authored 8 years ago by Stephan Bosch Committed by Stephan Bosch 8 years ago
--- a/src/lib-sieve/plugins/notify/cmd-notify.c
+++ b/src/lib-sieve/plugins/notify/cmd-notify.c
@@ -283,12 +283,12 @@ static int cmd_notify_address_validate
 	if ( sieve_argument_is_string_literal(arg) ) {
 		string_t *address = sieve_ast_argument_str(arg);
 		const char *error;
-		bool result = FALSE;
+		int result;

 		T_BEGIN {
-			result = sieve_address_validate(address, &error);
+			result = ( sieve_address_validate(address, &error) ? 1 : -1 );

-			if ( !result ) {
+			if ( result <= 0 ) {
 				sieve_argument_validate_error(valdtr, arg,
 					"specified :options address '%s' is invalid for "
 					"the mailto notify method: %s",
@@ -299,7 +299,7 @@ static int cmd_notify_address_validate
 		return result;
 	}

-	return TRUE;
+	return 1;
 }

 static bool cmd_notify_validate