why are docx files being corrupted when downloading from an asp net page

why are docx files being corrupted when downloading from an asp net page  using -'asp.net,httpresponse,docx'

I have this following code for bringing page attachments to the user:

private void GetFile(string package, string filename)
    var stream = new MemoryStream();

        using (ZipFile zip = ZipFile.Read(package))
    catch (System.Exception ex)
        throw new Exception("Resources_FileNotFound", ex);

    Response.ContentType = "application/unknown";

    if (filename.EndsWith(".docx"))
        Response.ContentType = "application/vnd.openxmlformats-officedocument.wordprocessingml.document";

    Response.AddHeader("Content-Disposition", "attachment;filename=\"" + filename + "\"");

The problem is that all supported files works properly (jpg, gif, png, pdf, doc, etc), but .docx files, when downloaded, are corrupted and they need to be fixed by Office in order to be opened.

At first I didn't know if the problem was at uncompressing the zip file that contained the .docx, so instead of putting the output file only in the response, I saved it first, and the file opened successfully, so I know the problem should be at response writing.

Do you know what can be happening?

asked Sep 24, 2015 by r3tt
0 votes

8 Answers

0 votes

I also ran into this problem and actually found the answer here: http://www.aspmessageboard.com/showthread.php?t=230778

It turns out that the docx format needs to have Response.End() right after the Response.BinaryWrite.

answered Sep 24, 2015 by deven.bendale
0 votes

When storing a binary file in SQL Server, keep in mind that a file is padded to the nearest word boundry, so you can potentially have an extra byte added to a file. The solution is to store the original file size in the db when you store the file, and use that for the length that needs to be passed to the write function of the Stream object. "Stream.Write(bytes(), 0, length)". This is the ONLY reliable way of getting the correct file size, which is very important for Office 2007 and up files, which do not allow extra characters to be on the end of them (most other file types like jpg's don't care).

answered Sep 24, 2015 by vibhorsingh
0 votes

You should not use stream.GetBuffer() because it returns the buffer array which might contain unused bytes. Use stream.ToArray() instead. Also, have you tried calling stream.Seek(0, SeekOrigin.Begin) before writing anything?

Best Regards,
Oliver Hanappi

answered Sep 24, 2015 by rajnipancholi
0 votes

If you use the approach above which uses response.Close(), Download managers such as IE10 will say 'cannot download file' because the byte lengths do not match the headers. See the documentation. Do NOT use response.Close. EVER. However, using the CompeteRequest verb alone does not shut off the writing of bytes to the ouput stream so XML based applications such as WORD 2007 will see the docx as corrupted. In this case, break the rule to NEVER use Response.End. The following code solves both problems. Your results may vary.

        '*** transfer package file memory buffer to output stream
        Response.AddHeader("content-disposition", "attachment; filename=" + NewDocFileName)
        Me.Response.ContentType = "application/vnd.ms-word.document.12"
        Response.ContentEncoding = System.Text.Encoding.UTF8
        strDocument.Position = 0
        'See documentation at http://blogs.msdn.com/b/aspnetue/archive/2010/05/25/response-end-response-close-and-how-customer-feedback-helps-us-improve-msdn-documentation.aspx
        HttpContext.Current.ApplicationInstance.CompleteRequest() 'This is the preferred method
        'Response.Close() 'BAD pattern. Do not use this approach, will cause 'cannot download file' in IE10 and other download managers that compare content-Header to actual byte count
        Response.End() 'BAD Pattern as well. However, CompleteRequest does not terminate sending bytes, so Word or other XML based appns will see the file as corrupted. So use this to solve it.

@Cesar: you are using response.Close--> can you try it with IE 10? bet it doesn't work (byte counts don't match)

answered Sep 24, 2015 by sanjaypal1983
0 votes

For what it's worth, I also ran into the same problem listed here. For me the issue was actually with the upload code not the download code:

    Public Sub ImportStream(FileStream As Stream)
        'Use this method with FileUpload.PostedFile.InputStream as a parameter, for example.
        Dim arrBuffer(FileStream.Length) As Byte
        FileStream.Seek(0, SeekOrigin.Begin)
        FileStream.Read(arrBuffer, 0, FileStream.Length)
        Me.FileImage = arrBuffer
    End Sub

In this example the problem is I declare the Byte array arrBuffer with a size one byte too large. This null byte is then saved with the file image to the DB and reproduced on download. The corrected code would be:

        Dim arrBuffer(FileStream.Length - 1) As Byte

Also for reference my HttpResponse code is as follows:

                'SetContentType() is a function which looks up the correct mime type
                'and also adds and informational header about the lookup process...
                context.Response.ContentType = SetContentType(objPostedFile.FileName, context.Response)
                context.Response.AddHeader("content-disposition", "attachment;filename=" & HttpUtility.UrlPathEncode(objPostedFile.FileName))
                'For reference: Public Property FileImage As Byte()
answered Sep 24, 2015 by suyesh.lokhande
0 votes

It all looks ok. My only idea is to try calling Dispose on your stream after calling Response.Flush instead of before, just in case the bytes aren't entirely written before flushing.

answered Sep 24, 2015 by yashwantpinge
0 votes

Take a look a this: Writing MemoryStream to Response Object

I had the same problem and the only solution that worked for me was:

    Response.ContentType = "Application/msword";
    Response.AddHeader("Content-Disposition", "attachment; filename=myfile.docx");
    // myMemoryStream.WriteTo(Response.OutputStream); //works too
answered Sep 24, 2015 by jekbishnoi
0 votes

I had the same problem while i try to open .docx and .xlsx documents. I solve the problem by defining the cacheability to ServerAndPrivate instead of NoCache

there is my method to call document:

public void ProcessRequest(HttpContext context)


       var fi = new FileInfo(context.Request.Path);
        var mediaId = ResolveMediaIdFromName(fi.Name);
        if (mediaId == null) return;

        int mediaContentId;
        if (!int.TryParse(mediaId, out mediaContentId)) return;

        var media = _repository.GetPublicationMediaById(mediaContentId);
        if (media == null) return;

        var fileNameFull = string.Format("{0}{1}", media.Name, media.Extension);
        context.Response.AddHeader("content-disposition", string.Format("attachment;filename={0}", fileNameFull));            
        context.Response.Charset = "";
        context.Response.ContentType = media.ContentType;
answered Sep 24, 2015 by devkumargupta