# HG changeset patch # User Valentin Bartenev # Date 1380571257 -14400 # Node ID 4d0c705417844ee5488261eff5311d263d102c4d # Parent 09fc4598fc8e427caf65412e02d011abf1558382 SPDY: fixed connection leak while waiting for request body. If an error occurs in a SPDY connection, the c->error flag is set on every fake request connection, and its read or write event handler is called, in order to finalize it. But while waiting for a request body, it was a no-op since the read event handler ngx_http_request_handler() calls r->read_event_handler that had been set to ngx_http_block_reading(). diff --git a/src/http/ngx_http_spdy.c b/src/http/ngx_http_spdy.c --- a/src/http/ngx_http_spdy.c +++ b/src/http/ngx_http_spdy.c @@ -1214,6 +1214,7 @@ ngx_http_spdy_state_data(ngx_http_spdy_c } if (rb->post_handler) { + r->read_event_handler = ngx_http_block_reading; rb->post_handler(r); } } @@ -2607,6 +2608,9 @@ ngx_http_spdy_read_request_body(ngx_http r->request_body->post_handler = post_handler; + r->read_event_handler = ngx_http_test_reading; + r->write_event_handler = ngx_http_request_empty_handler; + return NGX_AGAIN; }