Fix for possible mem leak and fix in refresh network map

Bug #1805773 reported by Parth Sarupria
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
New
Undecided
Unassigned
R4.1
Fix Committed
Undecided
Unassigned

Bug Description

Fix for the Possible leak in FindNetwork for unknown_location string object in structured_syslog_config.cc
Fix in RefreshNetworkMap in structured_syslog_config where vector begin element is deleted.
Commented out sending of unnecessary field (sampling percentage) in structured_syslog_server.cc for APPTRACK_SESSION_CLOSE syslog.

Tags: analytics
information type: Proprietary → Public
summary: - Addition of Tenant level UVE for KPI Metrics
+ Addition of Tenant level UVE for KPI Metrics, Other bug fixes
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/47918
Submitter: Parth Sarupria (<email address hidden>)

summary: - Addition of Tenant level UVE for KPI Metrics, Other bug fixes
+ Fix for possible mem leak and fix in refresh network map
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47999
Submitter: Parth Sarupria (<email address hidden>)

description: updated
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/48203
Submitter: Parth Sarupria (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/48203
Committed: http://github.com/Juniper/contrail-controller/commit/4e9b2a7e5747a7dc4ec9c5bc5388b58b29984d25
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 4e9b2a7e5747a7dc4ec9c5bc5388b58b29984d25
Author: Parth Sarupria <email address hidden>
Date: Thu Dec 13 22:04:30 2018 -0800

Fix for possible mem leak and fix in refresh network map

Fix for the Possible leak in FindNetwork for unknown_location string object in structured_syslog_config.cc
Fix in RefreshNetworkMap in structured_syslog_config where vector begin element is deleted,
For Fixing this Iteration is now done in reverse order to avoid erasing of 0th index vector element first
which results in loss off begin() pointer causing already stored indexes to become incorrect.
Commented out sending of unnecessary field (sampling percentage) in structured_syslog_server.cc for APPTRACK_SESSION_CLOSE syslog.
Added Send code for tenant metrics UVE for PSMR syslog.

Change-Id: I8627e535c01fc81c036c2babc7509e2ab411f7e6
Closes-Bug: #1805773

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.