]> git.ipfire.org Git - thirdparty/git.git/commit - git-compat-util.h
add xsnprintf helper function
authorJeff King <peff@peff.net>
Thu, 24 Sep 2015 21:05:37 +0000 (17:05 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 25 Sep 2015 17:18:18 +0000 (10:18 -0700)
commit7b03c89ebd10396ac7569f0c8c4fa0b4efd4f7ed
treece6c8096a760c64461325daabe6a9dbeb2e75b57
parentfbe85e73ce425b25c15f0b1f3900a9bf895a9793
add xsnprintf helper function

There are a number of places in the code where we call
sprintf(), with the assumption that the output will fit into
the buffer. In many cases this is true (e.g., formatting a
number into a large buffer), but it is hard to tell
immediately from looking at the code. It would be nice if we
had some run-time check to make sure that our assumption is
correct (and to communicate to readers of the code that we
are not blindly calling sprintf, but have actually thought
about this case).

This patch introduces xsnprintf, which behaves just like
snprintf, except that it dies whenever the output is
truncated. This acts as a sort of assert() for these cases,
which can help find places where the assumption is violated
(as opposed to truncating and proceeding, which may just
silently give a wrong answer).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-compat-util.h
wrapper.c