Fix ZipExtractor to dispose backing stream and skip failed entries

Co-authored-by: gfs <98900+gfs@users.noreply.github.com>

Author: Copilot

RecursiveExtractor/Extractors/ZipExtractor.cs

@@ -136,9 +136,10 @@ public async IAsyncEnumerable<FileEntry> ExtractAsync(FileEntry fileEntry, Extra
                     catch (Exception e)
                     {
                         Logger.Debug(Extractor.FAILED_PARSING_ERROR_MESSAGE_STRING, ArchiveFileType.ZIP, fileEntry.FullPath, zipEntry.Key, e.GetType());
+                        target?.Dispose();
+                        continue;
                     }
 
-                    target ??= new MemoryStream();
                     var name = zipEntry.Key?.Replace('/', Path.DirectorySeparatorChar) ?? "";
                     var newFileEntry = new FileEntry(name, target, fileEntry, modifyTime: zipEntry.LastModifiedTime, memoryStreamCutoff: options.MemoryStreamCutoff);
 
@@ -222,7 +223,7 @@ public IEnumerable<FileEntry> Extract(FileEntry fileEntry, ExtractorOptions opti
                 {
                     governor.CheckResourceGovernor(zipEntry.Size);
 
-                    using var fs = StreamFactory.GenerateAppropriateBackingStream(options, zipEntry.Size);
+                    var fs = StreamFactory.GenerateAppropriateBackingStream(options, zipEntry.Size);
 
                     try
                     {
@@ -232,6 +233,8 @@ public IEnumerable<FileEntry> Extract(FileEntry fileEntry, ExtractorOptions opti
                     catch (Exception e)
                     {
                         Logger.Debug(Extractor.FAILED_PARSING_ERROR_MESSAGE_STRING, ArchiveFileType.ZIP, fileEntry.FullPath, zipEntry.Key, e.GetType());
+                        fs?.Dispose();
+                        continue;
                     }
 
                     var name = zipEntry.Key?.Replace('/', Path.DirectorySeparatorChar) ?? "";