diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2013-02-01 14:40:19 +0000 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2013-02-01 14:40:19 +0000 |
| commit | 1c5fce75186ae792b76d055223cd0a70ac7370b0 (patch) | |
| tree | 70e1a9d8723e36bb4253f82fc1648ba586c915f6 /src/http/modules/ngx_http_fastcgi_module.c | |
| parent | e97e4124e372d2db118d984988632d28f8966130 (diff) | |
| download | nginx-1c5fce75186ae792b76d055223cd0a70ac7370b0.tar.gz nginx-1c5fce75186ae792b76d055223cd0a70ac7370b0.zip | |
FastCGI: fixed wrong connection close with fastcgi_keep_conn.
With fastcgi_keep_conn it was possible that connection was closed after
FCGI_STDERR record with zero padding and without any further data read yet.
This happended as f->state was set to ngx_http_fastcgi_st_padding and then
"break" happened, resulting in p->length being set to f->padding, i.e. 0
(which in turn resulted in connection close).
Fix is to make sure we continue the loop after f->state is set.
Diffstat (limited to 'src/http/modules/ngx_http_fastcgi_module.c')
| -rw-r--r-- | src/http/modules/ngx_http_fastcgi_module.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/http/modules/ngx_http_fastcgi_module.c b/src/http/modules/ngx_http_fastcgi_module.c index 123125a86..5e8ef40c1 100644 --- a/src/http/modules/ngx_http_fastcgi_module.c +++ b/src/http/modules/ngx_http_fastcgi_module.c @@ -1788,10 +1788,6 @@ ngx_http_fastcgi_input_filter(ngx_event_pipe_t *p, ngx_buf_t *buf) "FastCGI sent in stderr: \"%*s\"", m + 1 - msg, msg); - if (f->pos == f->last) { - break; - } - } else { if (f->padding) { f->state = ngx_http_fastcgi_st_padding; |