syslog() is a blocking call on eglibc. as procd provides the actual syslog, weneed...
authorJohn Crispin <blogic@openwrt.org>
Sat, 28 Sep 2013 19:01:20 +0000 (21:01 +0200)
committerJohn Crispin <blogic@openwrt.org>
Sat, 28 Sep 2013 19:47:43 +0000 (21:47 +0200)
Signed-off-by: John Crispin <blogic@openwrt.org>
Signed-off-by: Markus Stenberg <markus.stenberg@iki.fi>
procd.h
rcS.c
syslog.c
syslog.h

diff --git a/procd.h b/procd.h
index f7a333e79ed0af0aab111c50af0629fac4d1cc66..34cc9a1beb3e6570eb5aa356fa525603053e6833 100644 (file)
--- a/procd.h
+++ b/procd.h
                fprintf(stderr, "procd: %s(%d): " fmt, __func__, __LINE__, ## __VA_ARGS__); \
        } while (0)
 
-#define SYSLOG(p, fmt, ...) do { \
-       syslog(p, fmt, ## __VA_ARGS__); \
-       } while (0)
-
 #define LOG(fmt, ...) do { \
-       syslog(LOG_INFO, fmt, ## __VA_ARGS__); \
+       log_printf(fmt, ## __VA_ARGS__); \
        fprintf(stderr, "procd: "fmt, ## __VA_ARGS__); \
        } while (0)
 
 #define ERROR(fmt, ...) do { \
-       syslog(LOG_ERR, fmt, ## __VA_ARGS__); \
+       log_printf(fmt, ## __VA_ARGS__); \
        fprintf(stderr, "procd: "fmt, ## __VA_ARGS__); \
        } while (0)
 
diff --git a/rcS.c b/rcS.c
index 774c77e1aea51b9f71bff1bb8d372b09925526dc..f041f09a32db81203619f8945e93679a78e6ef69 100644 (file)
--- a/rcS.c
+++ b/rcS.c
@@ -55,7 +55,7 @@ static void pipe_cb(struct ustream *s, int bytes)
                        break;
                *newline = 0;
                len = newline + 1 - str;
-               SYSLOG(6, buf->data);
+               log_printf(buf->data);
                ustream_consume(s, len);
        } while (1);
 }
index 01b6e7ec8472f4b003f40acf1e06b4e0c318799b..dbb7a5a57a996317fa8e05559f63bd52e5422e26 100644 (file)
--- a/syslog.c
+++ b/syslog.c
@@ -63,6 +63,12 @@ void log_add(char *buf, int size, int source)
        int priority = 0;
        int ret;
 
+       /* bounce out if we don't have init'ed yet (regmatch etc will blow) */
+       if (!log) {
+               fprintf(stderr, buf);
+               return;
+       }
+
        /* strip trailing newline */
        if (buf[size - 2] == '\n') {
                buf[size - 2] = '\0';
@@ -121,6 +127,34 @@ void log_add(char *buf, int size, int source)
        newest = next;
 }
 
+void log_printf(char *fmt, ...)
+{
+       static int buffer_len = 128;
+       static char *buffer;
+       va_list ap;
+       int n = 0;
+
+       do {
+               if (n)
+                       buffer_len = n + 1;
+               if (!buffer)
+                       buffer = malloc(buffer_len);
+               if (!buffer)
+                       return;
+               va_start(ap, fmt);
+               n = vsnprintf(buffer, sizeof(buffer), fmt, ap);
+               va_end(ap);
+               if (n < 1)
+                       return;
+               if (n >= buffer_len) {
+                       free(buffer);
+                       buffer = NULL;
+               }
+       } while (n >= buffer_len);
+
+       log_add(buffer, n, SOURCE_INTERNAL);
+}
+
 static void slog_cb(struct ustream *s, int bytes)
 {
        struct ustream_buf *buf = s->r.head;
index b2471eb20fc0b62d006b817c05a81813b3f9a19a..fd78363d9a9fc3ce4badb93cd50f8033265bfb42 100644 (file)
--- a/syslog.h
+++ b/syslog.h
@@ -18,6 +18,7 @@
 enum {
        SOURCE_KLOG = 0,
        SOURCE_SYSLOG = 1,
+       SOURCE_INTERNAL = 2,
        SOURCE_ANY = 0xff,
 };
 
@@ -37,5 +38,6 @@ typedef void (*log_list_cb)(struct log_head *h);
 struct log_head* log_list(int count, struct log_head *h);
 int log_buffer_init(int size);
 void log_add(char *buf, int size, int source);
+void log_printf(char *fmt, ...);
 
 #endif