From 5d9d88c44d36eb16a6e31c03f82acbb1bc68cd5d Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Tue, 10 Nov 2015 23:43:40 +0100 Subject: [PATCH] CommonsMultipartFile removes mixed separator paths from original filename Issue: SPR-13662 --- .../web/multipart/commons/CommonsMultipartFile.java | 11 ++++++----- .../commons/CommonsMultipartResolverTests.java | 4 ++-- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/spring-web/src/main/java/org/springframework/web/multipart/commons/CommonsMultipartFile.java b/spring-web/src/main/java/org/springframework/web/multipart/commons/CommonsMultipartFile.java index 5e77e4c46d..ac7d9f11f4 100644 --- a/spring-web/src/main/java/org/springframework/web/multipart/commons/CommonsMultipartFile.java +++ b/spring-web/src/main/java/org/springframework/web/multipart/commons/CommonsMultipartFile.java @@ -78,12 +78,13 @@ public class CommonsMultipartFile implements MultipartFile, Serializable { // Should never happen. return ""; } + // Check for Unix-style path - int pos = filename.lastIndexOf("/"); - if (pos == -1) { - // Check for Windows-style path - pos = filename.lastIndexOf("\\"); - } + int unixSep = filename.lastIndexOf("/"); + // Check for Windows-style path + int winSep = filename.lastIndexOf("\\"); + // Cut off at latest possible point + int pos = (winSep > unixSep ? winSep : unixSep); if (pos != -1) { // Any sort of path separator found... return filename.substring(pos + 1); diff --git a/spring-web/src/test/java/org/springframework/web/multipart/commons/CommonsMultipartResolverTests.java b/spring-web/src/test/java/org/springframework/web/multipart/commons/CommonsMultipartResolverTests.java index b305c833f4..e0c963df8d 100644 --- a/spring-web/src/test/java/org/springframework/web/multipart/commons/CommonsMultipartResolverTests.java +++ b/spring-web/src/test/java/org/springframework/web/multipart/commons/CommonsMultipartResolverTests.java @@ -383,9 +383,9 @@ public class CommonsMultipartResolverTests { MockFileItem fileItem1x = new MockFileItem( "field1", "type1", empty ? "" : "field1.txt", empty ? "" : "text1"); MockFileItem fileItem2 = new MockFileItem( - "field2", "type2", empty ? "" : "C:/field2.txt", empty ? "" : "text2"); + "field2", "type2", empty ? "" : "C:\\mypath/field2.txt", empty ? "" : "text2"); MockFileItem fileItem2x = new MockFileItem( - "field2x", "type2", empty ? "" : "C:\\field2x.txt", empty ? "" : "text2"); + "field2x", "type2", empty ? "" : "C:/mypath\\field2x.txt", empty ? "" : "text2"); MockFileItem fileItem3 = new MockFileItem("field3", null, null, "value3"); MockFileItem fileItem4 = new MockFileItem("field4", "text/html; charset=iso-8859-1", null, "value4"); MockFileItem fileItem5 = new MockFileItem("field4", null, null, "value5");