The following document contains the results of FindBugs Report
FindBugs Version is 2.0.1
Threshold is medium
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.deegree.geometry.Geometries.collectMembersDeep(Geometry, List) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 167 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to tmp in org.deegree.geometry.GeometryTransformer.transform(Points, Transformation) | STYLE | DLS_DEAD_LOCAL_STORE | 822 | Medium |
| Dead store to result in org.deegree.geometry.GeometryTransformer.transform(Point, Transformation) | STYLE | DLS_DEAD_LOCAL_STORE | 849 | Medium |
| Dead store to c2 in org.deegree.geometry.GeometryTransformer.transform(CurveSegment, Transformation) | STYLE | DLS_DEAD_LOCAL_STORE | 691 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Null passed for nonnull parameter of writePolygonWithoutPrefix(Surface, Writer) in org.deegree.geometry.io.WKTWriter.writePolygonPatch(String, ICRS, PrecisionModel, PolygonPatch, Writer) | CORRECTNESS | NP_NULL_PARAM_DEREF | 456 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of solution in org.deegree.geometry.linearization.CurveLinearizer.solveLinearEquation(double[][], double[]) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 472 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Inconsistent synchronization of org.deegree.geometry.standard.DefaultEnvelope.transformer; locked 66% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 149 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Check for oddness that won't work for negative numbers in new org.deegree.geometry.standard.curvesegments.DefaultArcString(Points) | STYLE | IM_BAD_CHECK_FOR_ODD | 63 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.deegree.geometry.standard.curvesegments.DefaultArcStringByBulge.getBulges() may expose internal representation by returning DefaultArcStringByBulge.bulges | MALICIOUS_CODE | EI_EXPOSE_REP | 86 | Medium |
| new org.deegree.geometry.standard.curvesegments.DefaultArcStringByBulge(Points, double[], Points) may expose internal representation by storing an externally mutable object into DefaultArcStringByBulge.bulges | MALICIOUS_CODE | EI_EXPOSE_REP2 | 80 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.deegree.geometry.standard.points.PackedPoints.getAsArray() may expose internal representation by returning PackedPoints.coordinates | MALICIOUS_CODE | EI_EXPOSE_REP | 135 | Medium |
| new org.deegree.geometry.standard.points.PackedPoints(ICRS, double[], int) may expose internal representation by storing an externally mutable object into PackedPoints.coordinates | MALICIOUS_CODE | EI_EXPOSE_REP2 | 69 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.deegree.geometry.standard.points.PointsArray.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 203 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.deegree.geometry.standard.points.PointsList.clone() does not call super.clone() | BAD_PRACTICE | CN_IDIOM_NO_SUPER_CALL | 191 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unwritten field: org.deegree.geometry.standard.primitive.DefaultOrientableCurve.crs | CORRECTNESS | UWF_UNWRITTEN_FIELD | 99 | Medium |
| Unwritten field: org.deegree.geometry.standard.primitive.DefaultOrientableCurve.id | CORRECTNESS | UWF_UNWRITTEN_FIELD | 94 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unwritten field: org.deegree.geometry.standard.primitive.DefaultOrientableSurface.crs | CORRECTNESS | UWF_UNWRITTEN_FIELD | 97 | Medium |
| Unwritten field: org.deegree.geometry.standard.primitive.DefaultOrientableSurface.id | CORRECTNESS | UWF_UNWRITTEN_FIELD | 92 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.deegree.geometry.standard.primitive.DefaultPoint.getAsArray() may expose internal representation by returning DefaultPoint.coordinates | MALICIOUS_CODE | EI_EXPOSE_REP | 97 | Medium |
| new org.deegree.geometry.standard.primitive.DefaultPoint(String, ICRS, PrecisionModel, double[]) may expose internal representation by storing an externally mutable object into DefaultPoint.coordinates | MALICIOUS_CODE | EI_EXPOSE_REP2 | 74 | Medium |
| org.deegree.geometry.standard.primitive.DefaultPoint.toString() concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 154 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unused field: org.deegree.geometry.standard.primitive.DefaultPolygon.envelope | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultCone.columns | PERFORMANCE | URF_UNREAD_FIELD | 65 | Medium |
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultCone.rows | PERFORMANCE | URF_UNREAD_FIELD | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultCylinder.columns | PERFORMANCE | URF_UNREAD_FIELD | 65 | Medium |
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultCylinder.rows | PERFORMANCE | URF_UNREAD_FIELD | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultGriddedSurfacePatch.columns | PERFORMANCE | URF_UNREAD_FIELD | 65 | Medium |
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultGriddedSurfacePatch.rows | PERFORMANCE | URF_UNREAD_FIELD | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultSphere.columns | PERFORMANCE | URF_UNREAD_FIELD | 65 | Medium |
| Unread field: org.deegree.geometry.standard.surfacepatches.DefaultSphere.rows | PERFORMANCE | URF_UNREAD_FIELD | 64 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Switch statement found in org.deegree.geometry.utils.GeometryUtils.measurePathLengths(Shape) where one case falls through to the next case | STYLE | SF_SWITCH_FALLTHROUGH | 273 | Medium |
| Switch statement found in org.deegree.geometry.utils.GeometryUtils.measurePathLengths(Shape) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 265 | Medium |
| Switch statement found in org.deegree.geometry.utils.GeometryUtils.prettyPrintShape(Shape) where default case is missing | STYLE | SF_SWITCH_NO_DEFAULT | 205 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of fixedCurve in org.deegree.geometry.validation.GeometryFixer.fixCurve(Curve, Point) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 172 | Medium |
| Possible null pointer dereference of repaired in org.deegree.geometry.validation.GeometryFixer.fixUnclosedRing(Ring) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 106 | Medium |
| Possible null pointer dereference of fixedCurve in org.deegree.geometry.validation.GeometryFixer.invertOrientation(Curve) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 231 | Medium |
| Possible null pointer dereference of fixedRing in org.deegree.geometry.validation.GeometryFixer.invertOrientation(Ring) on exception path | CORRECTNESS | NP_NULL_ON_SOME_PATH_EXCEPTION | 254 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Doomed test for equality to NaN in org.deegree.geometry.validation.GeometryValidator.getPoint(Coordinate, ICRS) | CORRECTNESS | FE_TEST_IF_EQUAL_TO_NOT_A_NUMBER | 522 | High |
