From fb96936c1a376c8fc1641b5028ebcb909a451919 Mon Sep 17 00:00:00 2001 From: Maxim Dounin Date: Fri, 7 Nov 2014 17:38:55 +0300 Subject: [PATCH] SPDY: fixed "too long header line" logging. This fixes possible one byte buffer overrun and makes sure ellipsis are always added, see 21043ce2a005. --- src/http/ngx_http_spdy.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/http/ngx_http_spdy.c b/src/http/ngx_http_spdy.c index 3c57882a6..010b803b3 100644 --- a/src/http/ngx_http_spdy.c +++ b/src/http/ngx_http_spdy.c @@ -2656,11 +2656,10 @@ ngx_http_spdy_alloc_large_header_buffer(ngx_http_request_t *r) if (rest > NGX_MAX_ERROR_STR - 300) { rest = NGX_MAX_ERROR_STR - 300; - p[rest++] = '.'; p[rest++] = '.'; p[rest++] = '.'; } ngx_log_error(NGX_LOG_INFO, r->connection->log, 0, - "client sent too long header name or value: \"%*s\"", + "client sent too long header name or value: \"%*s...\"", rest, p); return NGX_DECLINED;