Fix potential null pointer dereference. (#103)

Signed-off-by: John Plevyak <jplevyak@gmail.com>
istio · Sep 13, 2019 · e5c31e3 · e5c31e3
1 parent 07a30a3
commit e5c31e3
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/source/common/upstream/cluster_manager_impl.cc b/source/common/upstream/cluster_manager_impl.cc
@@ -453,7 +453,9 @@ bool ClusterManagerImpl::addOrUpdateCluster(const envoy::api::v2::Cluster& clust
       existing_warming_cluster != warming_clusters_.end()) {
     // The following init manager remove call is a NOP in the case we are already initialized. It's
     // just kept here to avoid additional logic.
-    init_helper_.removeCluster(*existing_active_cluster->second->cluster_);
+    if (existing_active_cluster != active_clusters_.end()) {
+      init_helper_.removeCluster(*existing_active_cluster->second->cluster_);
+    }
     cm_stats_.cluster_modified_.inc();
   } else {
     cm_stats_.cluster_added_.inc();