Coverity issues under vrouter

Bug #1769083 reported by Andrei Bunghez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
In Progress
Medium
Andrei Bunghez
Trunk
Fix Committed
Medium
Andrei Bunghez
OpenContrail
New
Undecided
Unassigned

Bug Description

Hi all,

Fixing the following issues found by coverity in:

vr_interface.c: vif is leaked in vr_interface_add if allocation fails for vif_hw_queues. (68306)

unix_util.c: Using uninitialized value attr when calling nl_parse_dcb_response on default switch branch (59822) - moving fix from nl_util.c

unix_util.c: fd is leaked in the unlikely case it is 0 in vr_table_map. (67470)

Regards,
Andrei

Tags: vrouter
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/42756
Submitter: Andrei Bunghez (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/42757
Submitter: Andrei Bunghez (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/42757
Committed: http://github.com/Juniper/contrail-vrouter/commit/6b1226f196283f098acacde881b09eccfb228b2a
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 6b1226f196283f098acacde881b09eccfb228b2a
Author: Andrei Bunghez <email address hidden>
Date: Fri May 4 07:49:15 2018 +0000

Coverity issues in unix_util.c and vr_interface.c

vr_interface.c: vif is leaked in vr_interface_add if allocation fails for vif_hw_queues. (68306)

unix_util.c: Using uninitialized value attr when calling nl_parse_dcb_response on default switch branch (59822) - moving fix from nl_util.c

unix_util.c: fd is leaked in the unlikely case it is 0 in vr_table_map. (67470)

Change-Id: I7fa192d4cd8e643a86189f18e34d08c6a173154e
Closes-Bug: #1769083

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.