SpotBugs Bug Detector Report
The following document contains the results of SpotBugs
SpotBugs Version is 4.8.6
Threshold is medium
Effort is default
Summary
| Classes |
Bugs |
Errors |
Missing Classes |
| 70 |
23 |
0 |
0 |
org.opentrafficsim.base.HierarchicalType
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.HierarchicalType at new org.opentrafficsim.base.HierarchicalType(String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
45 |
Medium |
| Exception thrown in class org.opentrafficsim.base.HierarchicalType at new org.opentrafficsim.base.HierarchicalType(String, HierarchicalType) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
57 |
Medium |
| org.opentrafficsim.base.HierarchicalType.getChildren() may expose internal representation by returning HierarchicalType.children |
MALICIOUS_CODE |
EI_EXPOSE_REP |
88 |
Medium |
org.opentrafficsim.base.Resource
| Bug |
Category |
Details |
Line |
Priority |
| org.opentrafficsim.base.Resource.getResourceAsUri(String) may fail to clean up java.io.InputStream |
EXPERIMENTAL |
OBL_UNSATISFIED_OBLIGATION |
54 |
Medium |
org.opentrafficsim.base.TimeStampedObject
| Bug |
Category |
Details |
Line |
Priority |
| org.opentrafficsim.base.TimeStampedObject.timestamp() may expose internal representation by returning TimeStampedObject.timestamp |
MALICIOUS_CODE |
EI_EXPOSE_REP |
19 |
Medium |
| new org.opentrafficsim.base.TimeStampedObject(Object, Time) may expose internal representation by storing an externally mutable object into TimeStampedObject.timestamp |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
19 |
Medium |
org.opentrafficsim.base.geometry.BoundingBoxRounded
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.geometry.BoundingBoxRounded at new org.opentrafficsim.base.geometry.BoundingBoxRounded(double, double, double) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
68 |
Medium |
org.opentrafficsim.base.geometry.BoundingCircle
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.geometry.BoundingCircle at new org.opentrafficsim.base.geometry.BoundingCircle(double) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
36 |
Medium |
org.opentrafficsim.base.geometry.OtsRenderable
| Bug |
Category |
Details |
Line |
Priority |
| org.opentrafficsim.base.geometry.OtsRenderable.RENDERING_KEYS should be both final and package protected |
MALICIOUS_CODE |
MS_FINAL_PKGPROTECT |
34 |
Medium |
| org.opentrafficsim.base.geometry.OtsRenderable.RENDERING_VALUES should be both final and package protected |
MALICIOUS_CODE |
MS_FINAL_PKGPROTECT |
37 |
Medium |
org.opentrafficsim.base.parameters.ParameterSet
| Bug |
Category |
Details |
Line |
Priority |
| org.opentrafficsim.base.parameters.ParameterSet.setAllOneByOne(Parameters) makes inefficient use of keySet iterator instead of entrySet iterator |
PERFORMANCE |
WMI_WRONG_MAP_ITERATOR |
296 |
Medium |
org.opentrafficsim.base.parameters.ParameterType
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.parameters.ParameterType at new org.opentrafficsim.base.parameters.ParameterType(String, String, Class) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
50 |
Medium |
| Exception thrown in class org.opentrafficsim.base.parameters.ParameterType at new org.opentrafficsim.base.parameters.ParameterType(String, String, Class, Object) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
75 |
Medium |
| Exception thrown in class org.opentrafficsim.base.parameters.ParameterType at new org.opentrafficsim.base.parameters.ParameterType(String, String, Class, Object, Constraint) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
89 |
Medium |
| Exception thrown in class org.opentrafficsim.base.parameters.ParameterType at new org.opentrafficsim.base.parameters.ParameterType(String, String, Class, Object, Constraint, boolean) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
105 |
Medium |
| Exception thrown in class org.opentrafficsim.base.parameters.ParameterType at new org.opentrafficsim.base.parameters.ParameterType(String, String, Class, Constraint) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
63 |
Medium |
org.opentrafficsim.base.parameters.constraint.CollectionConstraint
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.parameters.constraint.CollectionConstraint at new org.opentrafficsim.base.parameters.constraint.CollectionConstraint(Collection) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
31 |
Medium |
| new org.opentrafficsim.base.parameters.constraint.CollectionConstraint(Collection) may expose internal representation by storing an externally mutable object into CollectionConstraint.objects |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
32 |
Medium |
org.opentrafficsim.base.parameters.constraint.MultiConstraint
| Bug |
Category |
Details |
Line |
Priority |
| new org.opentrafficsim.base.parameters.constraint.MultiConstraint(Set) may expose internal representation by storing an externally mutable object into MultiConstraint.constraints |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
52 |
Medium |
org.opentrafficsim.base.parameters.constraint.NumericConstraint
| Bug |
Category |
Details |
Line |
Priority |
| Return value of String.format(String, Object[]) ignored in new org.opentrafficsim.base.parameters.constraint.NumericConstraint(String, int, String) |
CORRECTNESS |
RV_RETURN_VALUE_IGNORED |
101 |
Medium |
org.opentrafficsim.base.parameters.constraint.SingleBound$Bound
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.parameters.constraint.SingleBound$Bound at new org.opentrafficsim.base.parameters.constraint.SingleBound$Bound(Number, String) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
225 |
Medium |
org.opentrafficsim.base.parameters.constraint.SubCollectionConstraint
| Bug |
Category |
Details |
Line |
Priority |
| Exception thrown in class org.opentrafficsim.base.parameters.constraint.SubCollectionConstraint at new org.opentrafficsim.base.parameters.constraint.SubCollectionConstraint(Collection) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. |
BAD_PRACTICE |
CT_CONSTRUCTOR_THROW |
31 |
Medium |
| new org.opentrafficsim.base.parameters.constraint.SubCollectionConstraint(Collection) may expose internal representation by storing an externally mutable object into SubCollectionConstraint.objects |
MALICIOUS_CODE |
EI_EXPOSE_REP2 |
32 |
Medium |
