diff options
| author | Junio C Hamano <junkio@cox.net> | 2007-04-28 00:15:48 -0700 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2007-04-29 01:33:13 -0700 |
| commit | 4e6380e5c3af2d8cea8ffac2a7759b4f4e62089b (patch) | |
| tree | fb09c2cc416d118c59e0f21668391e56d0c8e267 /builtin-fetch--tool.c | |
| parent | 5b5fe9a526e3841ff6f61dda743b9624dc8f6426 (diff) | |
| download | git-4e6380e5c3af2d8cea8ffac2a7759b4f4e62089b.tar.gz git-4e6380e5c3af2d8cea8ffac2a7759b4f4e62089b.tar.xz | |
Do not barf on too long action description
Reflog message is primarily about easier identification, and
leaving truncated entry is much better than dying.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-fetch--tool.c')
| -rw-r--r-- | builtin-fetch--tool.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/builtin-fetch--tool.c b/builtin-fetch--tool.c index e9d676455..2ad45dcd4 100644 --- a/builtin-fetch--tool.c +++ b/builtin-fetch--tool.c @@ -35,16 +35,13 @@ static int update_ref(const char *action, unsigned char *sha1, unsigned char *oldval) { - int len; char msg[1024]; char *rla = getenv("GIT_REFLOG_ACTION"); static struct ref_lock *lock; if (!rla) rla = "(reflog update)"; - len = snprintf(msg, sizeof(msg), "%s: %s", rla, action); - if (sizeof(msg) <= len) - die("insanely long action"); + snprintf(msg, sizeof(msg), "%s: %s", rla, action); lock = lock_any_ref_for_update(refname, oldval); if (!lock) return 1; |