[201911] Fixes for NAT lgtm alerts (#1391)

This PR is similar as #1320 which is merged on master branch. Signed-off-by: Akhilesh Samineni akhilesh.samineni@broadcom.com
sonic-net · Dec 16, 2020 · bb1b351 · bb1b351
1 parent 7f50b98
commit bb1b351
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/cfgmgr/natmgr.cpp b/cfgmgr/natmgr.cpp
@@ -4113,7 +4113,7 @@ void NatMgr::setDynamicAllForwardOrAclbasedRules(const string &opCmd, const stri
         /* After deletion, set acl_interface to None */  
         if (opCmd == DELETE)
         {
-            m_natBindingInfo[dynamicKey].acl_interface == NONE_STRING;
+            m_natBindingInfo[dynamicKey].acl_interface = NONE_STRING;
         }
     }
 

diff --git a/natsyncd/natsync.cpp b/natsyncd/natsync.cpp
@@ -61,6 +61,14 @@ NatSync::NatSync(RedisPipeline *pipelineAppDB, DBConnector *appDb, DBConnector *
     }
 }
 
+NatSync::~NatSync()
+{
+    if (m_AppRestartAssist)
+    {
+        delete m_AppRestartAssist;
+    }
+}
+
 /* To check the port init is done or not */
 bool NatSync::isPortInitDone(DBConnector *app_db)
 {

diff --git a/natsyncd/natsync.h b/natsyncd/natsync.h
@@ -43,6 +43,7 @@ class NatSync : public NetMsg
 {
 public:
     NatSync(RedisPipeline *pipelineAppDB, DBConnector *appDb, DBConnector *stateDb, NfNetlink *nfnl);
+    ~NatSync();
 
     virtual void onMsg(int nlmsg_type, struct nl_object *obj);