[trafficserver] branch quic-latest updated: Fix offset field length of STREAM frame when offset is 0

[ma...@apache.org]

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

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new 9ec0a4a  Fix offset field length of STREAM frame when offset is 0
9ec0a4a is described below

commit 9ec0a4a2742522d00c2f93e6c625d10b359292b9
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Mon Dec 18 09:32:31 2017 +0900

    Fix offset field length of STREAM frame when offset is 0
---
 iocore/net/quic/QUICFrame.cc | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/iocore/net/quic/QUICFrame.cc b/iocore/net/quic/QUICFrame.cc
index 0104dce..0c29f57 100644
--- a/iocore/net/quic/QUICFrame.cc
+++ b/iocore/net/quic/QUICFrame.cc
@@ -228,7 +228,7 @@ QUICStreamFrame::has_fin_flag() const
 size_t
 QUICStreamFrame::_get_stream_id_field_offset() const
 {
-  return 1;
+  return sizeof(QUICFrameType);
 }
 
 size_t
@@ -281,7 +281,11 @@ QUICStreamFrame::_get_offset_field_len() const
       return 0;
     }
   } else {
-    return QUICVariableInt::size(this->_offset);
+    if (this->_offset != 0) {
+      return QUICVariableInt::size(this->_offset);
+    } else {
+      return 0;
+    }
   }
 }
 

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].