diff options
author | Igor Sysoev <igor@sysoev.ru> | 2007-02-14 20:45:19 +0000 |
---|---|---|
committer | Igor Sysoev <igor@sysoev.ru> | 2007-02-14 20:45:19 +0000 |
commit | 65a7b8f988d8a766014d58e5d73d30b0199b1656 (patch) | |
tree | 772a45d991875e9d1cba2dbc4ecc2be8f5790df9 /src | |
parent | 722231f40788d1243bf302227ed7b6fdfeaea492 (diff) | |
download | nginx-65a7b8f988d8a766014d58e5d73d30b0199b1656.tar.gz nginx-65a7b8f988d8a766014d58e5d73d30b0199b1656.zip |
return error for too long commands
Diffstat (limited to 'src')
-rw-r--r-- | src/imap/ngx_imap_handler.c | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/src/imap/ngx_imap_handler.c b/src/imap/ngx_imap_handler.c index db0b8cef4..dad0994da 100644 --- a/src/imap/ngx_imap_handler.c +++ b/src/imap/ngx_imap_handler.c @@ -1212,6 +1212,7 @@ ngx_imap_read_command(ngx_imap_session_t *s) { ssize_t n; ngx_int_t rc; + ngx_str_t l; n = s->connection->recv(s->connection, s->buffer->last, s->buffer->end - s->buffer->last); @@ -1240,10 +1241,22 @@ ngx_imap_read_command(ngx_imap_session_t *s) rc = ngx_imap_parse_command(s); } - if (rc == NGX_AGAIN - || rc == NGX_IMAP_NEXT - || rc == NGX_IMAP_PARSE_INVALID_COMMAND) - { + if (rc == NGX_AGAIN) { + + if (s->buffer->last < s->buffer->end) { + return rc; + } + + l.len = s->buffer->last - s->buffer->start; + l.data = s->buffer->start; + + ngx_log_error(NGX_LOG_INFO, s->connection->log, 0, + "client sent too long command \"%V\"", &l); + + return NGX_IMAP_PARSE_INVALID_COMMAND; + } + + if (rc == NGX_IMAP_NEXT || rc == NGX_IMAP_PARSE_INVALID_COMMAND) { return rc; } |