Skip to content

V
varnish-cache
Commit 78694521 authored by Dridi Boukelmoune's avatar Dridi Boukelmoune
Browse files
Options

Turn the question into an explanation 

It was already answered outside of the commit log.
parent 0582753d
Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 2 deletions
bin/varnishd/http2/cache_http2_proto.c
View file @ 78694521
...@@ -644,8 +644,9 @@ h2_rx_headers(struct worker *wrk, struct h2_sess *h2, struct h2_req *r2) ...@@ -644,8 +644,9 @@ h2_rx_headers(struct worker *wrk, struct h2_sess *h2, struct h2_req *r2)
if (r2 == NULL) { if (r2 == NULL) {
if (h2->rxf_stream <= h2->highest_stream) if (h2->rxf_stream <= h2->highest_stream)
return (H2CE_PROTOCOL_ERROR); // rfc7540,l,1153,1158 return (H2CE_PROTOCOL_ERROR); // rfc7540,l,1153,1158
/* XXX: is it safe to load h2->open_streams without the h2 /* NB: we don't need to guard the read of h2->open_streams
* session lock held? * because headers are handled sequentially so it cannot
* increase under our feet.
*/ */
if (h2->open_streams >= if (h2->open_streams >=
h2->local_settings.max_concurrent_streams) { h2->local_settings.max_concurrent_streams) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment