[trafficserver] branch quic-latest updated: Postpone processing protected packets if handshake is not completed

[ma...@apache.org]

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

maskit 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 5182518  Postpone processing protected packets if handshake is not completed
5182518 is described below

commit 51825180400fbc25a756ed45854ab3f20740cfa5
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Thu Oct 5 00:33:53 2017 -0700

    Postpone processing protected packets if handshake is not completed
    
    There's no guarantee that protected packets arrive after handshake completion
    because actual stream data is not processed on PACKET_READ_READY events.
---
 iocore/net/QUICNetVConnection.cc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 5261450..cd2e832 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -399,6 +399,12 @@ QUICNetVConnection::state_handshake(int event, Event *data)
       error = this->_state_handshake_process_zero_rtt_protected_packet(std::move(p));
       break;
     }
+    case QUICPacketType::ONE_RTT_PROTECTED_KEY_PHASE_0:
+    case QUICPacketType::ONE_RTT_PROTECTED_KEY_PHASE_1:
+      // Postpone processing the packet
+      this->push_packet(std::move(p));
+      this_ethread()->schedule_imm_local(this, event);
+      break;
     default:
       error = QUICErrorUPtr(new QUICConnectionError(QUICErrorClass::QUIC_TRANSPORT, QUICErrorCode::INTERNAL_ERROR));
       break;

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