Changeset 32642 in osm for applications/editors/josm/plugins/mapillary/src

Timestamp:

2016-07-12T17:28:07+02:00 (9 years ago)

Author:

floscher

Message:

Fix NegativeArraySizeException in ImageUtil.ImageFileFilter

This was reported as https://josm.openstreetmap.de/ticket/13096 .
The fix is accompanied by a testcase to prevent future regressions.

File:

• applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/utils/ImageUtil.java (modified) (1 diff)

applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/utils/ImageUtil.java

@@ -146 +146 @@
       try (FileInputStream fis = new FileInputStream(f)) {
         int numBytes = fis.read(magic);
-        return Arrays.equals(JFIF_MAGIC, Arrays.copyOf(magic, Math.min(numBytes, JFIF_MAGIC.length)))
-            || Arrays.equals(PNG_MAGIC, Arrays.copyOf(magic, Math.min(numBytes, PNG_MAGIC.length)));
+        return numBytes >= 0 && (
+          Arrays.equals(JFIF_MAGIC, Arrays.copyOf(magic, Math.min(numBytes, JFIF_MAGIC.length)))
+          || Arrays.equals(PNG_MAGIC, Arrays.copyOf(magic, Math.min(numBytes, PNG_MAGIC.length)))
+        );
       } catch (FileNotFoundException e) {
         return false;
       } catch (IOException e) {
+        Main.warn("IO-exception while reading file "+f.getAbsolutePath());
         return false;
       }