andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #399 from fredericBregier/3

Browse Source 
Proposal to fix issue #398 by replacing transferTo by using loop on FileChannel
This commit is contained in:
Frédéric Brégier 2012-06-15 03:04:34 -07:00
commit 6d29d2feb1
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/main/java/org/jboss/netty/handler/codec/http/multipart/AbstractDiskHttpData.java
View File

@ -289,10 +289,17 @@ public abstract class AbstractDiskHttpData extends AbstractHttpData {
FileOutputStream outputStream = new FileOutputStream(dest); FileOutputStream outputStream = new FileOutputStream(dest);
FileChannel in = inputStream.getChannel(); FileChannel in = inputStream.getChannel();
FileChannel out = outputStream.getChannel(); FileChannel out = outputStream.getChannel();
long destsize = in.transferTo(0, size, out); int chunkSize = 8196;
long position = 0;
while (position < size) {
if (chunksize < size - position) {
chunksize = size - position;
}
position += in.transferTo(position, chunkSize , out);
}
in.close(); in.close();
out.close(); out.close();
if (destsize == size) { if (position == size) {
file.delete(); file.delete();
file = dest; file = dest;
isRenamed = true; isRenamed = true;