txtinfo: use httpStatusToReply in output_error
authorFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 14 Apr 2016 23:29:44 +0000 (01:29 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Fri, 15 Apr 2016 09:59:49 +0000 (11:59 +0200)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
lib/txtinfo/src/olsrd_txtinfo.c

index 4cc10db..3814334 100644 (file)
@@ -139,24 +139,18 @@ bool isCommand(const char *str, unsigned long long siw) {
   return !strcmp(str, cmd);
 }
 
-void output_error(struct autobuf *abuf, unsigned int status, const char * req, bool http_headers) {
+void output_error(struct autobuf *abuf, unsigned int status, const char * req __attribute__((unused)), bool http_headers) {
   if (http_headers || (status == INFO_HTTP_OK)) {
     return;
   }
 
-  switch (status) {
-    case INFO_HTTP_NOTFOUND:
-      abuf_appendf(abuf, "error: Invalid request '%s'\n", req);
-      break;
-
-    case INFO_HTTP_NOCONTENT:
-      /* wget can't handle output of zero length */
-      abuf_puts(abuf, "\n");
-      break;
+  /* !http_headers && !INFO_HTTP_OK */
 
-    default:
-      abuf_appendf(abuf, "error: Unknown status %d for request '%s'\n", status, req);
-      return;
+  if (status == INFO_HTTP_NOCONTENT) {
+    /* wget can't handle output of zero length */
+    abuf_puts(abuf, "\n");
+  } else {
+    abuf_appendf(abuf, "error: %s\n", httpStatusToReply(status));
   }
 }