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

fix(tar): permit slashed output dir #666

Merged
merged 1 commit into from
Sep 21, 2021
Merged

Conversation

piksel
Copy link
Member

@piksel piksel commented Sep 19, 2021

Fixes an issue where TarArchive.ExtractContents would cause an InvalidNameException when the destination dir ended with /.

Adds additional tests for combinations of tar file names and extraction destination paths.

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the SharpZipLib open source product (the "Contribution"). My Contribution is licensed under the MIT License.

@piksel piksel added the tar Related to TAR file format label Sep 19, 2021
@codecov
Copy link

codecov bot commented Sep 19, 2021

Codecov Report

Merging #666 (4db5a0c) into master (1b1ab01) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #666   +/-   ##
=======================================
  Coverage   73.31%   73.31%           
=======================================
  Files          68       68           
  Lines        8305     8305           
=======================================
  Hits         6089     6089           
  Misses       2216     2216           
Impacted Files Coverage Δ
src/ICSharpCode.SharpZipLib/Tar/TarArchive.cs 44.07% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b1ab01...4db5a0c. Read the comment docs.

@piksel piksel force-pushed the fix/tararchive-slashed-outpath branch from b089197 to 9c3fbfa Compare September 19, 2021 16:28
fixes an issue where TarArchive.ExtractContents would cause
an InvalidNameException when the destination dir ended with /

adds additional tests for combinations of tar file names
and extraction destination paths
@piksel piksel force-pushed the fix/tararchive-slashed-outpath branch from 9c3fbfa to 4db5a0c Compare September 19, 2021 17:23
@piksel piksel merged commit b5b1a92 into master Sep 21, 2021
@piksel piksel deleted the fix/tararchive-slashed-outpath branch September 21, 2021 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tar Related to TAR file format
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant