diff options
author | Valentin Bartenev <vbart@nginx.com> | 2016-06-16 20:55:11 +0300 |
---|---|---|
committer | Valentin Bartenev <vbart@nginx.com> | 2016-06-16 20:55:11 +0300 |
commit | 5c66c4aad01be83183395e1ecc241267d45c2e8d (patch) | |
tree | 59bb0d4a54ee5925174461786d9b6d0467410be4 | |
parent | a70e415ea39c903f4c387e35faa71e2ac29a8e3c (diff) |
HTTP/2: prevented double termination of a stream.
According to RFC 7540, an endpoint should not send more than one RST_STREAM
frame for any stream.
Also, now all the data frames will be skipped while termination.
-rw-r--r-- | src/http/v2/ngx_http_v2.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c index 94f57a717..99cb3fc70 100644 --- a/src/http/v2/ngx_http_v2.c +++ b/src/http/v2/ngx_http_v2.c @@ -3890,6 +3890,10 @@ ngx_http_v2_terminate_stream(ngx_http_v2_connection_t *h2c, ngx_event_t *rev; ngx_connection_t *fc; + if (stream->rst_sent) { + return NGX_OK; + } + if (ngx_http_v2_send_rst_stream(h2c, stream->node->id, status) == NGX_ERROR) { @@ -3897,6 +3901,7 @@ ngx_http_v2_terminate_stream(ngx_http_v2_connection_t *h2c, } stream->rst_sent = 1; + stream->skip_data = 1; fc = stream->request->connection; fc->error = 1; |