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 |
|---|---|---|---|
| 172 | 85 | 0 | 0 |
Files
org.opentrafficsim.draw.BoundsPaintScale
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.BoundsPaintScale at new org.opentrafficsim.draw.BoundsPaintScale(double[], Color[]) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 47 | Medium |
| org.opentrafficsim.draw.BoundsPaintScale.GREEN_RED is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 24 | High |
| org.opentrafficsim.draw.BoundsPaintScale.GREEN_RED_DARK is a mutable array | MALICIOUS_CODE | MS_MUTABLE_ARRAY | 28 | High |
org.opentrafficsim.draw.Colors
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.Colors.COLORS should be both final and package protected | MALICIOUS_CODE | MS_FINAL_PKGPROTECT | 18 | Medium |
| org.opentrafficsim.draw.Colors.NAMES should be both final and package protected | MALICIOUS_CODE | MS_FINAL_PKGPROTECT | 22 | Medium |
org.opentrafficsim.draw.TextProperties
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.TextProperties.getTextAttributes() may expose internal representation by returning TextProperties.textAttributes | MALICIOUS_CODE | EI_EXPOSE_REP | 106 | Medium |
org.opentrafficsim.draw.egtf.Egtf
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of wMean in org.opentrafficsim.draw.egtf.Egtf.filterSI(double[], double[], Quantity[]) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 670 | Medium |
org.opentrafficsim.draw.egtf.Egtf$DualWeightedMean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.opentrafficsim.draw.egtf.Egtf$DualWeightedMean be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1076-1152 | Medium |
org.opentrafficsim.draw.egtf.Egtf$WeightedMean
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.opentrafficsim.draw.egtf.Egtf$WeightedMean be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 1161-1193 | Medium |
org.opentrafficsim.draw.egtf.FilterDouble
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.egtf.FilterDouble.getLocation() may expose internal representation by returning FilterDouble.location | MALICIOUS_CODE | EI_EXPOSE_REP | 43 | Medium |
| org.opentrafficsim.draw.egtf.FilterDouble.getTime() may expose internal representation by returning FilterDouble.time | MALICIOUS_CODE | EI_EXPOSE_REP | 50 | Medium |
org.opentrafficsim.draw.graphs.AbstractContourPlot
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.graphs.AbstractContourPlot at new org.opentrafficsim.draw.graphs.AbstractContourPlot(String, PlotScheduler, ContourDataSource, Number, String, Number, 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 | 97 | Medium |
| org.opentrafficsim.draw.graphs.AbstractContourPlot.getBlockRenderer() may expose internal representation by returning AbstractContourPlot.blockRenderer | MALICIOUS_CODE | EI_EXPOSE_REP | 347 | Medium |
| org.opentrafficsim.draw.graphs.AbstractContourPlot.getDataPool() may expose internal representation by returning AbstractContourPlot.dataPool | MALICIOUS_CODE | EI_EXPOSE_REP | 199 | Medium |
| new org.opentrafficsim.draw.graphs.AbstractContourPlot(String, PlotScheduler, ContourDataSource, BoundsPaintScale, Number, String, String) may expose internal representation by storing an externally mutable object into AbstractContourPlot.dataPool | MALICIOUS_CODE | EI_EXPOSE_REP2 | 70 | Medium |
org.opentrafficsim.draw.graphs.AbstractGraphSpace
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.graphs.AbstractGraphSpace(List) may expose internal representation by storing an externally mutable object into AbstractGraphSpace.seriesNames | MALICIOUS_CODE | EI_EXPOSE_REP2 | 29 | Medium |
org.opentrafficsim.draw.graphs.AbstractPlot
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.AbstractPlot.getChart() may expose internal representation by returning AbstractPlot.chart | MALICIOUS_CODE | EI_EXPOSE_REP | 230 | Medium |
| org.opentrafficsim.draw.graphs.AbstractPlot.getUpdateTime() may expose internal representation by returning AbstractPlot.updateTime | MALICIOUS_CODE | EI_EXPOSE_REP | 273 | Medium |
| new org.opentrafficsim.draw.graphs.AbstractPlot(PlotScheduler, String, Duration, Duration) may expose internal representation by storing an externally mutable object into AbstractPlot.delay | MALICIOUS_CODE | EI_EXPOSE_REP2 | 100 | Medium |
| new org.opentrafficsim.draw.graphs.AbstractPlot(PlotScheduler, String, Duration, Duration) may expose internal representation by storing an externally mutable object into AbstractPlot.updateInterval | MALICIOUS_CODE | EI_EXPOSE_REP2 | 99 | Medium |
| org.opentrafficsim.draw.graphs.AbstractPlot.setUpdateInterval(Duration) may expose internal representation by storing an externally mutable object into AbstractPlot.updateInterval | MALICIOUS_CODE | EI_EXPOSE_REP2 | 262 | Medium |
org.opentrafficsim.draw.graphs.AbstractSamplerPlot
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.graphs.AbstractSamplerPlot(String, Duration, PlotScheduler, SamplerData, GraphPath, Duration) may expose internal representation by storing an externally mutable object into AbstractSamplerPlot.samplerData | MALICIOUS_CODE | EI_EXPOSE_REP2 | 55 | Medium |
org.opentrafficsim.draw.graphs.AbstractSpaceTimePlot
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.graphs.AbstractSpaceTimePlot(String, Duration, PlotScheduler, Duration, Time) may expose internal representation by storing an externally mutable object into AbstractSpaceTimePlot.initialEnd | MALICIOUS_CODE | EI_EXPOSE_REP2 | 49 | Medium |
org.opentrafficsim.draw.graphs.ContourDataSource
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.ContourDataSource.getSamplerData() may expose internal representation by returning ContourDataSource.samplerData | MALICIOUS_CODE | EI_EXPOSE_REP | 246 | Medium |
| new org.opentrafficsim.draw.graphs.ContourDataSource(SamplerData, Duration, GraphPath, double[], int, double[], int, Time, Time) may expose internal representation by storing an externally mutable object into ContourDataSource.delay | MALICIOUS_CODE | EI_EXPOSE_REP2 | 223 | Medium |
| new org.opentrafficsim.draw.graphs.ContourDataSource(SamplerData, Duration, GraphPath, double[], int, double[], int, Time, Time) may expose internal representation by storing an externally mutable object into ContourDataSource.samplerData | MALICIOUS_CODE | EI_EXPOSE_REP2 | 221 | Medium |
| Inconsistent synchronization of org.opentrafficsim.draw.graphs.ContourDataSource.redo; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 782 | Medium |
| Inconsistent synchronization of org.opentrafficsim.draw.graphs.ContourDataSource.smooth; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 441 | Medium |
| Inconsistent synchronization of org.opentrafficsim.draw.graphs.ContourDataSource.toTime; locked 83% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 504 | Medium |
| org.opentrafficsim.draw.graphs.ContourDataSource.DEFAULT_SPACE_GRANULARITIES should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 55 | Medium |
| org.opentrafficsim.draw.graphs.ContourDataSource.DEFAULT_TIME_GRANULARITIES should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 61 | Medium |
| org.opentrafficsim.draw.graphs.ContourDataSource.getGranularity(ContourDataSource$Dimension) is unsynchronized, org.opentrafficsim.draw.graphs.ContourDataSource.setGranularity(ContourDataSource$Dimension, double) is synchronized | MT_CORRECTNESS | UG_SYNC_SET_UNSYNC_GET | 362 | Medium |
| org.opentrafficsim.draw.graphs.ContourDataSource.update(Time) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 759 | Medium |
| org.opentrafficsim.draw.graphs.ContourDataSource.update(Time) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 776 | Medium |
org.opentrafficsim.draw.graphs.FundamentalDiagram
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.graphs.FundamentalDiagram at new org.opentrafficsim.draw.graphs.FundamentalDiagram(String, FundamentalDiagram$Quantity, FundamentalDiagram$Quantity, PlotScheduler, FundamentalDiagram$FdSource, FundamentalDiagram$FdLine) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 101 | Medium |
| org.opentrafficsim.draw.graphs.FundamentalDiagram.getLaneVisible() may expose internal representation by returning FundamentalDiagram.laneVisible | MALICIOUS_CODE | EI_EXPOSE_REP | 1429 | Medium |
| org.opentrafficsim.draw.graphs.FundamentalDiagram.getLegend() may expose internal representation by returning FundamentalDiagram.legend | MALICIOUS_CODE | EI_EXPOSE_REP | 1420 | Medium |
| org.opentrafficsim.draw.graphs.FundamentalDiagram.getSource() may expose internal representation by returning FundamentalDiagram.source | MALICIOUS_CODE | EI_EXPOSE_REP | 1411 | Medium |
| new org.opentrafficsim.draw.graphs.FundamentalDiagram(String, FundamentalDiagram$Quantity, FundamentalDiagram$Quantity, PlotScheduler, FundamentalDiagram$FdSource, FundamentalDiagram$FdLine) may expose internal representation by storing an externally mutable object into FundamentalDiagram.source | MALICIOUS_CODE | EI_EXPOSE_REP2 | 110 | Medium |
| org.opentrafficsim.draw.graphs.FundamentalDiagram.DEFAULT_PERIODS should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 53 | Medium |
| org.opentrafficsim.draw.graphs.FundamentalDiagram.DEFAULT_UPDATE_FREQUENCIES should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 56 | Medium |
org.opentrafficsim.draw.graphs.FundamentalDiagram$AbstractSpaceSamplerFdSource
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of org.opentrafficsim.draw.graphs.FundamentalDiagram$AbstractSpaceSamplerFdSource.firstMeasurement; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 1148 | Medium |
| Inconsistent synchronization of org.opentrafficsim.draw.graphs.FundamentalDiagram$AbstractSpaceSamplerFdSource.secondMeasurement; locked 71% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 1148 | Medium |
org.opentrafficsim.draw.graphs.FundamentalDiagram$MultiFdSource
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.graphs.FundamentalDiagram$MultiFdSource at new org.opentrafficsim.draw.graphs.FundamentalDiagram$MultiFdSource(Map) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 1216 | Medium |
| Possible null pointer dereference of sources in new org.opentrafficsim.draw.graphs.FundamentalDiagram$MultiFdSource(Map) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 1218 | Medium |
org.opentrafficsim.draw.graphs.GraphCrossSection
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.graphs.GraphCrossSection(List, GraphPath$Section, List) may expose internal representation by storing an externally mutable object into GraphCrossSection.positions | MALICIOUS_CODE | EI_EXPOSE_REP2 | 69 | Medium |
org.opentrafficsim.draw.graphs.GraphPath
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.GraphPath.getSpeedLimit() may expose internal representation by returning GraphPath.speedLimit | MALICIOUS_CODE | EI_EXPOSE_REP | 116 | Medium |
| org.opentrafficsim.draw.graphs.GraphPath.getTotalLength() may expose internal representation by returning GraphPath.totalLength | MALICIOUS_CODE | EI_EXPOSE_REP | 107 | Medium |
| new org.opentrafficsim.draw.graphs.GraphPath(List, List) may expose internal representation by storing an externally mutable object into GraphPath.sections | MALICIOUS_CODE | EI_EXPOSE_REP2 | 73 | Medium |
org.opentrafficsim.draw.graphs.GraphPath$Section
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.GraphPath$Section.length() may expose internal representation by returning GraphPath$Section.length | MALICIOUS_CODE | EI_EXPOSE_REP | 235 | Medium |
| org.opentrafficsim.draw.graphs.GraphPath$Section.sections() may expose internal representation by returning GraphPath$Section.sections | MALICIOUS_CODE | EI_EXPOSE_REP | 235 | Medium |
| org.opentrafficsim.draw.graphs.GraphPath$Section.speedLimit() may expose internal representation by returning GraphPath$Section.speedLimit | MALICIOUS_CODE | EI_EXPOSE_REP | 235 | Medium |
| new org.opentrafficsim.draw.graphs.GraphPath$Section(Length, Speed, List) may expose internal representation by storing an externally mutable object into GraphPath$Section.length | MALICIOUS_CODE | EI_EXPOSE_REP2 | 235 | Medium |
| new org.opentrafficsim.draw.graphs.GraphPath$Section(Length, Speed, List) may expose internal representation by storing an externally mutable object into GraphPath$Section.sections | MALICIOUS_CODE | EI_EXPOSE_REP2 | 235 | Medium |
| new org.opentrafficsim.draw.graphs.GraphPath$Section(Length, Speed, List) may expose internal representation by storing an externally mutable object into GraphPath$Section.speedLimit | MALICIOUS_CODE | EI_EXPOSE_REP2 | 235 | Medium |
org.opentrafficsim.draw.graphs.GraphUpdater
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exceptional return value of java.util.concurrent.BlockingQueue.offer(Object) ignored in org.opentrafficsim.draw.graphs.GraphUpdater.offer(Object) | BAD_PRACTICE | RV_RETURN_VALUE_IGNORED_BAD_PRACTICE | 68 | Medium |
org.opentrafficsim.draw.graphs.TrajectoryPlot
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.TrajectoryPlot.getLaneVisible() may expose internal representation by returning TrajectoryPlot.laneVisible | MALICIOUS_CODE | EI_EXPOSE_REP | 596 | Medium |
| org.opentrafficsim.draw.graphs.TrajectoryPlot.getLegend() may expose internal representation by returning TrajectoryPlot.legend | MALICIOUS_CODE | EI_EXPOSE_REP | 587 | Medium |
org.opentrafficsim.draw.graphs.TrajectoryPlot$OffsetTrajectory
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.opentrafficsim.draw.graphs.TrajectoryPlot$OffsetTrajectory be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 472-567 | Medium |
org.opentrafficsim.draw.graphs.XyInterpolatedBlockRenderer
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.graphs.XyInterpolatedBlockRenderer doesn't override org.jfree.chart.renderer.xy.XYBlockRenderer.equals(Object) | STYLE | EQ_DOESNT_OVERRIDE_EQUALS | 1 | Medium |
org.opentrafficsim.draw.graphs.XyInterpolatedBlockRenderer$1$1
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Integral division result cast to double or float in org.opentrafficsim.draw.graphs.XyInterpolatedBlockRenderer$1$1.getRaster(int, int, int, int) | STYLE | ICAST_IDIV_CAST_TO_DOUBLE | 182 | Medium |
org.opentrafficsim.draw.gtu.DefaultCarAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.gtu.DefaultCarAnimation$Text(DefaultCarAnimation, DefaultCarAnimation$GtuData, Supplier, float, float, TextAlignment, Color, Contextualized) may expose internal representation by storing an externally mutable object into DefaultCarAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 242 | Medium |
| new org.opentrafficsim.draw.gtu.DefaultCarAnimation$Text(DefaultCarAnimation, DefaultCarAnimation$GtuData, Supplier, float, float, TextAlignment, Color, Contextualized, TextAnimation$ContrastToBackground) may expose internal representation by storing an externally mutable object into DefaultCarAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 262 | Medium |
| Should org.opentrafficsim.draw.gtu.DefaultCarAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 226-270 | Medium |
org.opentrafficsim.draw.network.LinkAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.network.LinkAnimation at new org.opentrafficsim.draw.network.LinkAnimation(LinkAnimation$LinkData, Contextualized, float) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 78 | Medium |
org.opentrafficsim.draw.network.LinkAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.network.LinkAnimation$Text(LinkAnimation, LinkAnimation$LinkData, Supplier, float, float, TextAlignment, Color, Contextualized, TextAnimation$ScaleDependentRendering) may expose internal representation by storing an externally mutable object into LinkAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 187 | Medium |
| Should org.opentrafficsim.draw.network.LinkAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 187-195 | Medium |
org.opentrafficsim.draw.network.NodeAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.network.NodeAnimation$Text(NodeAnimation, NodeAnimation$NodeData, Supplier, float, float, TextAlignment, Color, Contextualized, TextAnimation$ScaleDependentRendering) may expose internal representation by storing an externally mutable object into NodeAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 123 | Medium |
| Should org.opentrafficsim.draw.network.NodeAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 123-133 | Medium |
org.opentrafficsim.draw.road.BusStopAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.road.BusStopAnimation.getText() may expose internal representation by returning BusStopAnimation.text | MALICIOUS_CODE | EI_EXPOSE_REP | 59 | Medium |
org.opentrafficsim.draw.road.BusStopAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.road.BusStopAnimation$Text(BusStopAnimation, BusStopAnimation$BusStopData, Supplier, float, float, TextAlignment, Color, Contextualized) may expose internal representation by storing an externally mutable object into BusStopAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 115 | Medium |
| Should org.opentrafficsim.draw.road.BusStopAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 115-123 | Medium |
org.opentrafficsim.draw.road.GtuGeneratorPositionAnimation$Queue
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.road.GtuGeneratorPositionAnimation$Queue(GtuGeneratorPositionAnimation, GtuGeneratorPositionAnimation$GtuGeneratorPositionData, Contextualized) may expose internal representation by storing an externally mutable object into GtuGeneratorPositionAnimation$Queue.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 113 | Medium |
| Should org.opentrafficsim.draw.road.GtuGeneratorPositionAnimation$Queue be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 113-116 | Medium |
org.opentrafficsim.draw.road.LaneAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.road.LaneAnimation.getText() may expose internal representation by returning LaneAnimation.text | MALICIOUS_CODE | EI_EXPOSE_REP | 75 | Medium |
org.opentrafficsim.draw.road.LaneAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.road.LaneAnimation$Text(LaneAnimation, LaneAnimation$LaneData, Supplier, float, float, TextAlignment, Color, Contextualized) may expose internal representation by storing an externally mutable object into LaneAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 231 | Medium |
| Should org.opentrafficsim.draw.road.LaneAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 231-239 | Medium |
org.opentrafficsim.draw.road.LaneDetectorAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.road.LaneDetectorAnimation.getText() may expose internal representation by returning LaneDetectorAnimation.text | MALICIOUS_CODE | EI_EXPOSE_REP | 102 | Medium |
org.opentrafficsim.draw.road.StripeAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Exception thrown in class org.opentrafficsim.draw.road.StripeAnimation at new org.opentrafficsim.draw.road.StripeAnimation(StripeAnimation$StripeData, Contextualized) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks. | BAD_PRACTICE | CT_CONSTRUCTOR_THROW | 54 | Medium |
org.opentrafficsim.draw.road.TrafficLightAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.opentrafficsim.draw.road.TrafficLightAnimation.getText() may expose internal representation by returning TrafficLightAnimation.text | MALICIOUS_CODE | EI_EXPOSE_REP | 59 | Medium |
org.opentrafficsim.draw.road.TrafficLightAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.road.TrafficLightAnimation$Text(TrafficLightAnimation, TrafficLightAnimation$TrafficLightData, Supplier, float, float, TextAlignment, Color, Contextualized) may expose internal representation by storing an externally mutable object into TrafficLightAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 117 | Medium |
| Should org.opentrafficsim.draw.road.TrafficLightAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 117-125 | Medium |
org.opentrafficsim.draw.road.TrafficLightDetectorAnimation
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.opentrafficsim.draw.road.TrafficLightDetectorAnimation defines non-transient non-serializable instance field detector | BAD_PRACTICE | SE_BAD_FIELD | Not available | Medium |
org.opentrafficsim.draw.road.TrafficLightDetectorAnimation$Text
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| new org.opentrafficsim.draw.road.TrafficLightDetectorAnimation$Text(TrafficLightDetectorAnimation, TrafficLightDetectorAnimation$TrafficLightDetectorData, Supplier, float, float, TextAlignment, Color, Contextualized) may expose internal representation by storing an externally mutable object into TrafficLightDetectorAnimation$Text.this$0 | MALICIOUS_CODE | EI_EXPOSE_REP2 | 121 | Medium |
| Should org.opentrafficsim.draw.road.TrafficLightDetectorAnimation$Text be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 121-129 | Medium |