Supply status description to html_status()

Signed-off-by: Lars Hjemli <hjemli@gmail.com>

4 files changed, 12 insertions, 14 deletions

diff --git a/html.c b/html.c
index 83fc7a9..36e9a2f 100644
--- a/html.c
+++ b/html.c
@@ -55,11 +55,11 @@ void htmlf(const char *format, ...)
         va_end(args);
         html(buf);
 }
 
-void html_status(int code, int more_headers)
+void html_status(int code, const char *msg, int more_headers)
 {
-        htmlf("Status: %d\n", code);
+        htmlf("Status: %d %s\n", code, msg);
         if (!more_headers)
                 html("\n");
 }
 
diff --git a/html.h b/html.h
index 49462a2..3c32935 100644
--- a/html.h
+++ b/html.h
@@ -5,9 +5,9 @@ extern int htmlfd;
 
 extern void html_raw(const char *txt, size_t size);
 extern void html(const char *txt);
 extern void htmlf(const char *format,...);
-extern void html_status(int code, int more_headers);
+extern void html_status(int code, const char *msg, int more_headers);
 extern void html_txt(char *txt);
 extern void html_ntxt(int len, char *txt);
 extern void html_attr(char *txt);
 extern void html_hidden(char *name, char *value);
diff --git a/ui-clone.c b/ui-clone.c
index 3a037ad..81e7a4e 100644
--- a/ui-clone.c
+++ b/ui-clone.c
@@ -47,22 +47,20 @@ static void print_pack_info(struct cgit_context *ctx)
 
 static void send_file(struct cgit_context *ctx, char *path)
 {
         struct stat st;
-        int err;
 
         if (stat(path, &st)) {
                 switch (errno) {
                 case ENOENT:
-                        err = 404;
+                        html_status(404, "Not found", 0);
                         break;
                 case EACCES:
-                        err = 403;
+                        html_status(403, "Forbidden", 0);
                         break;
                 default:
-                        err = 400;
+                        html_status(400, "Bad request", 0);
                 }
-                html_status(err, 0);
                 return;
         }
         ctx->page.mimetype = "application/octet-stream";
         ctx->page.filename = path;
@@ -85,9 +83,9 @@ void cgit_clone_info(struct cgit_context *ctx)
 
 void cgit_clone_objects(struct cgit_context *ctx)
 {
         if (!ctx->qry.path) {
-                html_status(400, 0);
+                html_status(400, "Bad request", 0);
                 return;
         }
 
         if (!strcmp(ctx->qry.path, "info/packs")) {
diff --git a/ui-plain.c b/ui-plain.c
index 28deae5..35888a0 100644
--- a/ui-plain.c
+++ b/ui-plain.c
@@ -21,15 +21,15 @@ static void print_object(const unsigned char *sha1, const char *path)
         size_t size;
 
         type = sha1_object_info(sha1, &size);
         if (type == OBJ_BAD) {
-                html_status(404, 0);
+                html_status(404, "Not found", 0);
                 return;
         }
 
         buf = read_sha1_file(sha1, &type, &size);
         if (!buf) {
-                html_status(404, 0);
+                html_status(404, "Not found", 0);
                 return;
         }
         ctx.page.mimetype = "text/plain";
         ctx.page.filename = fmt("%s", path);
@@ -65,18 +65,18 @@ void cgit_print_plain(struct cgit_context *ctx)
                 rev = ctx->qry.head;
 
         curr_rev = xstrdup(rev);
         if (get_sha1(rev, sha1)) {
-                html_status(404, 0);
+                html_status(404, "Not found", 0);
                 return;
         }
         commit = lookup_commit_reference(sha1);
         if (!commit || parse_commit(commit)) {
-                html_status(404, 0);
+                html_status(404, "Not found", 0);
                 return;
         }
         match_path = ctx->qry.path;
         fprintf(stderr, "[cgit] match_path=%s", match_path);
         read_tree_recursive(commit->tree, NULL, 0, 0, paths, walk_tree, NULL);
         if (!match)
-                html_status(404, 0);
+                html_status(404, "Not found", 0);
 }