| commit | 75faa45ae0230b321bf72027b2274315d7e14e34 | [log] [tgz] |
|---|---|---|
| author | Jeff King <peff@peff.net> | Thu Sep 24 17:07:03 2015 -0400 |
| committer | Junio C Hamano <gitster@pobox.com> | Fri Sep 25 10:18:18 2015 -0700 |
| tree | 3b4aa1b362078ba4db498a087f3330ffe7affbd8 | |
| parent | b7115a350b5c01ce0ae7a8735e4235d4b2367b5f [diff] |
replace trivial malloc + sprintf / strcpy calls with xstrfmt It's a common pattern to do: foo = xmalloc(strlen(one) + strlen(two) + 1 + 1); sprintf(foo, "%s %s", one, two); (or possibly some variant with strcpy()s or a more complicated length computation). We can switch these to use xstrfmt, which is shorter, involves less error-prone manual computation, and removes many sprintf and strcpy calls which make it harder to audit the code for real buffer overflows. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>