Coverity issues in vrouter dpdk
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Juniper Openstack | Status tracked in Trunk | |||||
R5.0 |
In Progress
|
High
|
Andrei Bunghez | |||
Trunk |
In Progress
|
High
|
Andrei Bunghez | |||
OpenContrail |
New
|
Undecided
|
Andrei Bunghez |
Bug Description
opencontrail master
Hi,
Coverity reports several issues in vrouter/dpdk. I would like to use this issue to track them as I address them. To begin with:
vr_dpdk_netlink.c: 289 - descriptor will be leaked in (the unlikely) case it is 0. (63834)
vr_dpdk_
vr_dpdk_
vr_uvhost.c: In vr_uvhost_start, s is a socket descriptor which can be >= 0 if socket returns success. The "if (s)" on error case (where this comparison is true as s might be -1) leads to unlinking an uninitialized sun.sun_path. (64029, 63838)
vr_dpdk_virtio.c: vring_idx check in vr_dpdk_
Changed in opencontrail: | |
assignee: | nobody → Andrei Bunghez (abunghez) |
Also: set_ring_ num_desc (63924)
vr_dpdk_virtio.c: vring_idx check in vr_dpdk_