diff options
author | Brandon Casey <drafnel@gmail.com> | 2011-10-06 13:22:22 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2011-10-06 13:54:32 -0700 |
commit | 040a655116c9755bbf30acd22c34eecb2f502c6d (patch) | |
tree | 6e1260d25fa2de20f150159de83cae2251c70824 /show-index.c | |
parent | 97410b27e91593c4096bb16abbace783895a1f62 (diff) | |
download | git-040a655116c9755bbf30acd22c34eecb2f502c6d.tar.gz |
cleanup: use internal memory allocation wrapper functions everywhere
The "x"-prefixed versions of strdup, malloc, etc. will check whether the
allocation was successful and terminate the process otherwise.
A few uses of malloc were left alone since they already implemented a
graceful path of failure or were in a quasi external library like xdiff.
Additionally, the call to malloc in compat/win32/syslog.c was not modified
since the syslog() implemented there is a die handler and a call to the
x-wrappers within a die handler could result in recursion should memory
allocation fail. This will have to be addressed separately.
Signed-off-by: Brandon Casey <drafnel@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'show-index.c')
-rw-r--r-- | show-index.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/show-index.c b/show-index.c index 4c0ac138af..63f9da5323 100644 --- a/show-index.c +++ b/show-index.c @@ -48,7 +48,7 @@ int main(int argc, char **argv) unsigned char sha1[20]; uint32_t crc; uint32_t off; - } *entries = malloc(nr * sizeof(entries[0])); + } *entries = xmalloc(nr * sizeof(entries[0])); for (i = 0; i < nr; i++) if (fread(entries[i].sha1, 20, 1, stdin) != 1) die("unable to read sha1 %u/%u", i, nr); |