diff options
author | Junio C Hamano <junkio@cox.net> | 2006-08-08 23:21:33 +0400 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-08-08 23:47:55 +0400 |
commit | 329a304714255f6e5528b4311d29f9d83c85d201 (patch) | |
tree | 22c27e9f7ada2b9bcd80a3d53b80ad152c5a1ea0 | |
parent | aa5481c1af08edbca821a0d025d26e9bc41a5c49 (diff) |
builtin-mv: fix use of uninitialized memory.
Juergen Ruehle noticed that add_slash() tries to strcat()
into uninitialized memory and fails.
Signed-off-by: Junio C Hamano <junkio@cox.net>
-rw-r--r-- | builtin-mv.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/builtin-mv.c b/builtin-mv.c index e47942c135..ce8187c1e9 100644 --- a/builtin-mv.c +++ b/builtin-mv.c @@ -48,7 +48,8 @@ static const char *add_slash(const char *path) if (path[len - 1] != '/') { char *with_slash = xmalloc(len + 2); memcpy(with_slash, path, len); - strcat(with_slash + len, "/"); + with_slash[len++] = '/'; + with_slash[len] = 0; return with_slash; } return path; |