Changeset 16692 in josm for trunk

Timestamp:

2020-06-21T10:02:29+02:00 (4 years ago)

Author:

simon04

Message:

fix #19422 - IAE in TMSCachedTileLoaderJob.parseHeaders

File:

• trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

@@ -261 +261 @@
         // keep the expiration time between MINIMUM_EXPIRES and MAXIMUM_EXPIRES, so we will cache the tiles
         // at least for some short period of time, but not too long
-        ret.setExpirationTime((long) Utils.clamp(
-                ret.getExpirationTime(),
-                now + Math.max(MINIMUM_EXPIRES.get(), TimeUnit.SECONDS.toMillis(options.getMinimumExpiryTime())),
-                now + Math.max(MAXIMUM_EXPIRES.get(), TimeUnit.SECONDS.toMillis(options.getMinimumExpiryTime()))
-        ));
+        long minimumExpiryTime = TimeUnit.SECONDS.toMillis(options.getMinimumExpiryTime());
+        long nowPlusMin = now + Math.max(MINIMUM_EXPIRES.get(), minimumExpiryTime);
+        if (ret.getExpirationTime() < nowPlusMin) {
+            ret.setExpirationTime(nowPlusMin);
+        }
+        long nowPlusMax = now + Math.max(MAXIMUM_EXPIRES.get(), minimumExpiryTime);
+        if (ret.getExpirationTime() > nowPlusMax) {
+            ret.setExpirationTime(nowPlusMax);
+        }
         return ret;
     }