aboutsummaryrefslogtreecommitdiff
diff refs
from: back
to: back
| flip
diff options
context:
space:
mode:
Diffstat
-rw-r--r--cgit.c18
-rw-r--r--ui-blob.c8
-rw-r--r--ui-commit.c4
-rw-r--r--ui-diff.c8
-rw-r--r--ui-patch.c4
-rw-r--r--ui-shared.c15
-rw-r--r--ui-shared.h5
-rw-r--r--ui-snapshot.c16
-rw-r--r--ui-stats.c5
-rw-r--r--ui-tag.c6
-rw-r--r--ui-tree.c13
11 files changed, 45 insertions, 57 deletions
diff --git a/cgit.c b/cgit.c
index 4e51283..6f75db1 100644
--- a/cgit.c
+++ b/cgit.c
@@ -459,6 +459,7 @@ static char *guess_defbranch(void)
static int prepare_repo_cmd(struct cgit_context *ctx)
{
+ char *tmp;
unsigned char sha1[20];
int nongit = 0;
int rc;
@@ -466,16 +467,17 @@ static int prepare_repo_cmd(struct cgit_context *ctx)
setenv("GIT_DIR", ctx->repo->path, 1);
setup_git_directory_gently(&nongit);
if (nongit) {
- const char *name = ctx->repo->name;
rc = errno;
ctx->page.title = fmt("%s - %s", ctx->cfg.root_title,
"config error");
+ tmp = fmt("Failed to open %s: %s",
+ ctx->repo->name,
+ rc ? strerror(rc) : "Not a valid git repository");
ctx->repo = NULL;
cgit_print_http_headers(ctx);
cgit_print_docstart(ctx);
cgit_print_pageheader(ctx);
- cgit_print_error("Failed to open %s: %s", name,
- rc ? strerror(rc) : "Not a valid git repository");
+ cgit_print_error(tmp);
cgit_print_docend();
return 1;
}
@@ -499,14 +501,14 @@ static int prepare_repo_cmd(struct cgit_context *ctx)
}
if (get_sha1(ctx->qry.head, sha1)) {
- char *tmp = xstrdup(ctx->qry.head);
+ tmp = xstrdup(ctx->qry.head);
ctx->qry.head = ctx->repo->defbranch;
ctx->page.status = 404;
ctx->page.statusmsg = "Not found";
cgit_print_http_headers(ctx);
cgit_print_docstart(ctx);
cgit_print_pageheader(ctx);
- cgit_print_error("Invalid branch: %s", tmp);
+ cgit_print_error(fmt("Invalid branch: %s", tmp));
cgit_print_docend();
return 1;
}
@@ -548,7 +550,7 @@ static void process_request(void *cbdata)
cgit_print_http_headers(ctx);
cgit_print_docstart(ctx);
cgit_print_pageheader(ctx);
- cgit_print_error("No repository selected");
+ cgit_print_error(fmt("No repository selected"));
cgit_print_docend();
return;
}
@@ -860,7 +862,7 @@ int main(int argc, const char **argv)
err = cache_process(ctx.cfg.cache_size, ctx.cfg.cache_root,
ctx.qry.raw, ttl, process_request, &ctx);
if (err)
- cgit_print_error("Error processing page: %s (%d)",
- strerror(err), err);
+ cgit_print_error(fmt("Error processing page: %s (%d)",
+ strerror(err), err));
return err;
}
diff --git a/ui-blob.c b/ui-blob.c
index 8f6989f..1da43c8 100644
--- a/ui-blob.c
+++ b/ui-blob.c
@@ -94,12 +94,12 @@ void cgit_print_blob(const char *hex, char *path, const char *head)
if (hex) {
if (get_sha1_hex(hex, sha1)) {
- cgit_print_error("Bad hex value: %s", hex);
+ cgit_print_error(fmt("Bad hex value: %s", hex));
return;
}
} else {
if (get_sha1(head, sha1)) {
- cgit_print_error("Bad ref: %s", head);
+ cgit_print_error(fmt("Bad ref: %s", head));
return;
}
}
@@ -113,13 +113,13 @@ void cgit_print_blob(const char *hex, char *path, const char *head)
}
if (type == OBJ_BAD) {
- cgit_print_error("Bad object name: %s", hex);
+ cgit_print_error(fmt("Bad object name: %s", hex));
return;
}
buf = read_sha1_file(sha1, &type, &size);
if (!buf) {
- cgit_print_error("Error reading object %s", hex);
+ cgit_print_error(fmt("Error reading object %s", hex));
return;
}
diff --git a/ui-commit.c b/ui-commit.c
index 6b41017..8310ce6 100644
--- a/ui-commit.c
+++ b/ui-commit.c
@@ -27,12 +27,12 @@ void cgit_print_commit(char *hex, const char *prefix)
hex = ctx.qry.head;
if (get_sha1(hex, sha1)) {
- cgit_print_error("Bad object id: %s", hex);
+ cgit_print_error(fmt("Bad object id: %s", hex));
return;
}
commit = lookup_commit_reference(sha1);
if (!commit) {
- cgit_print_error("Bad commit reference: %s", hex);
+ cgit_print_error(fmt("Bad commit reference: %s", hex));
return;
}
info = cgit_parse_commit(commit);
diff --git a/ui-diff.c b/ui-diff.c
index 8b38209..1115518 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -369,12 +369,12 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
get_sha1(new_rev, new_rev_sha1);
type = sha1_object_info(new_rev_sha1, &size);
if (type == OBJ_BAD) {
- cgit_print_error("Bad object name: %s", new_rev);
+ cgit_print_error(fmt("Bad object name: %s", new_rev));
return;
}
commit = lookup_commit_reference(new_rev_sha1);
if (!commit || parse_commit(commit)) {
- cgit_print_error("Bad commit: %s", sha1_to_hex(new_rev_sha1));
+ cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(new_rev_sha1)));
return;
}
@@ -388,12 +388,12 @@ void cgit_print_diff(const char *new_rev, const char *old_rev,
if (!is_null_sha1(old_rev_sha1)) {
type = sha1_object_info(old_rev_sha1, &size);
if (type == OBJ_BAD) {
- cgit_print_error("Bad object name: %s", sha1_to_hex(old_rev_sha1));
+ cgit_print_error(fmt("Bad object name: %s", sha1_to_hex(old_rev_sha1)));
return;
}
commit2 = lookup_commit_reference(old_rev_sha1);
if (!commit2 || parse_commit(commit2)) {
- cgit_print_error("Bad commit: %s", sha1_to_hex(old_rev_sha1));
+ cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1)));
return;
}
}
diff --git a/ui-patch.c b/ui-patch.c
index fbb92cc..66def3c 100644
--- a/ui-patch.c
+++ b/ui-patch.c
@@ -94,12 +94,12 @@ void cgit_print_patch(char *hex, const char *prefix)
hex = ctx.qry.head;
if (get_sha1(hex, sha1)) {
- cgit_print_error("Bad object id: %s", hex);
+ cgit_print_error(fmt("Bad object id: %s", hex));
return;
}
commit = lookup_commit_reference(sha1);
if (!commit) {
- cgit_print_error("Bad commit reference: %s", hex);
+ cgit_print_error(fmt("Bad commit reference: %s", hex));
return;
}
info = cgit_parse_commit(commit);
diff --git a/ui-shared.c b/ui-shared.c
index b93b77a..c1f3c20 100644
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -28,21 +28,10 @@ static char *http_date(time_t t)
tm->tm_hour, tm->tm_min, tm->tm_sec);
}
-void cgit_print_error(const char *fmt, ...)
+void cgit_print_error(const char *msg)
{
- va_list ap;
- va_start(ap, fmt);
- cgit_vprint_error(fmt, ap);
- va_end(ap);
-}
-
-void cgit_vprint_error(const char *fmt, va_list ap)
-{
- va_list cp;
html("<div class='error'>");
- va_copy(cp, ap);
- html_vtxtf(fmt, cp);
- va_end(cp);
+ html_txt(msg);
html("</div>\n");
}
diff --git a/ui-shared.h b/ui-shared.h
index 5987e77..5f8b629 100644
--- a/ui-shared.h
+++ b/ui-shared.h
@@ -52,10 +52,7 @@ extern void cgit_object_link(struct object *obj);
extern void cgit_submodule_link(const char *class, char *path,
const char *rev);
-__attribute__((format (printf,1,2)))
-extern void cgit_print_error(const char *fmt, ...);
-__attribute__((format (printf,1,0)))
-extern void cgit_vprint_error(const char *fmt, va_list ap);
+extern void cgit_print_error(const char *msg);
extern void cgit_print_date(time_t secs, const char *format, int local_time);
extern void cgit_print_age(time_t t, time_t max_relative, const char *format);
extern void cgit_print_http_headers(struct cgit_context *ctx);
diff --git a/ui-snapshot.c b/ui-snapshot.c
index a47884e..9be5dbe 100644
--- a/ui-snapshot.c
+++ b/ui-snapshot.c
@@ -100,11 +100,11 @@ static int make_snapshot(const struct cgit_snapshot_format *format,
unsigned char sha1[20];
if (get_sha1(hex, sha1)) {
- cgit_print_error("Bad object id: %s", hex);
+ cgit_print_error(fmt("Bad object id: %s", hex));
return 1;
}
if (!lookup_commit_reference(sha1)) {
- cgit_print_error("Not a commit reference: %s", hex);
+ cgit_print_error(fmt("Not a commit reference: %s", hex));
return 1;
}
ctx.page.mimetype = xstrdup(format->mimetype);
@@ -154,18 +154,13 @@ static const char *get_ref_from_filename(const char *url, const char *filename,
return NULL;
}
-__attribute__((format (printf, 1, 2)))
-static void show_error(char *fmt, ...)
+static void show_error(char *msg)
{
- va_list ap;
-
ctx.page.mimetype = "text/html";
cgit_print_http_headers(&ctx);
cgit_print_docstart(&ctx);
cgit_print_pageheader(&ctx);
- va_start(ap, fmt);
- cgit_vprint_error(fmt, ap);
- va_end(ap);
+ cgit_print_error(msg);
cgit_print_docend();
}
@@ -182,7 +177,8 @@ void cgit_print_snapshot(const char *head, const char *hex,
f = get_format(filename);
if (!f) {
- show_error("Unsupported snapshot format: %s", filename);
+ show_error(xstrdup(fmt("Unsupported snapshot format: %s",
+ filename)));
return;
}
diff --git a/ui-stats.c b/ui-stats.c
index 28b794f..52e9b91 100644
--- a/ui-stats.c
+++ b/ui-stats.c
@@ -374,11 +374,12 @@ void cgit_show_stats(struct cgit_context *ctx)
i = cgit_find_stats_period(code, &period);
if (!i) {
- cgit_print_error("Unknown statistics type: %c", code[0]);
+ cgit_print_error(fmt("Unknown statistics type: %c", code[0]));
return;
}
if (i > ctx->repo->max_stats) {
- cgit_print_error("Statistics type disabled: %s", period->name);
+ cgit_print_error(fmt("Statistics type disabled: %s",
+ period->name));
return;
}
authors = collect_stats(ctx, period);
diff --git a/ui-tag.c b/ui-tag.c
index 397e15b..5a22696 100644
--- a/ui-tag.c
+++ b/ui-tag.c
@@ -50,18 +50,18 @@ void cgit_print_tag(char *revname)
revname = ctx.qry.head;
if (get_sha1(fmt("refs/tags/%s", revname), sha1)) {
- cgit_print_error("Bad tag reference: %s", revname);
+ cgit_print_error(fmt("Bad tag reference: %s", revname));
return;
}
obj = parse_object(sha1);
if (!obj) {
- cgit_print_error("Bad object id: %s", sha1_to_hex(sha1));
+ cgit_print_error(fmt("Bad object id: %s", sha1_to_hex(sha1)));
return;
}
if (obj->type == OBJ_TAG) {
tag = lookup_tag(sha1);
if (!tag || parse_tag(tag) || !(info = cgit_parse_tag(tag))) {
- cgit_print_error("Bad tag object: %s", revname);
+ cgit_print_error(fmt("Bad tag object: %s", revname));
return;
}
html("<table class='commit-info'>\n");
diff --git a/ui-tree.c b/ui-tree.c
index aebe145..d713553 100644
--- a/ui-tree.c
+++ b/ui-tree.c
@@ -95,13 +95,15 @@ static void print_object(const unsigned char *sha1, char *path, const char *base
type = sha1_object_info(sha1, &size);
if (type == OBJ_BAD) {
- cgit_print_error("Bad object name: %s", sha1_to_hex(sha1));
+ cgit_print_error(fmt("Bad object name: %s",
+ sha1_to_hex(sha1)));
return;
}
buf = read_sha1_file(sha1, &type, &size);
if (!buf) {
- cgit_print_error("Error reading object %s", sha1_to_hex(sha1));
+ cgit_print_error(fmt("Error reading object %s",
+ sha1_to_hex(sha1)));
return;
}
@@ -207,7 +209,8 @@ static void ls_tree(const unsigned char *sha1, char *path, struct walk_tree_cont
tree = parse_tree_indirect(sha1);
if (!tree) {
- cgit_print_error("Not a tree object: %s", sha1_to_hex(sha1));
+ cgit_print_error(fmt("Not a tree object: %s",
+ sha1_to_hex(sha1)));
return;
}
@@ -270,12 +273,12 @@ void cgit_print_tree(const char *rev, char *path)
rev = ctx.qry.head;
if (get_sha1(rev, sha1)) {
- cgit_print_error("Invalid revision name: %s", rev);
+ cgit_print_error(fmt("Invalid revision name: %s", rev));
return;
}
commit = lookup_commit_reference(sha1);
if (!commit || parse_commit(commit)) {
- cgit_print_error("Invalid commit reference: %s", rev);
+ cgit_print_error(fmt("Invalid commit reference: %s", rev));
return;
}
generated by cgitext v1.3.1+17 (git 2.54.0) at 2026-09-09 14:44:21 +0000