Changeset 6809 in josm for trunk/src/org/openstreetmap/josm/tools

Timestamp:

2014-02-04T15:11:05+01:00 (11 years ago)

Author:

bastiK

Message:

fixed #9674 - Support for escaping special characters

File:

• trunk/src/org/openstreetmap/josm/tools/ImageProvider.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

@@ -26 +26 @@
 import java.net.URL;
 import java.net.URLDecoder;
+import java.net.URLEncoder;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -530 +531 @@
                 String base64 = m.group(2);
                 String data = m.group(3);
-                byte[] bytes = ";base64".equals(base64)
-                        ? Base64.decodeBase64(data)
-                                : URLDecoder.decode(data, "utf-8").getBytes();
+                byte[] bytes;
+                if (";base64".equals(base64)) {
+                    bytes = Base64.decodeBase64(data);
+                } else {
+                    try {
+                        bytes = URLDecoder.decode(data, "utf-8").getBytes();
+                    } catch (IllegalArgumentException ex) {
+                        Main.warn("Unable to decode URL data part: "+ex.getMessage() + " (" + data + ")");
+                        return null;
+                    }
+                }
                 if (mediatype != null && mediatype.contains("image/svg+xml")) {
-                    URI uri = getSvgUniverse().loadSVG(new StringReader(new String(bytes)), url);
-                    return new ImageResource(getSvgUniverse().getDiagram(uri));
+                    String s = new String(bytes);
+                    URI uri = getSvgUniverse().loadSVG(new StringReader(s), URLEncoder.encode(s, "UTF-8"));
+                    SVGDiagram svg = getSvgUniverse().getDiagram(uri);
+                    if (svg == null) {
+                        Main.warn("Unable to process svg: "+s);
+                        return null;
+                    }
+                    return new ImageResource(svg);
                 } else {
                     try {