Author: mdounin
Date: 2012-02-10 14:31:04 +0000 (Fri, 10 Feb 2012)
New Revision: 4461

Log:
Upstream: fixed "too big header" check.

If header filter postponed processing of a header by returning NGX_AGAIN
and not moved u->buffer->pos, previous check incorrectly assumed there
is additional space and did another recv() with zero-size buffer. This
resulted in "upstream prematurely closed connection" error instead
of correct "upstream sent too big header" one.

Patch by Feibo Li.


Modified:
trunk/src/http/ngx_http_upstream.c

Modified: trunk/src/http/ngx_http_upstream.c
===================================================================
--- trunk/src/http/ngx_http_upstream.c 2012-02-10 11:24:19 UTC (rev 4460)
+++ trunk/src/http/ngx_http_upstream.c 2012-02-10 14:31:04 UTC (rev 4461)
@@ -1591,7 +1591,7 @@

if (rc == NGX_AGAIN) {

- if (u->buffer.pos == u->buffer.end) {
+ if (u->buffer.last == u->buffer.end) {
ngx_log_error(NGX_LOG_ERR, c->log, 0,
"upstream sent too big header");


_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel