[trafficserver] branch quic-latest updated: Do not update peer connection id when src cid is zero

[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 77a09fb  Do not update peer connection id when src cid is zero
77a09fb is described below

commit 77a09fbda3d836a0df8770371a00462174b965fc
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Thu May 17 15:03:56 2018 +0900

    Do not update peer connection id when src cid is zero
---
 iocore/net/QUICNetVConnection.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 8c0867b..6822843 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1409,7 +1409,8 @@ QUICNetVConnection::_dequeue_recv_packet(QUICPacketCreationResult &result)
       // Reset CID if a server sent back a new CID
       // FIXME This should happen only once
       QUICConnectionId src_cid = packet->source_cid();
-      if (src_cid.length()) {
+      // FIXME src connection id could be zero ? if so, check packet header type.
+      if (src_cid != QUICConnectionId::ZERO()) {
         if (this->_peer_quic_connection_id != src_cid) {
           this->_peer_quic_connection_id = src_cid;
         }

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