[trafficserver] branch master updated: h2spec: Parse priority frames even if H2 stream priority is disabled

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

maskit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new c94075a  h2spec: Parse priority frames even if H2 stream priority is disabled
c94075a is described below

commit c94075a918cea1b9ca0df61e1d9d2ae085f401a1
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Tue May 8 15:12:02 2018 +0100

    h2spec: Parse priority frames even if H2 stream priority is disabled
---
 proxy/http2/Http2ConnectionState.cc | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/proxy/http2/Http2ConnectionState.cc b/proxy/http2/Http2ConnectionState.cc
index 0248c53..4191a90 100644
--- a/proxy/http2/Http2ConnectionState.cc
+++ b/proxy/http2/Http2ConnectionState.cc
@@ -377,10 +377,6 @@ rcv_priority_frame(Http2ConnectionState &cstate, const Http2Frame &frame)
                       "priority bad length");
   }
 
-  if (!Http2::stream_priority_enabled) {
-    return Http2Error(Http2ErrorClass::HTTP2_ERROR_CLASS_NONE);
-  }
-
   uint8_t buf[HTTP2_PRIORITY_LEN] = {0};
   frame.reader()->memcpy(buf, HTTP2_PRIORITY_LEN, 0);
 
@@ -396,6 +392,10 @@ rcv_priority_frame(Http2ConnectionState &cstate, const Http2Frame &frame)
                       "PRIORITY frame depends on itself");
   }
 
+  if (!Http2::stream_priority_enabled) {
+    return Http2Error(Http2ErrorClass::HTTP2_ERROR_CLASS_NONE);
+  }
+
   Http2StreamDebug(cstate.ua_session, stream_id, "PRIORITY - dep: %d, weight: %d, excl: %d, tree size: %d",
                    priority.stream_dependency, priority.weight, priority.exclusive_flag, cstate.dependency_tree->size());
 

-- 
To stop receiving notification emails like this one, please contact
maskit@apache.org.