diff options
| author | Jeff King <peff@peff.net> | 2015-09-24 21:06:42 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2015-09-25 17:18:18 (GMT) |
| commit | ef1286d3c0ba714c6c2ae87e14edf3c462aef114 (patch) | |
| tree | b43b47b015c9a6bc479bb221ddc299996e336183 /builtin/index-pack.c | |
| parent | f2f026752993054c1b712b6f4ae3ff167db38cbe (diff) | |
| download | git-ef1286d3c0ba714c6c2ae87e14edf3c462aef114.zip git-ef1286d3c0ba714c6c2ae87e14edf3c462aef114.tar.gz git-ef1286d3c0ba714c6c2ae87e14edf3c462aef114.tar.bz2 | |
use xsnprintf for generating git object headers
We generally use 32-byte buffers to format git's "type size"
header fields. These should not generally overflow unless
you can produce some truly gigantic objects (and our types
come from our internal array of constant strings). But it is
a good idea to use xsnprintf to make sure this is the case.
Note that we slightly modify the interface to
write_sha1_file_prepare, which nows uses "hdrlen" as an "in"
parameter as well as an "out" (on the way in it stores the
allocated size of the header, and on the way out it returns
the ultimate size of the header).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/index-pack.c')
| -rw-r--r-- | builtin/index-pack.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/index-pack.c b/builtin/index-pack.c index 3431de2..1ad1bde 100644 --- a/builtin/index-pack.c +++ b/builtin/index-pack.c @@ -441,7 +441,7 @@ static void *unpack_entry_data(unsigned long offset, unsigned long size, int hdrlen; if (!is_delta_type(type)) { - hdrlen = sprintf(hdr, "%s %lu", typename(type), size) + 1; + hdrlen = xsnprintf(hdr, sizeof(hdr), "%s %lu", typename(type), size) + 1; git_SHA1_Init(&c); git_SHA1_Update(&c, hdr, hdrlen); } else |