adamhathcock / sharpcompress

SharpCompress is a fully managed C# library to deal with many compression types and formats.
MIT License
2.26k stars 479 forks source link

SharpCompress hangs when tar contains symbolic link and is not possible to create it #464

Open justalemon opened 5 years ago

justalemon commented 5 years ago

I was currently working on one of my programs when I tried to extract a .tar.xz (you can get it from here) and the WriteEntryToDirectory call ended up hanging on a specific file:

image

Just to make sure that I didn't made a mistake on my code, I tried to extract the file with WinRAR and it warned me that it was not possible to create a symbolic link:

image

Unable to create the symbolic link C:\Users\lemon\AppData\Local\Temp\Rar$DRa13524.1044\alpine\usr\share\terminfo\l\linux Maybe you need to execute WinRAR with administrator permissions.

Maybe SharpCompress is not handling symbolic links errors correctly?

justalemon commented 5 years ago

Looks like this is related to either Mono or WSL/Ubuntu. On Windows you get:

SharpCompress.Common.ExtractionException: 'Entry is a symbolic link but ExtractionOptions.WriteSymbolicLink delegate is null'

adamhathcock commented 5 years ago

Anymore detail? Maybe this is a simple bug? I use a MacOS so I haven't tried to reproduce this.

RealOrko commented 4 years ago

Just found the same thing using v0.25.0 on Fedora 31.

SharpCompress.Common.ExtractionException: Entry is a symbolic link but ExtractionOptions.WriteSymbolicLink delegate is null
   at SharpCompress.Common.ExtractionMethods.WriteEntryToFile(IEntry entry, String destinationFileName, ExtractionOptions options, Action`2 openAndWrite)

By settings up the options as follows it stopped throwing an exception:

var opt = new ExtractionOptions {
   ExtractFullPath = true,
   Overwrite = true,
   WriteSymbolicLink = (source, target) => {} 
};

After doing a little investigation it would seem there is no clear way of dealing with this in the DOTNET Runtime until this issue is solved.

adamhathcock commented 4 years ago

Added default delegate in https://www.nuget.org/packages/sharpcompress/0.25.1

RealOrko commented 4 years ago

@justalemon can you provide a specimen demonstrating the problem? I am currently using this library to download tars from all over github and have found it works across fedora 31/32 && ubuntu(bionic).

RealOrko commented 4 years ago

@adamhathcock is there any appetite to add an implementation that delegates to the OS in the absence of https://github.com/dotnet/runtime/issues/24271?

justalemon commented 4 years ago

@RealOrko I got the files from https://runtime.fivem.net/artifacts/fivem/build_proot_linux/master/ they contain entire Alpine distros using symbolic links.

RealOrko commented 4 years ago

@justalemon I downloaded and unzipped image 2479, 2478 and 2477 without any issues on fedora 32.

I extracted the archive twice:

I then did a search for the symlink from your screen shot ./alpine/usr/share/terminfo/... but there wasn't any to be found in either of the extracted archives. Are you doing anything special to try and create the symlinks in your code? Once thing to try might be running your unzipping code through a console app without a GUI just to be sure that there is nothing in the UI that is blocking the unzipping process when it reports status.