11 files changed, 57 insertions, 45 deletions

diff --git a/cgit.c b/cgit.c
index 6f75db1..4e51283 100644
--- a/cgit.c
+++ b/cgit.c
@@ -459,7 +459,6 @@ 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;
@@ -467,17 +466,16 @@ 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(tmp);
+                cgit_print_error("Failed to open %s: %s", name,
+                                 rc ? strerror(rc) : "Not a valid git repository");
                 cgit_print_docend();
                 return 1;
         }
@@ -501,14 +499,14 @@ static int prepare_repo_cmd(struct cgit_context *ctx)
         }
 
         if (get_sha1(ctx->qry.head, sha1)) {
-                tmp = xstrdup(ctx->qry.head);
+                char *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(fmt("Invalid branch: %s", tmp));
+                cgit_print_error("Invalid branch: %s", tmp);
                 cgit_print_docend();
                 return 1;
         }
@@ -550,7 +548,7 @@ static void process_request(void *cbdata)
                 cgit_print_http_headers(ctx);
                 cgit_print_docstart(ctx);
                 cgit_print_pageheader(ctx);
-                cgit_print_error(fmt("No repository selected"));
+                cgit_print_error("No repository selected");
                 cgit_print_docend();
                 return;
         }
@@ -862,7 +860,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(fmt("Error processing page: %s (%d)",
-                                     strerror(err), err));
+                cgit_print_error("Error processing page: %s (%d)",
+                                 strerror(err), err);
         return err;
 }
diff --git a/ui-blob.c b/ui-blob.c
index 1da43c8..8f6989f 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(fmt("Bad hex value: %s", hex));
+                        cgit_print_error("Bad hex value: %s", hex);
                         return;
                 }
         } else {
                 if (get_sha1(head, sha1)) {
-                        cgit_print_error(fmt("Bad ref: %s", head));
+                        cgit_print_error("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(fmt("Bad object name: %s", hex));
+                cgit_print_error("Bad object name: %s", hex);
                 return;
         }
 
         buf = read_sha1_file(sha1, &type, &size);
         if (!buf) {
-                cgit_print_error(fmt("Error reading object %s", hex));
+                cgit_print_error("Error reading object %s", hex);
                 return;
         }
 
diff --git a/ui-commit.c b/ui-commit.c
index 8310ce6..6b41017 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(fmt("Bad object id: %s", hex));
+                cgit_print_error("Bad object id: %s", hex);
                 return;
         }
         commit = lookup_commit_reference(sha1);
         if (!commit) {
-                cgit_print_error(fmt("Bad commit reference: %s", hex));
+                cgit_print_error("Bad commit reference: %s", hex);
                 return;
         }
         info = cgit_parse_commit(commit);
diff --git a/ui-diff.c b/ui-diff.c
index 1115518..8b38209 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(fmt("Bad object name: %s", new_rev));
+                cgit_print_error("Bad object name: %s", new_rev);
                 return;
         }
         commit = lookup_commit_reference(new_rev_sha1);
         if (!commit || parse_commit(commit)) {
-                cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(new_rev_sha1)));
+                cgit_print_error("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(fmt("Bad object name: %s", sha1_to_hex(old_rev_sha1)));
+                        cgit_print_error("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(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1)));
+                        cgit_print_error("Bad commit: %s", sha1_to_hex(old_rev_sha1));
                         return;
                 }
         }
diff --git a/ui-patch.c b/ui-patch.c
index 66def3c..fbb92cc 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(fmt("Bad object id: %s", hex));
+                cgit_print_error("Bad object id: %s", hex);
                 return;
         }
         commit = lookup_commit_reference(sha1);
         if (!commit) {
-                cgit_print_error(fmt("Bad commit reference: %s", hex));
+                cgit_print_error("Bad commit reference: %s", hex);
                 return;
         }
         info = cgit_parse_commit(commit);
diff --git a/ui-shared.c b/ui-shared.c
index c1f3c20..b93b77a 100644
--- a/ui-shared.c
+++ b/ui-shared.c
@@ -28,10 +28,21 @@ static char *http_date(time_t t)
                    tm->tm_hour, tm->tm_min, tm->tm_sec);
 }
 
-void cgit_print_error(const char *msg)
+void cgit_print_error(const char *fmt, ...)
 {
+        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'>");
-        html_txt(msg);
+        va_copy(cp, ap);
+        html_vtxtf(fmt, cp);
+        va_end(cp);
         html("</div>\n");
 }
 
diff --git a/ui-shared.h b/ui-shared.h
index 5f8b629..5987e77 100644
--- a/ui-shared.h
+++ b/ui-shared.h
@@ -52,7 +52,10 @@ extern void cgit_object_link(struct object *obj);
 extern void cgit_submodule_link(const char *class, char *path,
                                 const char *rev);
 
-extern void cgit_print_error(const char *msg);
+__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_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 9be5dbe..a47884e 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(fmt("Bad object id: %s", hex));
+                cgit_print_error("Bad object id: %s", hex);
                 return 1;