You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by am...@apache.org on 2016/01/06 18:32:34 UTC
[2/3] struts git commit: WW-4583 JakartaStreamMultiPartRequest fails
on short file name and on empty file input
WW-4583 JakartaStreamMultiPartRequest fails on short file name and on empty file input
(cherry picked from commit a4f8d492144a62af2a7d5e934e62d737bdfc8058)
Project: http://git-wip-us.apache.org/repos/asf/struts/repo
Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/9a0a0a16
Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/9a0a0a16
Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/9a0a0a16
Branch: refs/heads/support-2-3
Commit: 9a0a0a16e9a61b55d3906b9b7059ebc3356efab0
Parents: bf46145
Author: Aleksandr Mashchenko <am...@apache.org>
Authored: Wed Jan 6 19:24:56 2016 +0200
Committer: Aleksandr Mashchenko <am...@apache.org>
Committed: Wed Jan 6 19:26:18 2016 +0200
----------------------------------------------------------------------
.../multipart/JakartaStreamMultiPartRequest.java | 10 ++++++++++
1 file changed, 10 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/struts/blob/9a0a0a16/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java
index 6fbd2c3..1270e07 100644
--- a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java
+++ b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaStreamMultiPartRequest.java
@@ -392,6 +392,12 @@ public class JakartaStreamMultiPartRequest implements MultiPartRequest {
* @param location
*/
private void processFileItemStreamAsFileField(FileItemStream itemStream, String location) {
+ // Skip file uploads that don't have a file name - meaning that no file was selected.
+ if (itemStream.getName() == null || itemStream.getName().trim().length() < 1) {
+ LOG.debug("No file has been uploaded for the field: {}", itemStream.getFieldName());
+ return;
+ }
+
File file = null;
try {
// Create the temporary upload file.
@@ -433,6 +439,10 @@ public class JakartaStreamMultiPartRequest implements MultiPartRequest {
suffix = name.substring(name.lastIndexOf('.'));
}
+ if (prefix.length() < 3) {
+ prefix = UUID.randomUUID().toString();
+ }
+
File file = File.createTempFile(prefix + "_", suffix, new File(location));
LOG.debug("Creating temporary file '#0' (originally '#1').", file.getName(), fileName);
return file;