You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2014/02/15 19:20:33 UTC

svn commit: r1568659 - /commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java

Author: sebb
Date: Sat Feb 15 18:20:32 2014
New Revision: 1568659

URL: http://svn.apache.org/r1568659
Log:
Document resource leak warning

Modified:
    commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java

Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java
URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java?rev=1568659&r1=1568658&r2=1568659&view=diff
==============================================================================
--- commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java (original)
+++ commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java Sat Feb 15 18:20:32 2014
@@ -590,7 +590,7 @@ public class MultipartStream {
     public int readBodyData(OutputStream output)
             throws MalformedStreamException, IOException {
         final InputStream istream = newInputStream();
-        return (int) Streams.copy(istream, output, false);
+        return (int) Streams.copy(istream, output, false); // N.B. Streams.copy closes the input stream
     }
 
     /**