[nginx] HTTP/2: fixed invalid headers handling (ticket #831).

Valentin Bartenev vbart at nginx.com
Fri Nov 13 17:11:51 UTC 2015


details:   http://hg.nginx.org/nginx/rev/932a465537ef
branches:  
changeset: 6291:932a465537ef
user:      Valentin Bartenev <vbart at nginx.com>
date:      Fri Nov 13 20:10:50 2015 +0300
description:
HTTP/2: fixed invalid headers handling (ticket #831).

The r->invalid_header flag wasn't reset once an invalid header appeared in a
request, resulting in all subsequent headers in the request were also marked
as invalid.

diffstat:

 src/http/v2/ngx_http_v2.c |  2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diffs (12 lines):

diff -r 4d5ac1a31d44 -r 932a465537ef src/http/v2/ngx_http_v2.c
--- a/src/http/v2/ngx_http_v2.c	Wed Nov 11 15:47:30 2015 +0300
+++ b/src/http/v2/ngx_http_v2.c	Fri Nov 13 20:10:50 2015 +0300
@@ -2949,6 +2949,8 @@ ngx_http_v2_validate_header(ngx_http_req
         return NGX_ERROR;
     }
 
+    r->invalid_header = 0;
+
     cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module);
 
     for (i = (header->name.data[0] == ':'); i != header->name.len; i++) {



More information about the nginx-devel mailing list