FindBugs Bug Detector Report
The following document contains the results of FindBugs
FindBugs Version is 3.0.1
Threshold is low
Effort is max
Summary
Classes |
Bugs |
Errors |
Missing Classes |
42 |
9 |
0 |
0 |
org.opendaylight.netvirt.openstack.netvirt.sfc.DelegatingDataTreeListener
Bug |
Category |
Details |
Line |
Priority |
Exceptional return value of java.util.concurrent.ExecutorService.submit(Runnable) ignored in org.opendaylight.netvirt.openstack.netvirt.sfc.DelegatingDataTreeListener.onDataTreeChanged(Collection) |
BAD_PRACTICE |
RV_RETURN_VALUE_IGNORED_BAD_PRACTICE |
91 |
Low |
org.opendaylight.netvirt.openstack.netvirt.sfc.NetvirtSfcProvider
Bug |
Category |
Details |
Line |
Priority |
Confusing to have methods org.opendaylight.netvirt.openstack.netvirt.sfc.NetvirtSfcProvider.setAddSfFlows(Boolean) and org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.AbstractNetvirtSfcImplModule.setAddsfflows(Boolean) |
BAD_PRACTICE |
NM_CONFUSING |
121 |
Low |
Confusing to have methods org.opendaylight.netvirt.openstack.netvirt.sfc.NetvirtSfcProvider.setOf13Provider(String) and org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.AbstractNetvirtSfcImplModule.setOf13provider(String) |
BAD_PRACTICE |
NM_CONFUSING |
37 |
Low |
org.opendaylight.netvirt.openstack.netvirt.sfc.standalone.openflow13.NetvirtSfcStandaloneOF13Provider
Bug |
Category |
Details |
Line |
Priority |
Private method org.opendaylight.netvirt.openstack.netvirt.sfc.standalone.openflow13.NetvirtSfcStandaloneOF13Provider.handleLocalEgressPort(long, String, long, short, short, boolean) is never called |
PERFORMANCE |
UPM_UNCALLED_PRIVATE_METHOD |
337 |
Low |
Uninitialized read of southbound in new org.opendaylight.netvirt.openstack.netvirt.sfc.standalone.openflow13.NetvirtSfcStandaloneOF13Provider(DataBroker) |
CORRECTNESS |
UR_UNINIT_READ |
87 |
Medium |
org.opendaylight.netvirt.openstack.netvirt.sfc.workaround.NetvirtSfcWorkaroundOF13Provider
Bug |
Category |
Details |
Line |
Priority |
Nullcheck of ovsdbNode at line 336 of value previously dereferenced in org.opendaylight.netvirt.openstack.netvirt.sfc.workaround.NetvirtSfcWorkaroundOF13Provider.getLocalip(Node) |
CORRECTNESS |
RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE |
334 |
Medium |
org.opendaylight.netvirt.openstack.netvirt.sfc.workaround.services.SfcClassifierService
Bug |
Category |
Details |
Line |
Priority |
Write to static field org.opendaylight.netvirt.openstack.netvirt.sfc.workaround.services.SfcClassifierService.cookieIndex from instance method org.opendaylight.netvirt.openstack.netvirt.sfc.workaround.services.SfcClassifierService.getCookie(SfcClassifierService$FlowID) |
STYLE |
ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD |
74 |
Low |
org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.NetvirtSfcImplBuilder$NetvirtSfcImplImpl
Bug |
Category |
Details |
Line |
Priority |
Unchecked/unconfirmed cast from org.opendaylight.yangtools.yang.binding.DataObject to org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.NetvirtSfcImplBuilder$NetvirtSfcImplImpl in org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.NetvirtSfcImplBuilder$NetvirtSfcImplImpl.equals(Object) |
STYLE |
BC_UNCONFIRMED_CAST |
208 |
Low |
org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.netvirt.sfc.impl.BrokerBuilder$BrokerImpl
Bug |
Category |
Details |
Line |
Priority |
Unchecked/unconfirmed cast from org.opendaylight.yangtools.yang.binding.DataObject to org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.netvirt.sfc.impl.BrokerBuilder$BrokerImpl in org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.netvirt.sfc.impl.rev141210.modules.module.configuration.netvirt.sfc.impl.BrokerBuilder$BrokerImpl.equals(Object) |
STYLE |
BC_UNCONFIRMED_CAST |
211 |
Low |