Mercurial > hg > nginx

changeset 5899:234c5ecb00c0

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
SPDY: fixed "too long header line" logging. This fixes possible one byte buffer overrun and makes sure ellipsis are always added, see 21043ce2a005.
author Maxim Dounin <mdounin@mdounin.ru>
date Fri, 07 Nov 2014 17:38:55 +0300
parents f0af7ba616d8
children 20d966ad5e89
files src/http/ngx_http_spdy.c
diffstat 1 files changed, 1 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- 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(
 
         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;