commit 470fb58bda0c54c85307527ed61e19029d03830c
author Valentin Bartenev <vbart@nginx.com> Thu Jun 16 20:55:11 2016 +0300
committer Valentin Bartenev <vbart@nginx.com> Thu Jun 16 20:55:11 2016 +0300
tree 874a6e4658367f8931ce2583be177c5fedfc52e5
parent bb1eab9cb33e22e545fa6798b0e79f2e6b7973ae

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.

src/http/v2/ngx_http_v2.c

diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c
index 94f57a7..99cb3fc 100644
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -3890,6 +3890,10 @@
     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 @@
     }
 
     stream->rst_sent = 1;
+    stream->skip_data = 1;
 
     fc = stream->request->connection;
     fc->error = 1;