This patch fixes a problem with the logread utility built into procd.
It adds a function to look up the correct priority and facility codes for 
output.

Tested on r37834.

It fixes Ticket #14079.

Signed-off-by: Mike Brady<mikebr...@eircom.net>


--- a/logread.c
+++ b/logread.c
@@ -60,6 +60,8 @@ static const char *log_file, *log_ip, *l
 static int log_type = LOG_STDOUT;
 static int log_size, log_udp;
 
+const char* getcodetext(int value, CODE *codetable);
+
 static void log_handle_reconnect(struct uloop_timeout *timeout)
 {
        sender.fd = usock((log_udp) ? (USOCK_UDP) : (USOCK_TCP), log_ip, 
log_port);
@@ -149,7 +151,7 @@ static int log_notify(struct ubus_contex
                }
        } else {
                snprintf(buf, sizeof(buf), "%s %s.%s%s %s\n",
-                       c, facilitynames[LOG_FAC(p)].c_name, 
prioritynames[LOG_PRI(p)].c_name,
+                       c, getcodetext(LOG_FAC(p)<<3,facilitynames), 
getcodetext(LOG_PRI(p),prioritynames),
                        (blobmsg_get_u32(tb[LOG_SOURCE])) ? ("") : (" kernel:"),
                        method);
                write(sender.fd, buf, strlen(buf));
@@ -217,6 +219,7 @@ enum {
        __READ_MAX
 };
 
+
 static const struct blobmsg_policy read_policy[] = {
        [READ_LINE] = { .name = "lines", .type = BLOBMSG_TYPE_ARRAY },
 };
@@ -252,7 +255,7 @@ static void read_cb(struct ubus_request
                c[strlen(c) - 1] = '\0';
 
                printf("%s %s.%s%s %s\n",
-                       c, facilitynames[LOG_FAC(p)].c_name, 
prioritynames[LOG_PRI(p)].c_name,
+                       c, getcodetext(LOG_FAC(p)<<3,facilitynames), 
getcodetext(LOG_PRI(p),prioritynames),
                        (blobmsg_get_u32(tb[LOG_SOURCE])) ? ("") : (" kernel:"),
                        blobmsg_get_string(tb[LOG_MSG]));
        }
@@ -339,3 +342,13 @@ int main(int argc, char **argv)
 
        return 0;
 }
+
+const char* getcodetext(int value, CODE *codetable) {
+       CODE *i;
+       if (value >= 0)
+               for (i = codetable; i->c_val != -1; i++)
+                       if (i->c_val == value)
+                               return (i->c_name);
+       return ("<unknown>");
+};
+
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to