Re: баг SPDY

Илья Шипицин chipitsine at gmail.com
Wed Aug 14 10:59:29 UTC 2013


да, с таким патчем все ок.

как пожелание - в ngx_http_spdy.c три места, откуда может прилететь
NGX_ERROR (после патча - 2 места), никакой отладки в лог не попадает,
сходу непонятно, почему порвалось.

на стенд патч не накатил, вдруг гугл будет смотреть, а там все работает.

14 августа 2013 г., 16:42 пользователь Валентин Бартенев
<vbart at nginx.com> написал:
> On Wednesday 14 August 2013 14:31:38 Илья Шипицин wrote:
>> учитывая ситуацию
>>
>> 1) сколько в мире инсталировано браузеров Chrome
>> 2) отсутствия негативных последствий пропуска пустого хедера (они
>> вообще могут быть ?)
>>
>> наверное, логично просто пропускать пустой хедер.
> [..]
>
> Можете опробовать:
>
> diff -r 7094bd12c1ff src/http/ngx_http_spdy.c
> --- a/src/http/ngx_http_spdy.c  Tue Aug 06 19:58:40 2013 +0400
> +++ b/src/http/ngx_http_spdy.c  Wed Aug 14 14:41:13 2013 +0400
> @@ -2001,10 +2001,6 @@ ngx_http_spdy_parse_header(ngx_http_requ
>
>          len = ngx_spdy_frame_parse_uint16(p);
>
> -        if (!len) {
> -            return NGX_ERROR;
> -        }
> -
>          p += NGX_SPDY_NV_VLEN_SIZE;
>
>          r->header_end = p + len;
>
> --
> Валентин Бартенев
> http://nginx.org/en/donation.html
> _______________________________________________
> nginx-ru mailing list
> nginx-ru at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-ru


Подробная информация о списке рассылки nginx-ru