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/04/01 23:17:28 UTC

[trafficserver] branch master updated: Add nullptr check on ConfigProcessor::release()

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

masaori pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new f32e5c1  Add nullptr check on ConfigProcessor::release()
f32e5c1 is described below

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

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

diff --git a/mgmt/ProxyConfig.cc b/mgmt/ProxyConfig.cc
index 4ea74d6..2753ca4 100644
--- a/mgmt/ProxyConfig.cc
+++ b/mgmt/ProxyConfig.cc
@@ -188,7 +188,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]);