Changeset 18819 in josm for trunk/src/org/openstreetmap/josm/io/GpxParser.java

Timestamp:

2023-08-23T13:53:32+02:00 (15 months ago)

Author:

taylor.smock

Message:

See #22652, r18817: Stop parsing gpx files when point elements have invalid coordinates

This adds back the wpt check that was temporarily removed in r18818 to get back
to the previous "known good" state and additionally adds checks for rtept and
trkpt elements.

This also removes a change to build.xml that was accidentally committed in r18818.
The specific change (adding Automatic-Module-Name to the JOSM manifest) will
probably be done in a separate commit in the future.

File:

• trunk/src/org/openstreetmap/josm/io/GpxParser.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/io/GpxParser.java

@@ -638 +638 @@
                 currentState = states.pop();
                 convertUrlToLink(currentWayPoint.attr);
+                if (!currentWayPoint.isLatLonKnown()) {
+                    throw new SAXException(tr("{0} element does not have valid latitude and/or longitude.", localName));
+                }
                 currentRoute.routePoints.add(currentWayPoint);
                 break;
@@ -643 +646 @@
                 currentState = states.pop();
                 convertUrlToLink(currentWayPoint.attr);
+                if (!currentWayPoint.isLatLonKnown()) {
+                    throw new SAXException(tr("{0} element does not have valid latitude and/or longitude.", localName));
+                }
                 currentTrackSeg.add(currentWayPoint);
                 break;
@@ -649 +655 @@
                 convertUrlToLink(currentWayPoint.attr);
                 currentWayPoint.getExtensions().addAll(currentExtensionCollection);
+                if (!currentWayPoint.isLatLonKnown()) {
+                    currentExtensionCollection.clear();
+                    throw new SAXException(tr("{0} element does not have valid latitude and/or longitude.", localName));
+                }
                 data.waypoints.add(currentWayPoint);
                 currentExtensionCollection.clear();