Changeset 32130 in osm for applications/editors/josm/plugins/opendata/src

Timestamp:

2016-03-27T23:17:02+02:00 (9 years ago)

Author:

donvip

Message:

fix #josm12694 - robustness to invalid KML colors

File:

• applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/KmlReader.java (modified) (5 diffs)

applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/KmlReader.java

@@ -9 +9 @@
 import java.util.List;
 import java.util.Map;
+import java.util.regex.Pattern;
 
 import javax.xml.stream.FactoryConfigurationError;
@@ -32 +33 @@
 
     public static final String KML_PLACEMARK   = "Placemark";
-    public static final String KML_NAME           = "name";
+    public static final String KML_NAME        = "name";
     public static final String KML_COLOR       = "color";
     public static final String KML_SIMPLE_DATA = "SimpleData";
@@ -43 +44 @@
     public static final String KML_COORDINATES = "coordinates";
     
+    public static Pattern COLOR_PATTERN = Pattern.compile("\\p{XDigit}{8}");
 
     private XMLStreamReader parser;
@@ -54 +56 @@
         InputStreamReader ir = UTFInputStreamReader.create(in);
         XMLStreamReader parser = XMLInputFactory.newInstance().createXMLStreamReader(ir);
-        //XMLStreamReader parser = XMLInputFactory.newInstance().createXMLStreamReader(in, UTF8);
         return new KmlReader(parser).parseDoc();
     }
@@ -92 +93 @@
                 if (parser.getLocalName().equals(KML_COLOR)) {
                     String s = parser.getElementText();
-                    // KML color format is aabbggrr, convert it to OSM (web) format: #rrggbb
-                    String rgbColor = '#'+s.substring(6,8)+s.substring(4,6)+s.substring(2,4);
-                    tags.put(KML_COLOR, rgbColor);
+                    if (COLOR_PATTERN.matcher(s).matches()) {
+                        // KML color format is aabbggrr, convert it to OSM (web) format: #rrggbb
+                        tags.put(KML_COLOR, '#'+s.substring(6,8)+s.substring(4,6)+s.substring(2,4));
+                    }
                 } else if (parser.getLocalName().equals(KML_NAME)) {
                     tags.put(KML_NAME, parser.getElementText());