diff options
author | Junio C Hamano <gitster@pobox.com> | 2010-01-27 14:56:38 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2010-01-27 14:56:38 -0800 |
commit | a0075d9e6ae211e8bde3eb40c8cdebb1772ee680 (patch) | |
tree | b29228a2bb894edd907b319b23c77025b5f9e407 /sha1_file.c | |
parent | f1694b62bb3a3e1766b92d64a38f61524cc730a0 (diff) | |
parent | 4ab07e4d1076a1b94b91d58913daeb20eb1c0e2d (diff) | |
download | git-a0075d9e6ae211e8bde3eb40c8cdebb1772ee680.tar.gz |
Merge branch 'il/maint-xmallocz'
* il/maint-xmallocz:
Fix integer overflow in unpack_compressed_entry()
Fix integer overflow in unpack_sha1_rest()
Fix integer overflow in patch_delta()
Add xmallocz()
Diffstat (limited to 'sha1_file.c')
-rw-r--r-- | sha1_file.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sha1_file.c b/sha1_file.c index 12478a3652..657825e14e 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -1166,7 +1166,7 @@ static int unpack_sha1_header(z_stream *stream, unsigned char *map, unsigned lon static void *unpack_sha1_rest(z_stream *stream, void *buffer, unsigned long size, const unsigned char *sha1) { int bytes = strlen(buffer) + 1; - unsigned char *buf = xmalloc(1+size); + unsigned char *buf = xmallocz(size); unsigned long n; int status = Z_OK; @@ -1194,7 +1194,6 @@ static void *unpack_sha1_rest(z_stream *stream, void *buffer, unsigned long size while (status == Z_OK) status = git_inflate(stream, Z_FINISH); } - buf[size] = 0; if (status == Z_STREAM_END && !stream->avail_in) { git_inflate_end(stream); return buf; @@ -1517,8 +1516,7 @@ static void *unpack_compressed_entry(struct packed_git *p, z_stream stream; unsigned char *buffer, *in; - buffer = xmalloc(size + 1); - buffer[size] = 0; + buffer = xmallocz(size); memset(&stream, 0, sizeof(stream)); stream.next_out = buffer; stream.avail_out = size + 1; |