diff options
author | Junio C Hamano <gitster@pobox.com> | 2012-07-11 23:45:07 +0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2012-07-11 23:45:07 +0400 |
commit | cf04a660bb471dae76772f63f071768afb6d347c (patch) | |
tree | 8fbba8586712569ff630638725145bb09950e0f3 | |
parent | e49bf52a1a6654f295718be61fc7a86cbee5eadf (diff) | |
parent | a5a46eb90f530c731084a57783ca9fdbf1a52163 (diff) |
Merge branch 'jc/ustar-checksum-is-unsigned' into maint
"git archive" incorrectly computed the header checksum; the symptom
was observed only when using pathnames with hi-bit set.
* jc/ustar-checksum-is-unsigned:
archive: ustar header checksum is computed unsigned
-rw-r--r-- | archive-tar.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/archive-tar.c b/archive-tar.c index dc91c6b50d..0ba3f25cf5 100644 --- a/archive-tar.c +++ b/archive-tar.c @@ -139,13 +139,13 @@ static void strbuf_append_ext_header(struct strbuf *sb, const char *keyword, static unsigned int ustar_header_chksum(const struct ustar_header *header) { - const char *p = (const char *)header; + const unsigned char *p = (const unsigned char *)header; unsigned int chksum = 0; - while (p < header->chksum) + while (p < (const unsigned char *)header->chksum) chksum += *p++; chksum += sizeof(header->chksum) * ' '; p += sizeof(header->chksum); - while (p < (const char *)header + sizeof(struct ustar_header)) + while (p < (const unsigned char *)header + sizeof(struct ustar_header)) chksum += *p++; return chksum; } |