From 614cdf543cc744dfc7c365629bce7569fe20bed3 Mon Sep 17 00:00:00 2001
From: Nischal Sheth <nsheth@juniper.net>
Date: Wed, 20 May 2015 18:05:21 -0700
Subject: [PATCH] Get rid of a few unnecessary asserts

Change-Id: I9e165d91c5c5485e908f44bee41688c52899c2d2
Partial-Bug: 1456193
---
 src/bgp/bgp_config_ifmap.cc                  | 10 +++++++---
 src/bgp/routing-instance/routing_instance.cc |  4 ++--
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/bgp/bgp_config_ifmap.cc b/src/bgp/bgp_config_ifmap.cc
index 5e57fcc1019..03206b91138 100644
--- a/src/bgp/bgp_config_ifmap.cc
+++ b/src/bgp/bgp_config_ifmap.cc
@@ -516,7 +516,8 @@ static void GetRoutingInstanceExportTargets(DBGraph *graph, IFMapNode *node,
             (GetInstanceTargetRouteTarget(graph, adj, &target))) {
             const autogen::InstanceTarget *itarget =
                     dynamic_cast<autogen::InstanceTarget *>(adj->GetObject());
-            assert(itarget);
+            if (!itarget)
+                continue;
             const autogen::InstanceTargetType &itt = itarget->data();
             if (itt.import_export != "import")
                 target_list->push_back(target);
@@ -632,7 +633,8 @@ void BgpIfmapInstanceConfig::Update(BgpIfmapConfigManager *manager,
             if (GetInstanceTargetRouteTarget(graph, adj, &target)) {
                 const autogen::InstanceTarget *itarget =
                     dynamic_cast<autogen::InstanceTarget *>(adj->GetObject());
-                assert(itarget);
+                if (!itarget)
+                    continue;
                 const autogen::InstanceTargetType &itt = itarget->data();
                 if (itt.import_export == "import") {
                     import_list.insert(target);
@@ -1332,7 +1334,9 @@ void BgpIfmapConfigManager::ProcessBgpPeering(const BgpConfigDelta &delta) {
         event = BgpConfigManager::CFG_ADD;
         string instance_name(IdentifierParent(routers.first->name()));
         BgpIfmapInstanceConfig *rti = config_->FindInstance(instance_name);
-        assert(rti != NULL);
+        if (rti == NULL) {
+            return;
+        }
         peering = config_->CreatePeering(rti, proxy);
     } else {
         const IFMapNode *node = peering->node();
diff --git a/src/bgp/routing-instance/routing_instance.cc b/src/bgp/routing-instance/routing_instance.cc
index 2400acd5d74..b7dfcab4db6 100644
--- a/src/bgp/routing-instance/routing_instance.cc
+++ b/src/bgp/routing-instance/routing_instance.cc
@@ -316,7 +316,7 @@ void RoutingInstanceMgr::UpdateRoutingInstance(
         RTINSTANCE_LOG_MESSAGE(server_,
             SandeshLevel::SYS_WARN, RTINSTANCE_LOG_FLAG_ALL, config->name(),
             "Instance not found during update");
-        assert(rtinstance != NULL);
+        return;
     }
 
     InstanceTargetRemove(rtinstance);
@@ -363,7 +363,7 @@ void RoutingInstanceMgr::DeleteRoutingInstance(const string &name) {
         RTINSTANCE_LOG_MESSAGE(server_,
             SandeshLevel::SYS_WARN, RTINSTANCE_LOG_FLAG_ALL, name,
             "Instance not found during delete");
-        assert(rtinstance != NULL);
+        return;
     }
 
     InstanceVnIndexRemove(rtinstance);