starred/sharpcompress
1
0
Fork 0
mirror of https://github.com/adamhathcock/sharpcompress.git synced 2026-02-09 05:24:55 +00:00
Code Issues 975 Packages Projects Releases 20 Wiki Activity

[PR #1037] [WIP] Fix incomplete iteration of TarArchive entries #1465

New Issue
Open
opened 2026-01-29 22:20:42 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 22:20:42 +00:00
Owner
Copy Link

Original Pull Request: https://github.com/adamhathcock/sharpcompress/pull/1037

State: closed
Merged: No

  • Understand the issue: TarArchive.ExtractAllEntries() fails because it's blocked by a check that only allows solid archives or 7Zip
  • Identify the root cause: The check if (!IsSolid && Type != ArchiveType.SevenZip) throws an exception for TAR archives
  • Fix: Modify the check to also allow TAR archives since TarArchive.CreateReaderForSolidExtraction() is implemented and works correctly
  • Add test to verify the fix
  • Run existing tests to ensure no regression
  • Run code review
  • Run CodeQL security check
Original prompt

This section details on the original issue you should resolve

<issue_title>Streaming TarArchive entries results in incomplete iteration</issue_title>
<issue_description>Using the following code on this archive will result in incomplete iteration of the archive, and the exception being thrown

string desktopEntriesArchiveFile = Path.Combine(Path.Combine(Program.BaseDirectoryInternal, "data"), "desktop_entries.tar");

using (var extractor = TarArchive.Open(desktopEntriesArchiveFile))
{
    using (var reader = extractor.ExtractAllEntries())
    {
        var readEntries = 0;
        while (reader.MoveToNextEntry())
        {
            Trace.WriteLine($"Processing entry {readEntries++}: {reader.Entry.Key}");

            if (reader.Entry.IsDirectory)
                continue;

            using (var ms = new MemoryStream())
            {
                reader.WriteEntryTo(ms);
            }
        }

        if (readEntries != extractor.Entries.Count())
        {
            throw new Exception("Tar archive read entries count mismatch.");
        }
    }
}

This however, works fine.

string desktopEntriesArchiveFile = Path.Combine(Path.Combine(Program.BaseDirectoryInternal, "data"), "desktop_entries.tar");

using (var extractor = TarArchive.Open(desktopEntriesArchiveFile))
{
    var readEntries = 0;
    foreach (var entry in extractor.Entries)
    {
        Trace.WriteLine($"Processing entry {readEntries++}: {entry.Key}");

        if (entry.IsDirectory)
            continue;

        using (var ms = new MemoryStream())
        {
            entry.WriteTo(ms);
        }
    }

    if (readEntries != extractor.Entries.Count())
    {
        throw new Exception("Tar archive read entries count mismatch.");
    }
}

The archive was created on Windows using busybox tar

Using SharpCompress 0.41.0 with .NET 4.8

The issue does not occur with SharpCompress 0.40.0

Possibly related: Streamed gzip data fails to decompress due to checksum issue on 0.40, but fixed on latest.</issue_description>

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

**Original Pull Request:** https://github.com/adamhathcock/sharpcompress/pull/1037 **State:** closed **Merged:** No --- - [x] Understand the issue: `TarArchive.ExtractAllEntries()` fails because it's blocked by a check that only allows solid archives or 7Zip - [x] Identify the root cause: The check `if (!IsSolid && Type != ArchiveType.SevenZip)` throws an exception for TAR archives - [x] Fix: Modify the check to also allow TAR archives since `TarArchive.CreateReaderForSolidExtraction()` is implemented and works correctly - [x] Add test to verify the fix - [x] Run existing tests to ensure no regression - [x] Run code review - [x] Run CodeQL security check <!-- START COPILOT CODING AGENT SUFFIX --> <details> <summary>Original prompt</summary> > > ---- > > *This section details on the original issue you should resolve* > > <issue_title>Streaming TarArchive entries results in incomplete iteration</issue_title> > <issue_description>Using the following code on [this archive](https://gist.github.com/DanTheMan827/5a56c5ff239e0ce0b0b228da155f78a6/raw/c709cf692fd49aeda903eaa7f63d90dbfa604ef7/desktop_entries.tar) will result in incomplete iteration of the archive, and the exception being thrown > ```c# > string desktopEntriesArchiveFile = Path.Combine(Path.Combine(Program.BaseDirectoryInternal, "data"), "desktop_entries.tar"); > > using (var extractor = TarArchive.Open(desktopEntriesArchiveFile)) > { > using (var reader = extractor.ExtractAllEntries()) > { > var readEntries = 0; > while (reader.MoveToNextEntry()) > { > Trace.WriteLine($"Processing entry {readEntries++}: {reader.Entry.Key}"); > > if (reader.Entry.IsDirectory) > continue; > > using (var ms = new MemoryStream()) > { > reader.WriteEntryTo(ms); > } > } > > if (readEntries != extractor.Entries.Count()) > { > throw new Exception("Tar archive read entries count mismatch."); > } > } > } > ``` > This however, works fine. > ```c# > string desktopEntriesArchiveFile = Path.Combine(Path.Combine(Program.BaseDirectoryInternal, "data"), "desktop_entries.tar"); > > using (var extractor = TarArchive.Open(desktopEntriesArchiveFile)) > { > var readEntries = 0; > foreach (var entry in extractor.Entries) > { > Trace.WriteLine($"Processing entry {readEntries++}: {entry.Key}"); > > if (entry.IsDirectory) > continue; > > using (var ms = new MemoryStream()) > { > entry.WriteTo(ms); > } > } > > if (readEntries != extractor.Entries.Count()) > { > throw new Exception("Tar archive read entries count mismatch."); > } > } > ``` > > The archive was created on Windows using busybox tar > > Using SharpCompress 0.41.0 with .NET 4.8 > > The issue does not occur with SharpCompress 0.40.0 > > Possibly related: Streamed gzip data fails to decompress due to checksum issue on 0.40, but fixed on latest.</issue_description> > > ## Comments on the Issue (you are @copilot in this section) > > <comments> > </comments> > </details> - Fixes adamhathcock/sharpcompress#1029 <!-- START COPILOT CODING AGENT TIPS --> --- 💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey).
claunia added the pull-request label 2026-01-29 22:20:42 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
duplicate
enhancement
feedback requested
not bug
pull-request
Mirrored from GitHub Pull Request
 question
up for grabs
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/sharpcompress#1465
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?