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]);