summaryrefslogtreecommitdiff
path: root/builtin/archive.c
diff options
context:
space:
mode:
authorLibravatar Michael Schubert <mschub@elegosoft.com>2011-09-21 14:42:22 +0200
committerLibravatar Junio C Hamano <gitster@pobox.com>2011-09-22 09:35:07 -0700
commitb9ab810b1826a7757e16f152faf0c28c2fdaa679 (patch)
tree13219d03e5a0e867c75b902a924376d68119be6a /builtin/archive.c
parentGit 1.7.6 (diff)
downloadtgif-b9ab810b1826a7757e16f152faf0c28c2fdaa679.tar.xz
patch-id.c: use strbuf instead of a fixed buffer
get_one_patchid() uses a rather dumb heuristic to determine if the passed buffer is part of the next commit. Whenever the first 40 bytes are a valid hexadecimal sha1 representation, get_one_patchid() returns next_sha1. Once the current line is longer than the fixed buffer, this will break (provided the additional bytes make a valid hexadecimal sha1). As a result patch-id returns incorrect results. Instead, use strbuf and read one line at a time. Helped-by: Jeff King <peff@peff.net> Signed-off-by: Michael Schubert <mschub@elegosoft.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/archive.c')
0 files changed, 0 insertions, 0 deletions