]> granicus.if.org Git - git/commitdiff
fast-import: use xsnprintf for formatting headers
authorJeff King <peff@peff.net>
Fri, 24 Mar 2017 17:26:24 +0000 (13:26 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 24 Mar 2017 19:34:07 +0000 (12:34 -0700)
The stream_blob() function checks the return value of
snprintf and dies. This is more simply done with
xsnprintf (and matches the similar call in store_object).

The message the user would get is less specific, but since
the point is that this _shouldn't_ ever happen, that's OK.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
fast-import.c

index 9ae2053a8f822d9222e2c233d4d5ca83a6661972..87c4e7e400229390c1141c69516c42793f254bab 100644 (file)
@@ -1237,9 +1237,7 @@ static void stream_blob(uintmax_t len, unsigned char *sha1out, uintmax_t mark)
        sha1file_checkpoint(pack_file, &checkpoint);
        offset = checkpoint.offset;
 
-       hdrlen = snprintf((char *)out_buf, out_sz, "blob %" PRIuMAX, len) + 1;
-       if (out_sz <= hdrlen)
-               die("impossibly large object header");
+       hdrlen = xsnprintf((char *)out_buf, out_sz, "blob %" PRIuMAX, len) + 1;
 
        git_SHA1_Init(&c);
        git_SHA1_Update(&c, out_buf, hdrlen);