diff options
| author |  Jeff King <peff@peff.net> | 2015-09-24 17:06:55 -0400 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2015-09-25 10:18:18 -0700 |
| commit | 48bcc1c3cc09db1a6da0ce47460fae6e5f7edd4b (patch) | |
| tree | 67e3469fd97096b0f49c91600abbedea8208931a /diff.c | |
| parent | entry.c: convert strcpy to xsnprintf (diff) | |
| download | tgif-48bcc1c3cc09db1a6da0ce47460fae6e5f7edd4b.tar.xz | |
add_packed_git: convert strcpy into xsnprintf
We have the path "foo.idx", and we create a buffer big
enough to hold "foo.pack" and "foo.keep", and then strcpy
straight into it. This isn't a bug (we have enough space),
but it's very hard to tell from the strcpy that this is so.
Let's instead use strip_suffix to take off the ".idx",
record the size of our allocation, and use xsnprintf to make
sure we don't violate our assumptions.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'diff.c')
0 files changed, 0 insertions, 0 deletions