You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2019/01/31 12:37:06 UTC

[trafficserver] branch quic-latest updated: Don't respond with RETRY if received packet has short header

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 8c0574a  Don't respond with RETRY if received packet has short header
8c0574a is described below

commit 8c0574a1dae84f6923c101f89469aa93f9fb2715
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Thu Jan 31 21:35:57 2019 +0900

    Don't respond with RETRY if received packet has short header
---
 iocore/net/QUICPacketHandler.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/iocore/net/QUICPacketHandler.cc b/iocore/net/QUICPacketHandler.cc
index 3278191..20a4b0c 100644
--- a/iocore/net/QUICPacketHandler.cc
+++ b/iocore/net/QUICPacketHandler.cc
@@ -280,7 +280,7 @@ QUICPacketHandlerIn::_recv_packet(int event, UDPPacket *udp_packet)
   // Server Stateless Retry
   QUICConfig::scoped_config params;
   QUICConnectionId cid_in_retry_token;
-  if (!vc && params->stateless_retry()) {
+  if (!vc && params->stateless_retry() && QUICInvariants::is_long_header(buf)) {
     int ret = this->_stateless_retry(buf, buf_len, udp_packet->getConnection(), udp_packet->from, dcid, scid, &cid_in_retry_token);
     if (ret < 0) {
       udp_packet->free();