Re: [PATCH] HTTP/2: fix flow control with padded DATA frames

2017-03-29 Thread Valentin V. Bartenev
On Sunday 26 March 2017 01:41:11 Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490516704 25200 > # Sun Mar 26 01:25:04 2017 -0700 > # Node ID 899a53d2789b8c6bafdd5e40d78b4e92dd32dd10 > # Parent

Re: [PATCH] HTTP/2: fix flow control with padded DATA frames

2017-03-27 Thread Piotr Sikora via nginx-devel
Hey Valentin, >> @@ -802,33 +805,32 @@ ngx_http_v2_state_data(ngx_http_v2_conne >> } >> >> h2c->state.padding = *pos++; >> -h2c->state.length--; >> - >> -if (h2c->state.padding > h2c->state.length) { >> + >> +if (h2c->state.padding >= size) { >>

Re: [PATCH] HTTP/2: fix flow control with padded DATA frames

2017-03-27 Thread Valentin V. Bartenev
On Sunday 26 March 2017 01:41:11 Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1490516704 25200 > # Sun Mar 26 01:25:04 2017 -0700 > # Node ID 899a53d2789b8c6bafdd5e40d78b4e92dd32dd10 > # Parent

[PATCH] HTTP/2: fix flow control with padded DATA frames

2017-03-26 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1490516704 25200 # Sun Mar 26 01:25:04 2017 -0700 # Node ID 899a53d2789b8c6bafdd5e40d78b4e92dd32dd10 # Parent 22be63bf21edaa1b8ea916c7d8cd4e5fe4892061 HTTP/2: fix flow control with padded DATA frames. Previously, flow