snapshot: don't reimplement cgit_print_error_page()

cgit_print_error_page() has the advantage that it sets a suitable HTTP
status code for the response.  Note that setting "mimetype" is redundant
here since it cannot have changed since being initialized in
cgit.c::prepare_context(), so we do not need to worry that
cgit_print_error_page() does not set it.

Signed-off-by: John Keeping <john@keeping.me.uk>
This commit is contained in:
John Keeping 2015-08-14 12:47:09 +01:00 committed by Jason A. Donenfeld
parent 58e827cbd9
commit fd00e71ab7

View File

@ -180,21 +180,6 @@ out:
return result ? strbuf_detach(&snapshot, NULL) : NULL; return result ? strbuf_detach(&snapshot, NULL) : NULL;
} }
__attribute__((format (printf, 1, 2)))
static void show_error(char *fmt, ...)
{
va_list ap;
ctx.page.mimetype = "text/html";
cgit_print_http_headers();
cgit_print_docstart();
cgit_print_pageheader();
va_start(ap, fmt);
cgit_vprint_error(fmt, ap);
va_end(ap);
cgit_print_docend();
}
void cgit_print_snapshot(const char *head, const char *hex, void cgit_print_snapshot(const char *head, const char *hex,
const char *filename, int dwim) const char *filename, int dwim)
{ {
@ -202,13 +187,15 @@ void cgit_print_snapshot(const char *head, const char *hex,
char *prefix = NULL; char *prefix = NULL;
if (!filename) { if (!filename) {
show_error("No snapshot name specified"); cgit_print_error_page(400, "Bad request",
"No snapshot name specified");
return; return;
} }
f = get_format(filename); f = get_format(filename);
if (!f) { if (!f) {
show_error("Unsupported snapshot format: %s", filename); cgit_print_error_page(400, "Bad request",
"Unsupported snapshot format: %s", filename);
return; return;
} }