OpenContrail

Coverity issue in contrail-controller/physical_port_ovsdb.cc

Bug #1777400 reported by Anda-Maria Nicolae
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
Medium
Anda-Maria Nicolae
Juniper Openstack r5.0.1
Trunk
Fix Committed
Medium
Anda-Maria Nicolae
Juniper Openstack r5.1.0
OpenContrail
New
Undecided
Unassigned

Bug Description

Hi all,

I have run Coverity on branch R5.0 and it reported the following issue in contrail-controller/physical_port_ovsdb.cc:EntryOvsdbUpdate()

uninit_use_in_call: Using uninitialized value key->table_ when calling VlanPortBindingEntry.

Solution is to modify in the first 2 constructors for VlanPortBindingEntry() to call OvsdbDBEntry(table) instead of OvsdbDBEntry(table_).

Thanks,
Anda

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

Review in progress for https://review.opencontrail.org/43910
Submitter: Anda Nicolae (<email address hidden>)

Jeba Paulaiyan (jebap)
tags: added: contrail-control
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/44091
Submitter: Anda Nicolae (<email address hidden>)

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

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

commit d1d7b263920f0da468501faf0c6680b65200bd36
Author: Anda Nicolae <email address hidden>
Date: Mon Jun 18 12:15:59 2018 +0300

Coverity issue in contrail-controller/physical_port_ovsdb.cc

I have run Coverity on branch R5.0 and it reported
the following issue in
contrail-controller/physical_port_ovsdb.cc:EntryOvsdbUpdate()

uninit_use_in_call: Using uninitialized value
key->table_ when calling VlanPortBindingEntry.

Solution is to modify in the first 2 constructors for
VlanPortBindingEntry() to call OvsdbDBEntry(table)
instead of OvsdbDBEntry(table_).

Change-Id: Ic82b315e9f880cb5129e67f3a74e4004b9474bfd
Closes-Bug: #1777400
Signed-off-by: Anda Nicolae <email address hidden>

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

Reviewed: https://review.opencontrail.org/43910
Committed: http://github.com/Juniper/contrail-controller/commit/c2f0bfa58a3e8823b8d2e6da4bbcd0165db7b21e
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit c2f0bfa58a3e8823b8d2e6da4bbcd0165db7b21e
Author: Anda Nicolae <email address hidden>
Date: Mon Jun 18 12:15:59 2018 +0300

Coverity issue in contrail-controller/physical_port_ovsdb.cc

I have run Coverity on branch R5.0 and it reported
the following issue in
contrail-controller/physical_port_ovsdb.cc:EntryOvsdbUpdate()

uninit_use_in_call: Using uninitialized value
key->table_ when calling VlanPortBindingEntry.

Solution is to modify in the first 2 constructors for
VlanPortBindingEntry() to call OvsdbDBEntry(table)
instead of OvsdbDBEntry(table_).

Change-Id: Ic82b315e9f880cb5129e67f3a74e4004b9474bfd
Closes-Bug: #1777400
Signed-off-by: Anda Nicolae <email address hidden>

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

Other bug subscribers

Remote bug watches

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