OpenContrail

Coverity issue in contrail-controller/ovsdb_client_connection_state.cc

Bug #1777411 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/ovsdb_client_connection_state.cc:PhysicalDeviceNotify()

deref_after_free: Dereferencing freed pointer state

Solution is to check if state!= NULL in "if (state->ha_stale_dev_vn_table_ != NULL) {".

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/43913
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/44090
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/43913
Committed: http://github.com/Juniper/contrail-controller/commit/820b40a672cddd9fbdd8a562e14ba4b115617b3e
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 820b40a672cddd9fbdd8a562e14ba4b115617b3e
Author: Anda Nicolae <email address hidden>
Date: Mon Jun 18 14:42:31 2018 +0300

Coverity issue in contrail-controller/ovsdb_client_connection_state.cc

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

deref_after_free: Dereferencing freed pointer state

Solution is to check if state!= NULL in:
"if (state->ha_stale_dev_vn_table_ != NULL) {".

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

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

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

commit 4853ef07f9b4899dd08c8234e853985749858eda
Author: Anda Nicolae <email address hidden>
Date: Mon Jun 18 14:42:31 2018 +0300

Coverity issue in contrail-controller/ovsdb_client_connection_state.cc

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

deref_after_free: Dereferencing freed pointer state

Solution is to check if state!= NULL in:
"if (state->ha_stale_dev_vn_table_ != NULL) {".

Change-Id: Iafcf498adde07483af059d2302b9101d4fe860e4
Closes-Bug: #1777411
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.