Coverity issue in contrail-controller/acl.cc
Bug #1777452 reported by
Anda-Maria Nicolae
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Juniper Openstack | Status tracked in Trunk | |||||
R5.0 |
Fix Committed
|
High
|
Anda-Maria Nicolae | |||
Trunk |
Fix Committed
|
High
|
Anda-Maria Nicolae | |||
OpenContrail |
New
|
Undecided
|
Unassigned |
Bug Description
I have run Coverity on branch R5.0 and it reported the following issue in contrail-
uninit_use_in_call: Using uninitialized value ace_spec.type when calling Reverse
I've noticed that type field is indeed set in Reverse() function, but its value is never checked (for instance, in an if clause).
A solution I have found is to add another field, named NOT_USED, in enum AclTypeSpec and initialize in constructor method, type field with NOT_USED.
Thanks,
Anda
tags: | added: contrail-control |
To post a comment you must log in.
Review in progress for https:/ /review. opencontrail. org/43919
Submitter: Anda Nicolae (<email address hidden>)