diff options
| author | Jochen Voss <voss@seehuhn.de> | 2008-06-28 16:04:24 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2008-06-28 20:43:21 (GMT) |
| commit | 74d817cf8cf68104564cf6c93c1361f66dad1901 (patch) | |
| tree | f0c09283b2cb90a64ad50d402943cf42807879b7 /builtin-upload-archive.c | |
| parent | 7ac749c96d143ba4f76723959892cbaddbe8ed07 (diff) | |
| download | git-74d817cf8cf68104564cf6c93c1361f66dad1901.zip git-74d817cf8cf68104564cf6c93c1361f66dad1901.tar.gz git-74d817cf8cf68104564cf6c93c1361f66dad1901.tar.bz2 | |
avoid off-by-one error in run_upload_archive
Make sure that buf has enough space to store the trailing \0 of
the command line argument, too.
Signed-off-by: Jochen Voss <voss@seehuhn.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-upload-archive.c')
| -rw-r--r-- | builtin-upload-archive.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-upload-archive.c b/builtin-upload-archive.c index 48ae09e..371400d 100644 --- a/builtin-upload-archive.c +++ b/builtin-upload-archive.c @@ -30,7 +30,7 @@ static int run_upload_archive(int argc, const char **argv, const char *prefix) if (argc != 2) usage(upload_archive_usage); - if (strlen(argv[1]) > sizeof(buf)) + if (strlen(argv[1]) + 1 > sizeof(buf)) die("insanely long repository name"); strcpy(buf, argv[1]); /* enter-repo smudges its argument */ |