Hi,
\r\n\r\n
When I call ZipInputStream.GetNextEntry(), it's throwing an exception that says:
\r\n+ exception {Ionic.Zip.BadReadException: ZipEntry::ReadHeader(): Bad signature (0x86635B9F)
\r\n\r\n
Now, this only happens when I am on the last entry, so the expected result of GetNextEntry is null, but instead it throws this error.
\r\n\r\n
Any ideas on why this is happening when there should be no more entries?
\r\n\r\n
Thanks
\r\n", "PostedDate": "2011-07-13T07:37:11.993-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "641799", "ThreadId": "264999", "Html": "Alright so in my zipping up function, I am setting the compression levels
\r\n\r\nif(compress)\r\n zipOutStream.CompressionLevel = CompressionLevel.BestSpeed;\r\nelse\r\n zipOutStream.CompressionLevel = CompressionLevel.None;\r\n
\r\n
\r\nBut if I remove this block of code, I dont get the error mentioned above.
\r\nWhy is this??
\r\n
\r\nI dont mind removing this code, but some of the files that are going to be added to the archive are already compressed.
\r\nIs this going to effect the already compressed files?
\r\n
Hi Bowman,
\r\nI have a test case that covers this kind of scenario, but it does not fail the way you are reporting.
\r\nSeems like you have 2 blocks of code: one that produces the zip and another that reads it. What I understand is that by modifying how you produce the zip file, it causes an error while attempting to read the zip file. Is that right? Can you provide a short and sweet code snip that reproduces this behavior ?
\r\nHere's the code I just used, it exhibits no problems:
\r\n\r\n[TestMethod]\r\npublic void ZIS_ZOS_VaryCompression()\r\n{\r\n string testBin = TestUtilities.GetTestBinDir(CurrentDir);\r\n string resourceDir = Path.Combine(testBin, \"Resources\");\r\n var filesToAdd = Directory.GetFiles(resourceDir);\r\n\r\n Func<int, int, bool> chooseCompression = (ix, cycle) => {\r\n var name = Path.GetFileName(filesToAdd[ix]);\r\n switch (cycle)\r\n {\r\n case 0:\r\n return !(name.EndsWith(\".zip\") ||\r\n name.EndsWith(\".docx\") ||\r\n name.EndsWith(\".xslx\"));\r\n case 1:\r\n return ((ix%2)==0);\r\n\r\n default:\r\n return (ix == filesToAdd.Length - 1);\r\n }\r\n };\r\n\r\n // Three cycles - three different ways to vary compression\r\n for (int k=0; k < 3; k++)\r\n {\r\n string zipFileToCreate = String.Format(\"VaryCompression-{0}.zip\", k);\r\n\r\n TestContext.WriteLine(\"\");\r\n TestContext.WriteLine(\"Creating zip, cycle {0}\", k);\r\n using (var fileStream = File.OpenWrite(zipFileToCreate))\r\n {\r\n using (var zos = new ZipOutputStream(fileStream, true))\r\n {\r\n for (int i=0; i < filesToAdd.Length; i++)\r\n {\r\n var file = filesToAdd[i];\r\n var shortName = Path.GetFileName(file);\r\n bool compress = chooseCompression(i, k);\r\n\r\n if (compress)\r\n zos.CompressionLevel = Ionic.Zlib.CompressionLevel.Default;\r\n else\r\n zos.CompressionLevel = Ionic.Zlib.CompressionLevel.None;\r\n\r\n zos.PutNextEntry(shortName);\r\n using (var input = File.OpenRead(file))\r\n {\r\n CopyStream(input, zos);\r\n }\r\n }\r\n }\r\n }\r\n\r\n TestContext.WriteLine(\"\");\r\n TestContext.WriteLine(\"Extracting cycle {0}\", k);\r\n string extractDir = \"extract-\" + k;\r\n Directory.CreateDirectory(extractDir);\r\n using (var raw = File.OpenRead(zipFileToCreate))\r\n {\r\n using (var input = new ZipInputStream(raw))\r\n {\r\n ZipEntry e;\r\n while ((e = input.GetNextEntry()) != null)\r\n {\r\n TestContext.WriteLine(\"entry: {0}\", e.FileName);\r\n string outputPath = Path.Combine(extractDir, e.FileName);\r\n if (e.IsDirectory)\r\n {\r\n // create the directory\r\n Directory.CreateDirectory(outputPath);\r\n }\r\n else\r\n {\r\n // create the file\r\n using (var output = File.Create(outputPath))\r\n {\r\n CopyStream(input,output);\r\n }\r\n }\r\n }\r\n }\r\n }\r\n\r\n string[] filesUnzipped = Directory.GetFiles(extractDir);\r\n Assert.AreEqual<int>(filesToAdd.Length, filesUnzipped.Length,\r\n \"Incorrect number of files extracted.\");\r\n\r\n }\r\n}\r\n\r\n\r\n
Alright Im just going to copy and paste the examples you have for zipping and unzipping using zipinputstream and zipoutputstream and add
\r\nsignificant part of my code in.
\r\nim using Ionic.Zip, not Ionic.Zlib
\r\nOk so here is for Zipping:
\r\n\r\n
private void Zipup(bool needsCompression)\r\n{\r\n if (filesToZip.Count == 0)\r\n {\r\n System.Console.WriteLine(\"Nothing to do.\");\r\n return;\r\n }\r\n\r\n using (var output= new ZipOutputStream(outputFileName))\r\n {\r\n output.Password = \"VerySecret!\";\r\n output.Encryption = EncryptionAlgorithm.WinZipAes256;\r\n\r\n foreach (string inputFileName in filesToZip)\r\n {\r\n System.Console.WriteLine(\"file: {0}\", inputFileName);\r\n\r\n if(needsCompression)\r\n output.CompressionLevel = CompressionLevel.BestSpeed; //<--causing badread exception during unzip (it seems)\r\n else\r\n output.CompressionLevel = CompressionLevel.None;\r\n\r\n //NOTE TO CHEESO:\r\n //CompressionLevel seems to belong to Zlib, not\r\n //not Zip, so it wount let me use CompressionMethod\r\n //instead of CompressionLevel =(\r\n\r\n output.PutNextEntry(inputFileName);\r\n using (var input = File.Open(inputFileName, FileMode.Open, FileAccess.Read,\r\n FileShare.Read | FileShare.Write ))\r\n {\r\n byte[] buffer= new byte[2048];\r\n int n;\r\n while ((n= input.Read(buffer,0,buffer.Length)) > 0)\r\n {\r\n output.Write(buffer,0,n);\r\n }\r\n }\r\n }\r\n }\r\n}\r\n\r\n
And for Unzipping:\r\n
\r\n
//this method fails when the compression levels are explicitly set\r\n
//while zipping, works fine otherwise\r\n
\r\nI hope this helps ", "PostedDate": "2011-07-13T12:44:32.927-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "642052", "ThreadId": "264999", "Html": "private void Unzip(filePath)\r\n{\r\n byte[] buffer= new byte[2048];\r\n int n;\r\n using (var input= new ZipInputStream(inputFileName))\r\n {\r\n ZipEntry e;\r\n while (( e = input.GetNextEntry()) != null)\r\n {\r\n if (e.IsDirectory) continue;\r\n string outputPath = Path.Combine(extractDir, e.FileName);\r\n using (var output = File.Open(outputPath, FileMode.Create, FileAccess.ReadWrite))\r\n {\r\n while ((n= input.Read(buffer, 0, buffer.Length)) > 0)\r\n {\r\n output.Write(buffer,0,n);\r\n }\r\n }\r\n if(Path.GetExtension(fileName).Equals(ZIP_EXTENSION))\r\n {\r\n Unzip(filePath); //Recursion here is zip file within zip file\r\n }\r\n }\r\n }\r\n}\r\n
ok, I just ran the code you gave me - still no error. I had to make a few changes to get it to compile. I tried with v1.9.1.5, and also with v1.9.1.6001 (prelim build for next version).
\r\nI've enclosed the code I used in its entirety, below. If I can't reproduce the problem you're reporting, then I won't be able to fix it. Maybe you can have a look at what I produced and see if you can cajole it into something that produces the problem you see.
\r\nI noticed that in Unzip() you had some uncompilable parts; also in your code the recursion appears to use the original file. I'm not sure because as I said it does not compile, the way you gave it to me. You might want to check that part in your code - confused paths could be the source of your troubles.
\r\n\r\n
// BowmanZis.cs\r\n// ------------------------------------------------------------------\r\n//\r\n// Test something for Bowman\r\n//\r\n// Author : Dino\r\n// Created : Wed Jul 13 20:06:47 2011\r\n// Last Saved : <2011-July-13 20:38:50>\r\n//\r\n// ------------------------------------------------------------------\r\n//\r\n// Copyright (c) 2011 by Dino Chiesa\r\n// All rights reserved!\r\n//\r\n// ------------------------------------------------------------------\r\n//\r\n// compile: csc.exe /t:exe /debug+ /r:Ionic.Zip.dll BowmanZis.cs\r\n//\r\n\r\nusing System;\r\nusing System.IO;\r\nusing System.Collections.Generic;\r\nusing System.Linq;\r\nusing System.Reflection;\r\nusing Ionic.Zlib;\r\nusing Ionic.Zip;\r\n\r\n\r\n\r\nnamespace Ionic.ToolsAndTests\r\n{\r\n public class BowmanZis\r\n {\r\n List<String> filesToZip;\r\n string basefile = \"BowmanZis.zip\";\r\n string baseExtractDir = \"BowmanZis\";\r\n string password = \"VerySecret!\";\r\n\r\n public BowmanZis () {}\r\n public BowmanZis (string[] args) { }\r\n\r\n\r\n private void Zipup(bool needsCompression)\r\n {\r\n Console.WriteLine(\"\\nZip up {0}, {1} compression\",\r\n this.basefile, needsCompression ? \"with\" : \"without\");\r\n\r\n if (filesToZip.Count == 0)\r\n {\r\n Console.WriteLine(\"Nothing to do.\");\r\n return;\r\n }\r\n\r\n using (var output= new ZipOutputStream(this.basefile))\r\n {\r\n output.Password = this.password;\r\n output.Encryption = EncryptionAlgorithm.WinZipAes256;\r\n\r\n foreach (string inputFileName in filesToZip)\r\n {\r\n Console.WriteLine(\" file: {0}\", inputFileName);\r\n\r\n if(needsCompression)\r\n output.CompressionLevel = CompressionLevel.BestSpeed; //<--causing badread exception during unzip (it seems)\r\n else\r\n output.CompressionLevel = CompressionLevel.None;\r\n\r\n //NOTE TO CHEESO:\r\n //CompressionLevel seems to belong to Zlib, not\r\n //not Zip, so it wount let me use CompressionMethod\r\n //instead of CompressionLevel =(\r\n\r\n output.PutNextEntry(inputFileName);\r\n using (var input = File.Open(inputFileName, FileMode.Open, FileAccess.Read,\r\n FileShare.Read | FileShare.Write ))\r\n {\r\n byte[] buffer= new byte[2048];\r\n int n;\r\n while ((n= input.Read(buffer,0,buffer.Length)) > 0)\r\n {\r\n output.Write(buffer,0,n);\r\n }\r\n }\r\n }\r\n }\r\n }\r\n\r\n\r\n private void Unzip()\r\n {\r\n Unzip(this.basefile, this.baseExtractDir);\r\n }\r\n\r\n\r\n private void Unzip(string filePath, string extractDir)\r\n {\r\n Console.WriteLine(\"\\nUnzip {0}\", filePath);\r\n byte[] buffer= new byte[2048];\r\n int n;\r\n using (var input= new ZipInputStream(filePath))\r\n {\r\n input.Password = this.password;\r\n ZipEntry e;\r\n while (( e = input.GetNextEntry()) != null)\r\n {\r\n if (e.IsDirectory) continue;\r\n string outputPath = Path.Combine(extractDir, e.FileName);\r\n Console.WriteLine(\" file: {0}\", outputPath);\r\n\r\n string containingDir = Path.GetDirectoryName(outputPath);\r\n if (!Directory.Exists(containingDir))\r\n Directory.CreateDirectory(containingDir);\r\n\r\n using (var output = File.Create(outputPath))\r\n {\r\n while ((n= input.Read(buffer, 0, buffer.Length)) > 0)\r\n {\r\n output.Write(buffer,0,n);\r\n }\r\n }\r\n if (Path.GetExtension(outputPath).Equals(\".zip\")) // recurse\r\n Unzip(outputPath,\r\n Path.Combine(extractDir,\r\n Path.GetFileNameWithoutExtension(outputPath)));\r\n }\r\n }\r\n }\r\n\r\n\r\n void Reset()\r\n {\r\n if (File.Exists(this.basefile))\r\n {\r\n Console.WriteLine(\"Removing {0}\", this.basefile);\r\n File.Delete(this.basefile);\r\n }\r\n if (Directory.Exists(this.baseExtractDir))\r\n {\r\n Console.WriteLine(\"Removing {0}\", this.baseExtractDir);\r\n Directory.Delete(this.baseExtractDir,true);\r\n }\r\n }\r\n\r\n\r\n public void Run()\r\n {\r\n var fileInfos = (from fi in (from fn in Directory.GetFiles(\".\", \"*.cs\")\r\n select new FileInfo(fn))\r\n orderby fi.Length descending\r\n select fi).Take(12);\r\n\r\n filesToZip = fileInfos.ToList().ConvertAll((fi)=>fi.Name);\r\n\r\n // pass 1: use compression\r\n Reset();\r\n\r\n Zipup(true);\r\n Unzip();\r\n\r\n Console.WriteLine(\"--------------------------------------------\");\r\n\r\n // pass 2: no compression\r\n Reset();\r\n\r\n Zipup(false);\r\n Unzip();\r\n\r\n Reset();\r\n }\r\n\r\n\r\n public static void Usage()\r\n {\r\n Console.WriteLine(\"\\nBowmanZis: test a DotNetZip scenario for bowman.\\n\");\r\n Console.WriteLine(\"Usage:\\n BowmanZis\");\r\n }\r\n\r\n\r\n public static void Main(string[] args)\r\n {\r\n try\r\n {\r\n new BowmanZis(args)\r\n .Run();\r\n }\r\n catch (System.Exception exc1)\r\n {\r\n Console.WriteLine(\"Exception: {0}\", exc1.ToString());\r\n Usage();\r\n }\r\n }\r\n\r\n }\r\n}\r\n\r\n\r\n
", "PostedDate": "2011-07-13T17:44:24.153-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "642380", "ThreadId": "264999", "Html": "
Thanks Cheeso, Ill try and reproduce the error with the above code. Did you try zipping/unzipping a zip file with multiple zip files inside>?
", "PostedDate": "2011-07-14T08:07:52.573-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "642467", "ThreadId": "264999", "Html": "No - i hadn't tried nested zip files.
\r\nThe use of nested zips should have no bearing on the failure you described - ZipEntry::ReadHeader(): Bad signature (0x86635B9F) - unless you have a bug in your code.
\r\nI just tried nested zips now - it works fine for me. I think maybe you have a bug in your code. If you try to wrap a ZipInputStream over a stream that is already open, you would get the error you're seeing. Check your code again, especially the recursion.
\r\n\r\n