From 3dc635c6fdeae6024be3510c18821d6f9dc5b550 Mon Sep 17 00:00:00 2001 From: lookshe Date: Sat, 12 Mar 2016 15:04:07 +0100 Subject: small bugfixes --- src/main/java/eu/siacs/conversations/entities/Message.java | 3 +++ src/main/java/eu/siacs/conversations/utils/FileUtils.java | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/main/java/eu/siacs/conversations/entities/Message.java b/src/main/java/eu/siacs/conversations/entities/Message.java index 12127ecc..184d7e56 100644 --- a/src/main/java/eu/siacs/conversations/entities/Message.java +++ b/src/main/java/eu/siacs/conversations/entities/Message.java @@ -470,6 +470,9 @@ public class Message extends AbstractEntity { } private String extractRelevantExtension(URL url) { + if (url == null) { + return null; + } String path = url.getPath(); return extractRelevantExtension(path); } diff --git a/src/main/java/eu/siacs/conversations/utils/FileUtils.java b/src/main/java/eu/siacs/conversations/utils/FileUtils.java index 727891ae..7c2a51c5 100644 --- a/src/main/java/eu/siacs/conversations/utils/FileUtils.java +++ b/src/main/java/eu/siacs/conversations/utils/FileUtils.java @@ -167,7 +167,7 @@ public class FileUtils { final int lastDotPosition = filename.lastIndexOf('.'); final int secondToLastDotPosition = filename.lastIndexOf('.', lastDotPosition - 1); final String secondToLastPart = secondToLastDotPosition != -1 ? - filename.substring(secondToLastDotPosition + 1, lastDotPosition - 1) : ""; + filename.substring(secondToLastDotPosition + 1, lastDotPosition) : ""; return secondToLastPart; } } -- cgit v1.2.3