Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unvendoring of 'tar' breaks unpacking #3390

Closed
davean opened this issue May 2, 2016 · 6 comments
Closed

Unvendoring of 'tar' breaks unpacking #3390

davean opened this issue May 2, 2016 · 6 comments
Assignees

Comments

@davean
Copy link

davean commented May 2, 2016

Commit 0db3b21 introduces a dependancy on a tarbomb check that can't handle many hackage packages.

For example:

$ cabal get binary-0.6.4.0
Downloading binary-0.6.4.0...
Unpacking to binary-0.6.4.0/
File in tar archive is not in the expected directory "binary-0.6.4.0"

The old code handled POSIX.1-2001/pax tarballs specificly, of which there are many on hackage. The code in the tar package does not understand the pax records, and errors on them thinkng they're "normal files" in the wrong location.

These packages did install properly in the last cabal-install version.

@23Skidoo
Copy link
Member

23Skidoo commented May 2, 2016

@dcoutts, can you please look into this?

@23Skidoo
Copy link
Member

23Skidoo commented May 2, 2016

Same issue on the tar bug tracker: haskell/tar#1.

@dcoutts
Copy link
Contributor

dcoutts commented May 3, 2016

Ok.

@23Skidoo 23Skidoo added this to the cabal-install 1.24.0.1 milestone May 3, 2016
@dcoutts
Copy link
Contributor

dcoutts commented May 3, 2016

@23Skidoo @hvr @davean please try with the new release 0.5.0.3 and if it's ok then lets just tweak the cabal-install constraints (no need to re-release).

@dcoutts
Copy link
Contributor

dcoutts commented May 3, 2016

Note that haskell/tar#1 is a different issue really. That's about wholesale support for PAX, this issue was just about ignoring PAX headers for the purposes of the tarbomb check. This check had been added to the tar code in cabal-install but never ported over to the main tar package.

23Skidoo added a commit that referenced this issue May 3, 2016
@23Skidoo
Copy link
Member

23Skidoo commented May 3, 2016

Confirmed that tar-0.5.0.3 fixes the issue. Updated the version bound in the repo and on Hackage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants