[trafficserver] branch quic-latest updated: Fix a heap-use-after-free

[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 2cb99d2  Fix a heap-use-after-free
2cb99d2 is described below

commit 2cb99d218a84453a488a02c9f7406a4d87d1523c
Author: Masakazu Kitajo <ma...@apache.org>
AuthorDate: Mon May 14 14:07:48 2018 +0900

    Fix a heap-use-after-free
---
 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 af64ddb..2557e25 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1445,10 +1445,11 @@ QUICNetVConnection::_dequeue_recv_packet(QUICPacketCreationResult &result)
     written += b->read_avail();
     b = b->next.get();
   }
-  udp_packet->free();
 
   quic_packet =
     this->_packet_factory.create(udp_packet->from, std::move(pkt), written, this->largest_received_packet_number(), result);
+  udp_packet->free();
+
   switch (result) {
   case QUICPacketCreationResult::NOT_READY:
     QUICConDebug("Not ready to decrypt the packet");

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