logd: enforce line length limit for ubus based log messages as well
authorFelix Fietkau <nbd@nbd.name>
Mon, 6 Nov 2017 10:34:31 +0000 (11:34 +0100)
committerFelix Fietkau <nbd@nbd.name>
Mon, 6 Nov 2017 10:34:31 +0000 (11:34 +0100)
Signed-off-by: Felix Fietkau <nbd@nbd.name>
log/logd.c
log/syslog.c
log/syslog.h

index ee30984e69721769fe2d1bd11d9b3c032e058efd..9931723f6974fd35c156bc94345df0f858537f2b 100644 (file)
@@ -167,10 +167,18 @@ write_log(struct ubus_context *ctx, struct ubus_object *obj,
        char *event;
 
        if (msg) {
+               int len;
+
                blobmsg_parse(&write_policy, 1, &tb, blob_data(msg), blob_len(msg));
                if (tb) {
                        event = blobmsg_get_string(tb);
-                       log_add(event, strlen(event) + 1, SOURCE_SYSLOG);
+                       len = strlen(event) + 1;
+                       if (len > LOG_LINE_SIZE) {
+                               len = LOG_LINE_SIZE;
+                               event[len - 1] = 0;
+                       }
+
+                       log_add(event, len, SOURCE_SYSLOG);
                }
        }
 
index f1c7606d95fb37eca27792c912b9f511c69354c7..7e98d365e6ed085aed35c05f5ecb66dbc6d22af5 100644 (file)
@@ -37,8 +37,6 @@
 #define LOG_DEFAULT_SOCKET     "/dev/log"
 #define SYSLOG_PADDING         16
 
-#define MAXLINE                        1024
-
 #define KLOG_DEFAULT_PROC      "/proc/kmsg"
 
 #define PAD(x) (x % 4) ? (((x) - (x % 4)) + 4) : (x)
@@ -133,13 +131,13 @@ log_add(char *buf, int size, int source)
 static void
 syslog_handle_fd(struct uloop_fd *fd, unsigned int events)
 {
-       static char buf[MAXLINE];
+       static char buf[LOG_LINE_SIZE];
        int len;
 
        while (1) {
                char *c;
 
-               len = recv(fd->fd, buf, MAXLINE - 1, 0);
+               len = recv(fd->fd, buf, LOG_LINE_SIZE - 1, 0);
                if (len < 0) {
                        if (errno == EINTR)
                                continue;
index 81a039f3263e90b44e3568803e1d7aee58cfe386..56ccbb9d9020ef63b2e99e90b87b476db4888a4b 100644 (file)
@@ -14,6 +14,8 @@
 #ifndef __SYSLOG_H
 #define __SYSLOG_H
 
+#define LOG_LINE_SIZE          1024
+
 enum {
        SOURCE_KLOG = 0,
        SOURCE_SYSLOG = 1,