I noticed this while working on #117410:
|
int checksum = (int)TarHelpers.ParseOctal<uint>(spanChecksum); |
|
// Zero checksum means the whole header is empty |
|
if (checksum == 0) |
|
{ |
|
return null; |
|
} |
There's no reason to treat zero separate from any other checksum value.
I think this code should be removed.
@ericstj @dotnet/area-system-formats-tar wdyt?
