diff options
| author | Jeff King <peff@peff.net> | 2016-02-08 17:21:55 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-02-08 14:42:32 -0800 |
| commit | 7b11a18a2ee04380c1c698635f1ef2c4eb3324fb (patch) | |
| tree | 01dec3bb08d3a5117c1a8fdae17642f9690eb403 | |
| parent | e01c6b15c97e30baedc45021e6dcbd90140616cd (diff) | |
| download | git-7b11a18a2ee04380c1c698635f1ef2c4eb3324fb.tar.gz git-7b11a18a2ee04380c1c698635f1ef2c4eb3324fb.tar.xz | |
test-path-utils: use xsnprintf in favor of strcpy
This strcpy will never overflow because it's copying from
baked-in test data. But we would prefer to avoid strcpy
entirely, as it makes it harder to audit for real security
bugs.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | test-path-utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test-path-utils.c b/test-path-utils.c index c3adcd87b..6232dfe66 100644 --- a/test-path-utils.c +++ b/test-path-utils.c @@ -56,7 +56,7 @@ static int test_function(struct test_data *data, char *(*func)(char *input), if (!data[i].from) to = func(NULL); else { - strcpy(buffer, data[i].from); + xsnprintf(buffer, sizeof(buffer), "%s", data[i].from); to = func(buffer); } if (!strcmp(to, data[i].to)) |