doveadm-http: Fixed crash on unknown commands.

Also if command had already failed, don't override the existing method_err with 403.
dovecot · Feb 29, 2016 · ec862f5 · ec862f5
1 parent a81dccf
commit ec862f5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/doveadm/client-connection-http.c b/src/doveadm/client-connection-http.c
@@ -279,7 +279,7 @@ doveadm_http_server_command_execute(struct client_connection_http *conn)
 	struct doveadm_cmd_context cctx;
 
 	/* final preflight check */
-	if (!doveadm_client_is_allowed_command(conn->client.set, conn->cmd->name))
+	if (conn->method_err == 0 && !doveadm_client_is_allowed_command(conn->client.set, conn->cmd->name))
 		conn->method_err = 403;
 	if (conn->method_err != 0) {
 		if (conn->method_err == 404) {