[trafficserver] branch quic-latest updated: Add nullptr check before calling QUICConnectionTable::erase()

[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 8185c20  Add nullptr check before calling QUICConnectionTable::erase()
8185c20 is described below

commit 8185c20aad939cfe0187e1b7b711bd973e4b60e1
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Mon Apr 2 15:18:56 2018 +0900

    Add nullptr check before calling QUICConnectionTable::erase()
    
    QUICNetVConnection::_ctable is nullptr when QVC is out going connection
---
 iocore/net/QUICNetVConnection.cc | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 540b39c..cac33f3 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -245,8 +245,11 @@ QUICNetVConnection::free()
 void
 QUICNetVConnection::remove_connection_ids()
 {
-  this->_ctable->erase(this->_original_quic_connection_id, this);
-  this->_ctable->erase(this->_quic_connection_id, this);
+  if (this->_ctable) {
+    this->_ctable->erase(this->_original_quic_connection_id, this);
+    this->_ctable->erase(this->_quic_connection_id, this);
+  }
+
   if (this->_alt_con_manager) {
     this->_alt_con_manager->invalidate_alt_connections();
   }

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