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 2b61787327ff60371fb5378cd5c3bed866a98f0c..e622691fa0dd5a4c1933c62349af3d33ae921b31 100644 (file)
--- a/src/director/doveadm-connection.c
+++ b/src/director/doveadm-connection.c
@@ -592,7 +592,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;
        }