[master] ee3abf8 New take on #2624 after Martin's RxStuff overflow fixes
Dag Haavi Finstad
daghf at varnish-software.com
Tue Jun 12 09:39:20 UTC 2018
commit ee3abf8eb30ae03e6e518945fa7927362cd81d1d
Author: Dag Haavi Finstad <daghf at varnish-software.com>
Date: Tue Jun 12 11:28:59 2018 +0200
New take on #2624 after Martin's RxStuff overflow fixes
diff --git a/bin/varnishd/http2/cache_http2_proto.c b/bin/varnishd/http2/cache_http2_proto.c
index 64d55a6..c43a65f 100644
--- a/bin/varnishd/http2/cache_http2_proto.c
+++ b/bin/varnishd/http2/cache_http2_proto.c
@@ -882,8 +882,6 @@ h2_frame_complete(struct http_conn *htc)
u = vbe32dec(htc->rxbuf_b) >> 8;
if (l >= u + 9)
return (HTC_S_COMPLETE);
- else if (l > h2->local_settings.max_frame_size + 9)
- return (HTC_S_OVERFLOW);
return (HTC_S_MORE);
}
@@ -1068,7 +1066,7 @@ h2_rxframe(struct worker *wrk, struct h2_sess *h2)
hs = HTC_RxStuff(h2->htc, h2_frame_complete,
NULL, NULL, NAN,
h2->sess->t_idle + cache_param->timeout_idle,
- 16384 + 9); // rfc7540,l,4228,4228
+ h2->local_settings.max_frame_size + 9);
switch (hs) {
case HTC_S_COMPLETE:
break;
More information about the varnish-commit
mailing list