Fix valgrind errors for agent

Fixed following errors 1. Use of delete instead of delete[] 2. Use of unitialized variables in couple of places. Partial-Bug: #1641833 (cherry picked from commit 3def48d) Change-Id: I8edac4497a673f9d923a47bfb94fb7ff66bb3d23
Juniper · Nov 30, 2016 · edb3fb3 · edb3fb3
1 parent fdeeb68
commit edb3fb3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/ksync/ksync_sock.cc b/src/ksync/ksync_sock.cc
@@ -1197,7 +1197,7 @@ KSyncBulkMsgContext::~KSyncBulkMsgContext() {
     assert(vr_response_count_ == io_context_list_.size()); 
     io_context_list_.clear_and_dispose(IoContextDisposer());
     for (uint32_t i = 0; i < rx_buffer_index_; i++) {
-        delete rx_buffers_[i];
+        delete[] rx_buffers_[i];
     }
 }
 

diff --git a/src/ksync/ksync_tx_queue.cc b/src/ksync/ksync_tx_queue.cc
@@ -44,6 +44,7 @@ KSyncTxQueue::KSyncTxQueue(KSyncSock *sock) :
     measure_busy_time_(false) {
     queue_len_ = 0;
     shutdown_ = false;
+    ClearStats();
 }
 
 KSyncTxQueue::~KSyncTxQueue() {

diff --git a/src/vnsw/agent/vrouter/ksync/flowtable_ksync.cc b/src/vnsw/agent/vrouter/ksync/flowtable_ksync.cc
@@ -130,6 +130,7 @@ void FlowTableKSyncEntry::Reset() {
     trap_flow_ = false;
     old_drop_reason_ = 0;
     ecmp_ = false;
+    enable_rpf_ = true;
     src_nh_id_ = NextHopTable::kRpfDiscardIndex;
     last_event_ = FlowEvent::INVALID;
     token_.reset();