Skip to content

Commit 3cb405d

Browse files
committed
Removed Unecessary write to buffer
Removed unnecessary write to the buffer since it sometimes causes out of memory exceptions and you can write the inputStream directly to the file.
1 parent 24df8cc commit 3cb405d

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

jQuery-File-Upload.MVC3/Controllers/HomeController.cs

+3-12
Original file line numberDiff line numberDiff line change
@@ -91,24 +91,15 @@ private string EncodeFile(string fileName)
9191
private void UploadPartialFile(string fileName, HttpRequestBase request, List<ViewDataUploadFilesResult> statuses)
9292
{
9393
if (request.Files.Count != 1) throw new HttpRequestValidationException("Attempt to upload chunked file containing more than one fragment per request");
94-
var file = request.Files[0];
95-
var inputStream = file.InputStream;
9694

95+
var file = request.Files[0];
9796
var fullName = Path.Combine(StorageRoot, Path.GetFileName(fileName));
9897

9998
using (var fs = new FileStream(fullName, FileMode.Append, FileAccess.Write))
10099
{
101-
var buffer = new byte[1024];
102-
103-
var l = inputStream.Read(buffer, 0, 1024);
104-
while (l > 0)
105-
{
106-
fs.Write(buffer, 0, l);
107-
l = inputStream.Read(buffer, 0, 1024);
108-
}
109-
fs.Flush();
110-
fs.Close();
100+
file.InputStream.CopyTo(files);
111101
}
102+
112103
statuses.Add(new ViewDataUploadFilesResult()
113104
{
114105
name = fileName,

0 commit comments

Comments
 (0)