diff options
| author | Miklos Vajna <vmiklos@frugalware.org> | 2008-11-17 21:48:35 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2008-11-18 15:40:02 -0800 |
| commit | 3c59c50d76cd479caf14cab73fdb09b68597d5e5 (patch) | |
| tree | 79749fcfafcf3a65d13e7a79fb314069436fb8e9 | |
| parent | 47577456bfe61c32df73a7cf4a388f007af3b18f (diff) | |
| download | git-3c59c50d76cd479caf14cab73fdb09b68597d5e5.tar.gz git-3c59c50d76cd479caf14cab73fdb09b68597d5e5.tar.xz | |
builtin-branch: use strbuf in delete_branches()
In case the length of branch name is greather then PATH_MAX-7, we write
to unallocated memory otherwise.
Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin-branch.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/builtin-branch.c b/builtin-branch.c index 2b3613fea..b9149b78f 100644 --- a/builtin-branch.c +++ b/builtin-branch.c @@ -97,7 +97,6 @@ static int delete_branches(int argc, const char **argv, int force, int kinds) unsigned char sha1[20]; char *name = NULL; const char *fmt, *remote; - char section[PATH_MAX]; int i; int ret = 0; @@ -165,11 +164,12 @@ static int delete_branches(int argc, const char **argv, int force, int kinds) argv[i]); ret = 1; } else { + struct strbuf buf = STRBUF_INIT; printf("Deleted %sbranch %s.\n", remote, argv[i]); - snprintf(section, sizeof(section), "branch.%s", - argv[i]); - if (git_config_rename_section(section, NULL) < 0) + strbuf_addf(&buf, "branch.%s", argv[i]); + if (git_config_rename_section(buf.buf, NULL) < 0) warning("Update of config-file failed"); + strbuf_release(&buf); } } |