Skip to content

Fix prefix writing on TarHeaderWrite #75350

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.

Sign up for GitHub

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

Merged
jozkee merged 5 commits into from
Sep 9, 2022
Merged

Fix prefix writing on TarHeaderWrite #75350

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
52c472f
Fix buffer too small error on prefix
jozkee Sep 9, 2022
862b3c4
Add test
jozkee Sep 9, 2022
95512e3
Remove var usage
jozkee Sep 9, 2022
e8983ba
Remove assert
jozkee Sep 9, 2022
de0c269
Update src/libraries/System.Formats.Tar/tests/TarWriter/TarWriter.Wri…
jozkee Sep 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove assert
  • Loading branch information
@jozkee
jozkee committed Sep 9, 2022
commit e8983baeafb7179fd40680de39b3e4261e8eab39
1 change: 0 additions & 1 deletion src/libraries/System.Formats.Tar/src/System/Formats/Tar/TarHeader.Write.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,6 @@ private int WritePosixName(Span<byte> buffer)
if (_name.Length > FieldLengths.Name)
{
int prefixBytesLength = Math.Min(_name.Length - FieldLengths.Name, FieldLengths.Prefix);
Debug.Assert(prefixBytesLength <= 256);
Span<byte> remaining = stackalloc byte[prefixBytesLength];
int encoded = Encoding.ASCII.GetBytes(_name.AsSpan(FieldLengths.Name, prefixBytesLength), remaining);
Debug.Assert(encoded == remaining.Length);
Expand Down