[PR #785] [MERGED] Handle tar files generated with tar -H oldgnu that has large uid/gid values #1226

Closed
opened 2026-01-29 22:19:34 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/adamhathcock/sharpcompress/pull/785
Author: @Erior
Created: 12/4/2023
Status: Merged
Merged: 12/11/2023
Merged by: @adamhathcock

Base: masterHead: feature/Issue-782


📝 Commits (1)

  • 18c7f58 Handle tar files generated with tar -H oldgnu that has large uid/gid values

📊 Changes

3 files changed (+24 additions, -3 deletions)

View changed files

📝 src/SharpCompress/Common/Tar/Headers/TarHeader.cs (+21 -3)
📝 tests/SharpCompress.Test/Tar/TarReaderTests.cs (+3 -0)
tests/TestArchives/Archives/Tar.oldgnu.tar.gz (+0 -0)

📄 Description

The tar file in #782 seems to be generated in oldgnu format, and sharpcompress failed to extract gid in the provided test, I have generated a general test with large value for both uid and gid using the normal setup of files.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/adamhathcock/sharpcompress/pull/785 **Author:** [@Erior](https://github.com/Erior) **Created:** 12/4/2023 **Status:** ✅ Merged **Merged:** 12/11/2023 **Merged by:** [@adamhathcock](https://github.com/adamhathcock) **Base:** `master` ← **Head:** `feature/Issue-782` --- ### 📝 Commits (1) - [`18c7f58`](https://github.com/adamhathcock/sharpcompress/commit/18c7f580939ca745d9a26165f6d6f8cd9aa6ec44) Handle tar files generated with tar -H oldgnu that has large uid/gid values ### 📊 Changes **3 files changed** (+24 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/SharpCompress/Common/Tar/Headers/TarHeader.cs` (+21 -3) 📝 `tests/SharpCompress.Test/Tar/TarReaderTests.cs` (+3 -0) ➕ `tests/TestArchives/Archives/Tar.oldgnu.tar.gz` (+0 -0) </details> ### 📄 Description The tar file in #782 seems to be generated in oldgnu format, and sharpcompress failed to extract gid in the provided test, I have generated a general test with large value for both uid and gid using the normal setup of files. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 22:19:34 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/sharpcompress#1226