director: Make sure HOST-RESET-USERS isn't used with max_moving_users=0

The reset command would just hang in that case. doveadm would never have
sent this, so this is just an extra sanity check.

diff --git a/src/director/doveadm-connection.c b/src/director/doveadm-connection.c
index 9f5ddb7266ddcd0d7d7f05753bea0e06386914c6..4957fc2297e2daab0ece50e56879d0d5f01a9006 100644 (file)
--- a/src/director/doveadm-connection.c
+++ b/src/director/doveadm-connection.c
@@ -590,7 +590,8 @@ doveadm_cmd_host_reset_users(struct doveadm_connection *conn,
        unsigned int max_moving_users = DEFAULT_MAX_MOVING_USERS;
 
        if (args[0] != NULL && args[1] != NULL &&
-           str_to_uint(args[1], &max_moving_users) < 0) {
+           (str_to_uint(args[1], &max_moving_users) < 0 ||
+            max_moving_users == 0)) {
                i_error("doveadm sent invalid HOST-RESET-USERS parameters");
                return DOVEADM_DIRECTOR_CMD_RET_FAIL;
        }