From 93ba78d6f4632ef1c5228965e3edc8c0faf88c1e Mon Sep 17 00:00:00 2001 From: Vincent Ambo Date: Tue, 26 May 2020 00:06:52 +0100 Subject: revert(3p/git): Revert merge of git upstream at v2.26.2 This causes cgit to serve error pages, which is undesirable. This reverts commit 5229c9b232de5bfa959ad6ebbb4c8192ac513352, reversing changes made to f2b211131f2347342dde63975b09cf603149f1a3. --- third_party/git/usage.c | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) (limited to 'third_party/git/usage.c') diff --git a/third_party/git/usage.c b/third_party/git/usage.c index 58fb5fff5f24..2fdb20086bd6 100644 --- a/third_party/git/usage.c +++ b/third_party/git/usage.c @@ -9,26 +9,14 @@ void vreportf(const char *prefix, const char *err, va_list params) { char msg[4096]; - char *p, *pend = msg + sizeof(msg); - size_t prefix_len = strlen(prefix); + char *p; - if (sizeof(msg) <= prefix_len) { - fprintf(stderr, "BUG!!! too long a prefix '%s'\n", prefix); - abort(); - } - memcpy(msg, prefix, prefix_len); - p = msg + prefix_len; - if (vsnprintf(p, pend - p, err, params) < 0) - *p = '\0'; /* vsnprintf() failed, clip at prefix */ - - for (; p != pend - 1 && *p; p++) { + vsnprintf(msg, sizeof(msg), err, params); + for (p = msg; *p; p++) { if (iscntrl(*p) && *p != '\t' && *p != '\n') *p = '?'; } - - *(p++) = '\n'; /* we no longer need a NUL */ - fflush(stderr); - write_in_full(2, msg, p - msg); + fprintf(stderr, "%s%s\n", prefix, msg); } static NORETURN void usage_builtin(const char *err, va_list params) -- cgit 1.4.1