Author: markt Date: Tue Mar 19 10:56:17 2013 New Revision: 1458220 URL: http://svn.apache.org/r1458220 Log: There needs to be the same number of place-holders as there are replacements. Fixes an issue introduced in r1453239. Identified by FindBugs running against the Tomcat code base (which has a package renamed copy of FileUpload).
Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/FileUploadBase.java Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/FileUploadBase.java URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/FileUploadBase.java?rev=1458220&r1=1458219&r2=1458220&view=diff ============================================================================== --- commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/FileUploadBase.java (original) +++ commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/FileUploadBase.java Tue Mar 19 10:56:17 2013 @@ -348,7 +348,7 @@ public abstract class FileUploadBase { } catch (FileUploadIOException e) { throw (FileUploadException) e.getCause(); } catch (IOException e) { - throw new IOFileUploadException(format("Processing of %s request failed. ", + throw new IOFileUploadException(format("Processing of %s request failed. %s", MULTIPART_FORM_DATA, e.getMessage()), e); } final FileItemHeaders fih = item.getHeaders();