[PATCH] ngx_gzip might hang the request for slow downstreams

Maxim Dounin mdounin at mdounin.ru
Sun Oct 27 14:06:36 UTC 2013


Hello!

On Sun, Oct 27, 2013 at 03:14:08PM +0400, Maxim Dounin wrote:

[...]

> > -    if (ctx->nomem) {
> > +    if (ctx->nomem || in == NULL) {
> > 
> >          /* flush busy buffers */
> > 
> > -        if (ngx_http_next_body_filter(r, NULL) == NGX_ERROR) {
> > +        rc = ngx_http_next_body_filter(r, NULL);
> > +        if (rc == NGX_ERROR) {
> >              goto failed;
> >          }
> 
> Just a side note: this change is unrelated.

Err, missed the use of "rc" in "return" below, disregrad this 
comment.

[...]

-- 
Maxim Dounin
http://nginx.org/en/donation.html



More information about the nginx-devel mailing list