[PATCH 1 of 4] HTTP/2: emit new frames only after applying all SETTINGS params

Piotr Sikora piotrsikora at google.com
Thu Jun 1 21:55:06 UTC 2017


Hey Valentin,

> The new initial window size can be lower than the previous one,
> so the difference can be negative (that's why the delta parameter
> of ngx_http_v2_adjust_windows() is ssize_t).

Oops, good catch, thanks!

Funnily enough, the original patch worked just fine (at least on
systems where size of ssize_t was equal to size of ngx_uint_t), since
the value would overflow and result in the same negative number when
casted to ssize_t.

I updated patchset with suggested changes, please take a look.

Best regards,
Piotr Sikora


More information about the nginx-devel mailing list