trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bc...@apache.org
Subject [trafficserver] branch quic-latest updated: Call the QUICFrame destructor before freeing into the ClassAllocator
Date Thu, 05 Oct 2017 21:14:21 GMT
This is an automated email from the ASF dual-hosted git repository.

bcall 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 6df84c9  Call the QUICFrame destructor before freeing into the ClassAllocator
6df84c9 is described below

commit 6df84c9fb9a4ba936bd747e8509e56ea32d4f903
Author: Bryan Call <bcall@apache.org>
AuthorDate: Thu Oct 5 13:21:29 2017 -0700

    Call the QUICFrame destructor before freeing into the ClassAllocator
---
 iocore/net/quic/QUICFrame.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/iocore/net/quic/QUICFrame.h b/iocore/net/quic/QUICFrame.h
index 0b1e554..772d65a 100644
--- a/iocore/net/quic/QUICFrame.h
+++ b/iocore/net/quic/QUICFrame.h
@@ -40,6 +40,7 @@ public:
   virtual void store(uint8_t *buf, size_t *len) const = 0;
   virtual void reset(const uint8_t *buf, size_t len);
   static QUICFrameType type(const uint8_t *buf);
+  virtual ~QUICFrame() {}
 
   LINK(QUICFrame, link);
 
@@ -182,7 +183,7 @@ public:
   QUICAckFrame() : QUICFrame() {}
   QUICAckFrame(const uint8_t *buf, size_t len);
   QUICAckFrame(QUICPacketNumber largest_acknowledged, uint16_t ack_delay, uint64_t first_ack_block_length);
-  ~QUICAckFrame();
+  virtual ~QUICAckFrame();
   virtual void reset(const uint8_t *buf, size_t len) override;
   virtual QUICFrameType type() const override;
   virtual size_t size() const override;
@@ -486,6 +487,7 @@ public:
   static void
   delete_ack_frame(QUICFrame *frame)
   {
+    frame->~QUICFrame();
     quicAckFrameAllocator.free(static_cast<QUICAckFrame *>(frame));
   }
 

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

Mime
View raw message