lib-smtp: smtp-server-cmd-auth - Fix AUTH response error handling so that it stops...

Otherwise, it may try to read more data from the stream as the next AUTH
response, which causes an assert crash in the command parser later on. Even when
the parser finds no input from the stream, it will advance its state
towards AUTH response parsing, which is a problem when the next command is
subsequently being parsed.

Panic was:

Panic: file smtp-command-parser.c: line 438 (smtp_command_parse_next): assertion failed: (!parser->auth_response || parser->state.state == SMTP_COMMAND_PARSE_STATE_INIT || parser->state.state == SMTP_COMMAND_PARSE_STATE_ERROR)

diff --git a/src/lib-smtp/smtp-server-cmd-auth.c b/src/lib-smtp/smtp-server-cmd-auth.c
index 24cb55adef62eee77f978ee864fab2185bcb7ba8..0d8062caa7f658de7db5fd2df99241804c75fd8e 100644 (file)
--- a/src/lib-smtp/smtp-server-cmd-auth.c
+++ b/src/lib-smtp/smtp-server-cmd-auth.c
@@ -99,6 +99,7 @@ static void cmd_auth_input(struct smtp_server_cmd_ctx *cmd)
                        smtp_server_connection_debug(conn,
                                "Client sent invalid AUTH response: %s", error);
 
+                       smtp_server_command_input_lock(cmd);
                        switch (error_code) {
                        case SMTP_COMMAND_PARSE_ERROR_BROKEN_COMMAND:
                                conn->input_broken = TRUE;