bugfix/dispose-stream-for-corrupted-archive-check

SevenZip.Tests/SevenZip.Tests.csproj

@@ -48,6 +48,9 @@
     <None Include="TestData\multiple_files.7z">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Include="TestData\CorruptedArchive.7z">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Include="TestData\multivolume.part0001.rar">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>

SevenZip.Tests/SevenZipExtractorTests.cs

@@ -21,7 +21,7 @@ public static List<TestFile> TestFiles
 
                 foreach (var file in Directory.GetFiles(Path.Combine(TestContext.CurrentContext.TestDirectory, "TestData")))
                 {
-                    if (file.Contains("multi") || file.Contains("long_path"))
+                    if (file.Contains("multi") || file.Contains("long_path") || file.Contains("CorruptedArchive.7z"))
                     {
                         continue;
                     }
@@ -32,6 +32,20 @@ public static List<TestFile> TestFiles
                 return result;
             }
         }
+
+        [Test]
+        public void CheckCorruptedFile_DisposeStream()
+        {
+            using (var extractor = new SevenZipExtractor(@"TestData\CorruptedArchive.7z"))
+            {
+                if (!extractor.Check())
+                {
+
+                }
+            }
+
+            File.OpenWrite(@"TestData\CorruptedArchive.7z");
+        }
 
         [Test]
         public void ExtractFilesTest()

SevenZip/SevenZipExtractor.cs

@@ -985,6 +985,11 @@ public bool Check()
             }
             catch (Exception)
             {
+                if (_archiveStream is DisposeVariableWrapper disposeVariableWrapper)
+                {
+                    disposeVariableWrapper.DisposeStream = true;
+	        }
+
                 return false;
             }
             finally