diff --git a/SharpCompress.Test/Tar/TarArchiveTests.cs b/SharpCompress.Test/Tar/TarArchiveTests.cs index 3cbd6998..71a11e97 100644 --- a/SharpCompress.Test/Tar/TarArchiveTests.cs +++ b/SharpCompress.Test/Tar/TarArchiveTests.cs @@ -30,13 +30,17 @@ namespace SharpCompress.Test [TestMethod] - public void TarArchivePathReadLongName() + public void Tar_NonUstarArchiveWithLongNameDoesNotSkipEntriesAfterTheLongOne() { string unmodified = Path.Combine(TEST_ARCHIVES_PATH, "very long filename.tar"); using (var archive = TarArchive.Open(unmodified)) { - Assert.AreEqual(2, archive.Entries.Count); - Assert.AreEqual(archive.Entries.Last().Key, @"very long filename/very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename.jpg"); + Assert.AreEqual(5, archive.Entries.Count); + Assert.IsTrue(archive.Entries.Any(entry => entry.Key == "very long filename/")); + Assert.IsTrue(archive.Entries.Any(entry => entry.Key == "very long filename/very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename very long filename.jpg")); + Assert.IsTrue(archive.Entries.Any(entry => entry.Key == "z_file 1.txt")); + Assert.IsTrue(archive.Entries.Any(entry => entry.Key == "z_file 2.txt")); + Assert.IsTrue(archive.Entries.Any(entry => entry.Key == "z_file 3.txt")); } } diff --git a/SharpCompress/Archive/Tar/TarArchive.cs b/SharpCompress/Archive/Tar/TarArchive.cs index e801854b..4258a37e 100644 --- a/SharpCompress/Archive/Tar/TarArchive.cs +++ b/SharpCompress/Archive/Tar/TarArchive.cs @@ -169,11 +169,21 @@ namespace SharpCompress.Archive.Tar { var entry = new TarArchiveEntry(this, new TarFilePart(previousHeader, stream), CompressionType.None); - var memoryStream = new MemoryStream(); - entry.WriteTo(memoryStream); - memoryStream.Position = 0; - var bytes = memoryStream.ToArray(); - header.Name = ArchiveEncoding.Default.GetString(bytes, 0, bytes.Length).TrimNulls(); + + var oldStreamPos = stream.Position; + + using(var entryStream = entry.OpenEntryStream()) + using(var memoryStream = new MemoryStream()) + { + entryStream.TransferTo(memoryStream); + memoryStream.Position = 0; + var bytes = memoryStream.ToArray(); + + header.Name = ArchiveEncoding.Default.GetString(bytes, 0, bytes.Length).TrimNulls(); + } + + stream.Position = oldStreamPos; + previousHeader = null; } yield return new TarArchiveEntry(this, new TarFilePart(header, stream), CompressionType.None); diff --git a/TestArchives/Archives/very long filename.tar b/TestArchives/Archives/very long filename.tar index fb4f0daa..d7abd2f1 100644 Binary files a/TestArchives/Archives/very long filename.tar and b/TestArchives/Archives/very long filename.tar differ