From c6cc6705ef21f01be046374fd90c4d62567d1b73 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Tue, 8 Sep 2020 12:23:27 +0200 Subject: [PATCH] FileUrlResource.isWritable() exclusively relies on getFile() implementation Closes gh-25584 --- .../org/springframework/core/io/FileUrlResource.java | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/spring-core/src/main/java/org/springframework/core/io/FileUrlResource.java b/spring-core/src/main/java/org/springframework/core/io/FileUrlResource.java index 4e1b0fa809..b910bbc916 100644 --- a/spring-core/src/main/java/org/springframework/core/io/FileUrlResource.java +++ b/spring-core/src/main/java/org/springframework/core/io/FileUrlResource.java @@ -89,15 +89,8 @@ public class FileUrlResource extends UrlResource implements WritableResource { @Override public boolean isWritable() { try { - URL url = getURL(); - if (ResourceUtils.isFileURL(url)) { - // Proceed with file system resolution - File file = getFile(); - return (file.canWrite() && !file.isDirectory()); - } - else { - return true; - } + File file = getFile(); + return (file.canWrite() && !file.isDirectory()); } catch (IOException ex) { return false;