archive: ustar header checksum is computed unsigned
commita5a46eb90f530c731084a57783ca9fdbf1a52163
authorJunio C Hamano <gitster@pobox.com>
Wed, 13 Jun 2012 17:42:25 +0000 (13 10:42 -0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 Jun 2012 17:47:21 +0000 (13 10:47 -0700)
tree13b77bd8f9ad3a31b25b48b1af41993fafc9ae94
parentbefc5ed3799cb6fcbaa7de03e7fa1760e846853c
archive: ustar header checksum is computed unsigned

POSIX.1 (pax) is pretty clear on this:

  The chksum field shall be the ISO/IEC 646:1991 standard IRV
  representation of the octal value of the simple sum of all octets
  in the header logical record. Each octet in the header shall be
  treated as an unsigned value. These values shall be added to an
  unsigned integer, initialized to zero, the precision of which is
  not less than 17 bits. When calculating the checksum, the chksum
  field is treated as if it were all <space> characters.

so is GNU:

  http://www.gnu.org/software/tar/manual/html_node/Checksumming.html

Found by 7zip folks and reported by Rafał Mużyło.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
archive-tar.c