Merge pull request #247 from janjonas/DNN-5206

DNN-5206: DotNetNuke.Services.FileSystemFileManager.WriteFileToHttpContext(): Do not send negative "Content-Length" header
dnnsoftware · Mar 13, 2015 · 55d5765 · 55d5765
2 parents 6c198bb + 128cb07
commit 55d5765
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/DNN Platform/Library/Services/FileSystem/FileManager.cs b/DNN Platform/Library/Services/FileSystem/FileManager.cs
@@ -22,6 +22,7 @@
 using System;
 using System.Collections.Generic;
 using System.Drawing;
+using System.Globalization;
 using System.IO;
 using System.Linq;
 using System.Security.Cryptography;
@@ -1777,7 +1778,8 @@ internal virtual void WriteFileToHttpContext(IFileInfo file, ContentDisposition
                     throw new ArgumentOutOfRangeException("contentDisposition");
             }
 
-            objResponse.AppendHeader("Content-Length", file.Size.ToString());
+            // Do not send negative Content-Length (file.Size could be negative due to integer overflow for files > 2GB)
+            if (file.Size >= 0) objResponse.AppendHeader("Content-Length", file.Size.ToString(CultureInfo.InvariantCulture));
             objResponse.ContentType = GetContentType(file.Extension.Replace(".", ""));
 
             try