From 0dc3b035e03a4028a22cd2a8b5f21086e3227047 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Tue, 28 Mar 2017 15:46:53 -0400 Subject: combine-diff: replace malloc/snprintf with xstrfmt There's no need to use the magic "100" when a strbuf can do it for us. Signed-off-by: Jeff King diff --git a/combine-diff.c b/combine-diff.c index 59501db..d356057 100644 --- a/combine-diff.c +++ b/combine-diff.c @@ -292,9 +292,10 @@ static char *grab_blob(const struct object_id *oid, unsigned int mode, enum object_type type; if (S_ISGITLINK(mode)) { - blob = xmalloc(100); - *size = snprintf(blob, 100, - "Subproject commit %s\n", oid_to_hex(oid)); + struct strbuf buf = STRBUF_INIT; + strbuf_addf(&buf, "Subproject commit %s\n", oid_to_hex(oid)); + *size = buf.len; + blob = strbuf_detach(&buf, NULL); } else if (is_null_oid(oid)) { /* deleted blob */ *size = 0; -- cgit v0.10.2-6-g49f6