'xmalloc' followed immediately by 'memset' is replaced with 'xcalloc', and a simple grep in this project seems to show that it's the only place. Signed-off-by: Guanqun Lu <guanqun.lu@xxxxxxxxx> --- sha1_file.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/sha1_file.c b/sha1_file.c index 8868b80..93e5fc0 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -858,8 +858,7 @@ unsigned char* use_pack(struct packed_git *p, static struct packed_git *alloc_packed_git(int extra) { - struct packed_git *p = xmalloc(sizeof(*p) + extra); - memset(p, 0, sizeof(*p)); + struct packed_git *p = xcalloc(1, sizeof(*p) + extra); p->pack_fd = -1; return p; } -- 1.6.1.2.392.gb04d1 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html