You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2020/03/18 20:57:21 UTC

[trafficserver] branch 8.1.x updated: Add nullptr check on ConfigProcessor::release()

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

zwoop pushed a commit to branch 8.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/8.1.x by this push:
     new 5ee66a2  Add nullptr check on ConfigProcessor::release()
5ee66a2 is described below

commit 5ee66a240bd9b1fac1952e0c017a1e8f7ac20c98
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Mon Mar 11 14:50:42 2019 +0900

    Add nullptr check on ConfigProcessor::release()
    
    (cherry picked from commit f32e5c1b76e2cc5f4583028a05d03ceb00fa5105)
---
 mgmt/ProxyConfig.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mgmt/ProxyConfig.cc b/mgmt/ProxyConfig.cc
index 4c885da..3c9bcde 100644
--- a/mgmt/ProxyConfig.cc
+++ b/mgmt/ProxyConfig.cc
@@ -196,7 +196,7 @@ ConfigProcessor::release(unsigned int id, ConfigInfo *info)
 
   idx = id - 1;
 
-  if (info->refcount_dec() == 0) {
+  if (info && info->refcount_dec() == 0) {
     // When we release, we should already have replaced this object in the index.
     Debug("config", "Release config %d 0x%" PRId64, id, (int64_t)info);
     ink_release_assert(info != this->infos[idx]);