[nginx] SPDY: fixed connection leak while waiting for request body.

Valentin Bartenev vbart at nginx.com
Mon Sep 30 20:39:55 UTC 2013


details:   http://hg.nginx.org/nginx/rev/4d0c70541784
branches:  
changeset: 5402:4d0c70541784
user:      Valentin Bartenev <vbart at nginx.com>
date:      Tue Oct 01 00:00:57 2013 +0400
description:
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().

diffstat:

 src/http/ngx_http_spdy.c |  4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diffs (21 lines):

diff -r 09fc4598fc8e -r 4d0c70541784 src/http/ngx_http_spdy.c
--- a/src/http/ngx_http_spdy.c	Mon Sep 30 22:10:13 2013 +0400
+++ b/src/http/ngx_http_spdy.c	Tue Oct 01 00:00:57 2013 +0400
@@ -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;
 }
 



More information about the nginx-devel mailing list